Vandelay inspect queue does not list Bib Queues on initial load

Bug #1010636 reported by Bill Erickson
16
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Evergreen
Fix Released
Medium
Unassigned

Bug Description

Confirmed in master, but I suspect it's the same in 2.2.

In Vandelay, choosing the "Inspect Queue" option defaults to Bibliograph Records as the queue type. However, the Queue selector remains empty, even when there are bib queues in the system. Selecting a different queue type (e.g. Authority) then returning to Bibliographic Records does populate the Queue drop down.

Bill Erickson (berick)
Changed in evergreen:
assignee: nobody → Bill Erickson (erickson-esilibrary)
status: New → In Progress
Revision history for this message
Bill Erickson (berick) wrote :

Fix pushed to working => user/berick/lp1010636-vandelay-inspect-queue-load-default

Should be applied to master and 2.2.

http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/berick/lp1010636-vandelay-inspect-queue-load-default

tags: added: pullrequest
Changed in evergreen:
assignee: Bill Erickson (erickson-esilibrary) → nobody
status: In Progress → Confirmed
milestone: none → 2.2.0
Revision history for this message
Thomas Berezansky (tsbere) wrote :
Revision history for this message
Bill Erickson (berick) wrote :

The ticket Thomas linked includes a fix for this among other things. However, since Thomas' branch is considerably more than just a bug fix, I'm not sure how 2.2 back-portable it is. Opinions?

tags: removed: pullrequest
Revision history for this message
Mike Rylander (mrylander) wrote :

My vote is to merge Bill's, since Thomas' hasn't gotten a sign-off yet, and is indeed much bigger. I say that without any tuits to apply toward reviewing Thomas' branch, however.

Revision history for this message
Bob Wicksall (bwicksall) wrote :

Bill's fix seems to solve the problem on my 2.2 RC1 server.

Revision history for this message
Mike Rylander (mrylander) wrote :

Sorry, mea culpa!

Thomas' /does/ have a signoff from Mike Peters. My vote is still for merging the smaller patch now, with a rebase of the bigger one for master only, however. That has the benefit of keeping things less confusing and more linear -- they get the same fix for this bug, and new stuff only goes into master.

Changed in evergreen:
importance: Undecided → Medium
Revision history for this message
Lebbeous Fogle-Weekley (lebbeous) wrote :

I have to agree with favoring a simpler solution for an imminent release. Will test Thomas' latest approach on bug #959412 for master.

Bill's branch is now in master and rel_2_2

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.

Duplicates of this bug

Other bug subscribers

Remote bug watches

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