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, 15 Apr 2015 14:34:36 +0000 (10:34 -0400)
commit156186d7ac8b4f3016142d686b135a9bdbb03e4d
treeccce11adcdbf9bffdf36638af41033459dc336ef
parentdc41355a7000407365350f1719c723ebc38c714d
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