OPAC - Editing holds causes display issue in holds tab

[Mary Jinglewski]

This affects 3.0.0 through 3.0.3 of Evergreen. Behavior observed in both Chrome and Firefox.

When logged in as a patron in the OPAC, a display issue occurs, which restricts the listed holds in My Account > Holds to just one hold based on hold ID number. The workflow to cause this involves being in the holds tab of My Account (aka Holds display), editing the hold (via the Edit link), and when in the editing hold details screen, clicking on the holds tab (to return to the Holds display in theory). After that point, the Holds tab display will only show that hold when there is supposed to be a longer list of holds.

After this workflow occurs, the URL is also changed. e.g. [shortened URL]/eg/opac/myopac/holds?id=4468265

If you navigate away from this Holds screen in the My Account, the URL reflects the expected URL within the OPAC (including outside of the My Account area) but has the id number appended. e.g. /eg/opac/myopac/prefs?id=4468265

The only way to remove this singular hold view is to remove the ?id=##### from the URL manually.

Reported by Jason Goodson and confirmed by me, Mary Jinglewski.

[Cesar V (cesardv)]

Nice.

[Cesar V (cesardv)]

A fix for this is here:

http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/cesardv/lp1746815_fix_myopac_hold_edit_urlparam

[John Amundson (jamundson)]

We also had a library report this, and I was able to duplicate the issue using the steps outlined above by Mary.

We are on Evergreen 2.12.6. Tested in Chrome.

[Michele Morgan (mmorgan)]

This works as advertised. While logged into the opac, clicking on Edit for a hold, then navigating away from the edit hold screen, the hold id is no longer retained on the url.

My signoff branch is at:

http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/mmorgan/lp1746815_fix_myopac_hold_edit_urlparam_signoff

[Galen Charlton (gmc)]

Pushed to master, rel_3_1, and rel_3_0. Thanks, Jason, Mary, Cesar, and Michele!