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>
Wed, 5 Mar 2014 22:21:21 +0000 (17:21 -0500)
commit8ea8f03896f1d97513393ef6cd2f94f9b359f31b
treea48a37b0e873f3934bb607cbe36be5280a15f1c8
parentedea292e878866120f4d419b07cd7a3538d143f1
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