Circ modifier column appears twice in OPAC view

Bug #1930308 reported by Jeff Davis
14
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Evergreen
Fix Released
Medium
Unassigned

Bug Description

When you do a traditional catalog search in the client and view a record in OPAC View, the circ modifier column appears twice in the copy table. The Circulation Modifier column heading only appears once, so the remaining columns are all off by one.

The same issue appears in the Patron View tab in the Angular catalog, but only after you have done a traditional catalog search (otherwise the circ modifier column is hidden).

Revision history for this message
Jeff Davis (jdavis-sitka) wrote :

Here's a screenshot.

Revision history for this message
Jeff Davis (jdavis-sitka) wrote :

This only affects sites where OPAC view/patron view uses TPAC. Bootstrap OPAC is unaffected.

Revision history for this message
Jeff Davis (jdavis-sitka) wrote :

Working branch user/jeffdavis/lp1930308-duplicate-circ-modifier-column removes the extra circ_modifier data cell, which snuck in during an unrelated update:

https://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/jeffdavis/lp1930308-duplicate-circ-modifier-column

To test:

1. Ensure your test environment is using TPAC.
2. Do a traditional catalog search and click on a record with holdings. Circ modifiers (if any) will appear twice in the copy table, and remaining fields in each row are incorrectly offset to the right.
3. Apply the fix.
4. Repeat step 2. The duplicate circ modifier field is now gone, and remaining fields are in the right place.

tags: added: pullrequest
Changed in evergreen:
assignee: nobody → Josh Stompro (u-launchpad-stompro-org)
Revision history for this message
Josh Stompro (u-launchpad-stompro-org) wrote :

Tested this on Terran's Test Server 1 in TPAC mode.

Everything looks correct after following the testing steps. Only one Circ Modifier column is shown.

I tried the angular catalog, patron view also and it looks fine also.

I'll work on a signoff branch as soon as I can.
Josh

tags: added: signedoff
Changed in evergreen:
assignee: Josh Stompro (u-launchpad-stompro-org) → nobody
Revision history for this message
Josh Stompro (u-launchpad-stompro-org) wrote :
Revision history for this message
Galen Charlton (gmc) wrote :

Pushed to master and rel_3_7. Thanks, Jeff and Josh!

Changed in evergreen:
milestone: none → 3.7.2
importance: Undecided → Medium
status: New → Confirmed
status: Confirmed → Fix Committed
Changed in evergreen:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

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