Comment 11 for bug 1350042

Here's a branch that, while not changing behavior, removes some magic ("return last lvalue") in case the method becomes more complex in the future and breaks the contract with Perl that it currently depends on. It also improves readability and maintainability, IMO. It's the same as user/berick/web-staff-phase1-squash, but with one commit on top to use explicit "return"s.

http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/collab/miker/web-staff-phase1-squash-with-returns