Item Status - Alternate View - Holds/Transit tab: Transit and Hold information does not refresh

[Michele Morgan]

With multiple items listed on the Item Status list, information on the Alternate View -> Holds/Transit tab does not always refresh when viewing a different item. Some information from an Available item will persists when subsequently viewing an In Transit item, and vice versa.

The attached .doc file shows an example and screenshots illustrating the information that is not refreshed.

Related bugs are:

Item Status Alternate View Residual Times Data
https://bugs.launchpad.net/evergreen/+bug/1312703

and

Item Details->Alt View->Hold/Transit tab is showing two transit lists rather than a hold list and a transit list
https://bugs.launchpad.net/evergreen/+bug/949101

[Michele Morgan (mmorgan)]

Attaching the .doc

[Erica Rohlfs (erohlfs)]

Confirmed on version 2.6.0. The Item Status screen lacks a way for the user to select a Refresh or Reload Page, as well (unless I'm completely over looking it).

[Josh Stompro (u-launchpad-stompro-org)]

This is still an issue in 2.10.6 also. This issue also happens between tabs. I opened an item in items status alternative view and looked at the Holds/Transit info, that item was in transit to fill a hold.

I then opened a new tab and looked at a new item that was on the holdshelf. That item showed the same transit info in the data list as the first item.

Josh

[Kathy Lussier (klussier)]

Has anyone checked to see if this is an issue in the web client?

[Josh Stompro (u-launchpad-stompro-org)]

I found a fix for this. (I know it is tacky to work on XUL on web client day... but this was bugging me.)

Also, my statement about it happening between tabs is wrong, I just lost track of where I had loaded some records.

I just noticed that in alternate_copy_summary.js, where there was hold and transit data, a hold_list.clear() and transit_list.clear() function is called, but when there is no hold or transit data, the list isn't touched. Adding the clear functions to the case where there is no hold or transit data clears out the old data from the previous view.

I also added a call to set the hold_patron_name label value to blank. That clears out the patron name when it is already set. That had the side effect of adding an extra space between the transit label and the list though. I need to find a better way to clear that.

Branch on the way.
Josh

[Josh Stompro (u-launchpad-stompro-org)]

Working Branch at
user/stompro/lp1657861_alt_item_status_hold_transit

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

I needed to add two strings to staff_client/server/locale/en_US/circ.properties. Please let me know if there is something else that needs to be done when adding translatable strings.

Josh

[Jennifer Pringle (jpringle-u)]

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

[Jennifer Pringle (jpringle-u)]

I briefly tested this issue a 2.12 web client (without the fix) and the sandbox with the fix and I didn't see this issue so it may not have ever been an issue in the web client.

[Kathy Lussier (klussier)]

Thank you Josh and Jennifer!

Since there are two new strings in this fix, I'll leave it to the release 2.11 maintainer to decide if the fix should be backported to 2.11.

I've merged the code to master and release 2.12.