Local Catalog should be at top of Z39.50 target list

Bug #1787676 reported by Jeff Davis
18
This bug affects 3 people
Affects Status Importance Assigned to Milestone
Evergreen
Fix Released
Low
Unassigned
3.1
Fix Released
Low
Unassigned
3.2
Fix Released
Low
Unassigned

Bug Description

EG 3.1.2+

The fix for bug 1119370 corrected an issue with sorting of Z39.50 sources in the web client. However, as a side-effect of the fix, the "Local Catalog" entry ended up at the bottom of the list, whereas it had previously been at the top.

We've had a request here to restore Local Catalog to its original position at the top of the list. I have a patch for this which I'll share momentarily.

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

Working branch user/jeffdavis/lp1787676-z3950-sort-local-catalog has a commit which reverts the previous fix and sorts Z39.50 sources in JS instead, allowing us to keep Local Catalog at the top of the list.

http://git.evergreen-ils.org/?p=working/Evergreen.git;a=commitdiff;h=e4bcdfa1

tags: added: pullrequest z3950
Changed in evergreen:
milestone: none → 3.1.5
importance: Undecided → Low
Changed in evergreen:
milestone: 3.1.5 → 3.1.6
Changed in evergreen:
milestone: 3.1.6 → 3.2.1
Changed in evergreen:
milestone: 3.2.1 → 3.2.2
Changed in evergreen:
milestone: 3.2.2 → 3.2.3
Revision history for this message
Jane Sandberg (sandbergja) wrote :

Works well for me. Thanks, Jeff! I created a rebased signoff branch called user/sandbergja/lp1787676-z3950-sort-local-catalog-signedoff

tags: added: signedoff
Changed in evergreen:
milestone: 3.2.3 → 3.3-beta1
Changed in evergreen:
milestone: 3.3-beta1 → 3.3-rc
Changed in evergreen:
milestone: 3.3-rc → 3.3.1
Changed in evergreen:
status: New → Confirmed
Changed in evergreen:
milestone: 3.3.1 → 3.3.2
Changed in evergreen:
milestone: 3.3.2 → 3.3.3
Revision history for this message
Galen Charlton (gmc) wrote :

Pushed to master, rel_3_3, rel_3_2, and rel_3_1. Thanks, Jeff and Jane!

Changed in evergreen:
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.

Other bug subscribers

Remote bug watches

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