LP#1078593 Assorted small Serial.pm fixes
authorDan Wells <dbw2@calvin.edu>
Tue, 8 Apr 2014 20:11:31 +0000 (16:11 -0400)
committerBen Shum <bshum@biblio.org>
Tue, 3 Feb 2015 21:05:05 +0000 (16:05 -0500)
commitbbf4fe9aba76779cc2c8225c906d2a46a20deee5
tree14ed23baebecdfd89b15c93de2d52ce619da7f04
parent3753e262f00aa0fe45ee62586a6f1b6ca096d640
LP#1078593 Assorted small Serial.pm fixes

1) fleshed_issuance_alter() changes
  - First, if one of the inner updates returns an event, abort early.
    Otherwise, we might overwrite the event while looping and lose
    the error.
  - Second, add authtoken to the created editor.  This editor gets
    passed around, and other functions might look for it there.

2) pass $type down into _summarize_contents()
  - Right now, we have a bug where supplement/index contents in a
    merged MFHD record will get added to the basic contents (and
    vice-(vice-)versa).  By passing in $type, we can assure that
    we only return the type of contents we are looking for.

3) _prepare_summaries() should acknowledge empty summaries
  - Under normal use, summaries will generally grow, and never shrink
    to nothing.  However, we must still handle cases where we have
    discarded/deleted our last issuance, and in those cases, make sure
    we "empty" the generated_coverage field in our summary.

Signed-off-by: Dan Wells <dbw2@calvin.edu>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Ben Shum <bshum@biblio.org>
Open-ILS/src/perlmods/lib/OpenILS/Application/Serial.pm