LP 1198465: Fix and improve void/adjustment code
authorDan Wells <dbw2@calvin.edu>
Wed, 26 Feb 2014 22:44:45 +0000 (17:44 -0500)
committerDan Wells <dbw2@calvin.edu>
Tue, 11 Nov 2014 20:30:02 +0000 (15:30 -0500)
commit8e9bac8ef393aa4f1343ec618c9ec4dc453ef025
tree458df8c12e833d65c6ba1cdb5efef7ee6a7f69ed
parent12cbb437b7d2b2f4f652dfac0b950eedbe8a9b03
LP 1198465: Fix and improve void/adjustment code

This commit does three things:

- Replace ou_ancestor_setting() with ou_ancestor_setting_value() calls
  This also fixed a bug where we were expecting just the setting, not
  a HASH

- Reword interval checking
  This fix is two part.  First, we simplify the check to not require
  the whole payment map.  Second, we use this newfound simplicity to
  push this check up into the gatekeeper functions, further clarifying
  the code paths.

- make $note into $for_note for void_or_zero_bills_of_type()
  Because the function can both void and adjust, we can't supply a
  complete note, so let's just supply text of what the void/adjustment
  is for.

Signed-off-by: Dan Wells <dbw2@calvin.edu>
Open-ILS/src/perlmods/lib/OpenILS/Application/Circ/CircCommon.pm
Open-ILS/src/perlmods/lib/OpenILS/Application/Circ/Circulate.pm