Holds history displays canceled holds as active

[Kathy Lussier]

When a patron is tracking holds history, canceled holds display as active. From the patron point of view, the hold is no longer active after it has been canceled. Ideally, it would be nice to display canceled here, but inactive would do too.

[Terran McCanna (tmccanna)]

I've submitted a suggested fix for this:

http://git.evergreen-ils.org/?p=working/Evergreen.git;a=commit;h=0726f21ecca8ec80afecde14def6d4885ae3563e

To test, make sure a test patron's hold history settings are enabled. Then, set a number of holds on the account and then suspend some of them and cancel some of them. Check the patron's hold history page to be sure the statuses are appearing correctly.

[Erica Rohlfs (erohlfs)]

Looks good. Under the "Active" column, the suspended holds read "Suspended" and the canceled holds read "Canceled."

I have tested this code and consent to signing off on it with my name, [Erica Rohlfs] and email address, [<email address hidden>].

[Kathy Lussier (klussier)]

This looks great Terran! I signed off on your commit and added Erica's signoff to the commit too.

When I was looking at it, I realized that the fulfilled holds also should not display as active since a user would consider those holds to be complete. I added a commit on top of the branch to change those holds to display as fulfilled. In the Active column, then, we would have active holds and three different types of holds users may consider to be no longer active: suspended, canceled and fulfilled.

While I was there, I also changed the "Active On" column header to "Activate" to make it more consistent with the label used in the regular holds interface. I also though the label was a better reflection of what is displaying in that column. New working branch at http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/kmlussier/lp1275118_myopac_canceled_holds

Thank you Terran and Erica!

[Galen Charlton (gmc)]

I've pushed Terran's patch. Thanks!

I did not push Kathy's follow-up, as it introduces a template syntax error that it turns leads to an internal server error:

 egweb: template error: file error - parse error - opac/myopac/hold_history.tt2 line 105-111: unexpected token (l)\n [% IF ahr.fulfillment_time;\n l('Fulfilled');\n ELSIF hold.hold.status == 6;\n l('Canceled'); \n ELSE \n l(ahr.frozen == 'f' ? 'Active' : 'Suspended');\n

I agree with Kathy about not displaying fulfilled holds as active, but please open a new bug for that and rework that follow-up patch.