LP#1078593 Assorted small Serial.pm fixes
authorDan Wells <dbw2@calvin.edu>
Tue, 8 Apr 2014 20:11:31 +0000 (16:11 -0400)
committerDan Wells <dbw2@calvin.edu>
Tue, 8 Apr 2014 20:51:37 +0000 (16:51 -0400)
commitaa50fea35da2ba0e420d222d1ca1ec46274f3d77
tree132933c2ed1dd272bf489b6bd3bff55dda434205
parentf23274d750213f8d9332ee5edc443f70b432b1de
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>
Open-ILS/src/perlmods/lib/OpenILS/Application/Serial.pm