Code comments and tweaks for lost processing collab/dyrcona/lp851000
authorDan Wells <dbw2@calvin.edu>
Mon, 5 Dec 2011 16:44:41 +0000 (11:44 -0500)
committerJason Stephenson <jstephenson@mvlc.org>
Mon, 5 Dec 2011 19:30:00 +0000 (14:30 -0500)
commit41ffc994f3c1ced2982f6dd4793b30c1e34312e8
treed3f5a0e46d023cec7a39f0f3c55bc365bc73b7da
parent314c5ef6fa1b41d424a4f8ae38d39d83f874fb2f
Code comments and tweaks for lost processing

This commit is intended to do the following:

* Fix case where MISSING status is not retained for items
  being sent 'home' (as it was in previous versions)
* Be more explicit about when we unset the LOST status and when
  we do not in checkin_handle_lost()
* Prevent update of the copy in checkin_handle_lost() unless we
  actually change the status
* Restore log messages for special handling of LOST/MISSING
  checkins away from 'home'
* Provide additional code comments to clarify intended behavior

Note: Given the current restructure, MISSING item behavior is no
longer affected by 'lost_immediately_available' setting.  That
seemed more bug than feature, and can be implemented later
(perhaps as a separate setting) if required.

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