Comment 2 for bug 803512

Revision history for this message
Bill Erickson (berick) wrote :

Minor style/consistency suggestion... cuddle the else's. For example

    if ($is_renew) {
        $method = 'open-ils.circ.renew';
    }
    else {
        $method = 'open-ils.circ.checkout.full';
    }

---
    if ($is_renew) {
        $method = 'open-ils.circ.renew';
    } else {
        $method = 'open-ils.circ.checkout.full';
    }
---

With that and a squash, I think the branch is ready to go.