Include call number prefix/suffix in catalog checked out items

Bug #1642000 reported by Josh Stompro
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Evergreen
Fix Released
Undecided
Unassigned

Bug Description

EG 2.10.6

Currently only the call number label is displayed in the checked out item display in myopac.

The call number prefix/suffix can help customers identify what type of item they have checked out in some situations.

For our purposes we would want the prefix/suffix included in the sort, but I know that other locations don't feel that way, so if i was going to add this I wouldn't include it in the sort, I would just add it to the display when the prefix/suffix exist.

This enhancement may also make sense for the checkout history and holds/holds history displays.

Josh

Revision history for this message
Josh Stompro (u-launchpad-stompro-org) wrote :

I have this working for the 3.3 opac, along with showing parts.

I changed the sort so that it uses the label_sortkey values for the call number prefix, call number, call number suffix and parts.

Here is a working branch
user/stompro/lp1642000_opac_checkout_cn_prefix_suffix_parts

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

tags: added: pullrequest
Revision history for this message
Josh Stompro (u-launchpad-stompro-org) wrote :

Updated the previous commit with updates for the bootstrap opac, but I haven't tested it.
Josh

Revision history for this message
Josh Stompro (u-launchpad-stompro-org) wrote :

I've tested this out with the circulation history display, and it is very similar to add.

Using production data with 500 records in my history, the sorting by call number is too slow to test on my test system. The sorting is done in the TT2 template, and I think it just takes too long with larger numbers of history items. But I think there are other bug reports about that.

I pushed a few more commits to the above branch to add the prefix,suffix and part to the circ history and bootstrab circ history.

Josh

Revision history for this message
Elaine Hardy (ehardy) wrote :

Using https://terran-master.gapines.org/eg/staff/, I have tested this code and consent to signing off on it with my name, Elaine Hardy and my email address, <email address hidden>

Revision history for this message
Chris Sharp (chrissharp123) wrote :

Pushed to master. Thanks, Josh and Elaine!

tags: added: signedoff
Changed in evergreen:
status: New → Fix Committed
Changed in evergreen:
milestone: none → 3.8-beta
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.