Inconsistent display of Patron Search screen

Bug #975422 reported by Angela Kilsdonk
14
This bug affects 3 people
Affects Status Importance Assigned to Milestone
Evergreen
Status tracked in Main
2.1
Fix Released
Undecided
Unassigned
Main
Fix Released
Low
Unassigned

Bug Description

Evergreen 2.1.1

Clicking on “Search for Patron by Name” on the default/portal screen in the staff client results in a vertical display of the Patron Search screen, even if the Library Setting: “Patron circulation summary is horizontal” is set to True. The display of this screen should be consistent throughout the staff client.

Tags: pullrequest
Revision history for this message
Ben Shum (bshum) wrote :

Confirmed, we noticed this during our testing but didn't get to finish figuring out the problem. We took the "easy" way out and just removed that option from the portal page in our production build.

Changed in evergreen:
status: New → Confirmed
importance: Undecided → Low
milestone: none → 2.2.0beta2
Revision history for this message
Jason Etheridge (phasefx) wrote : Re: [Bug 975422] Re: Inconsistent display of Patron Search screen

> Confirmed, we noticed this during our testing but didn't get to finish
> figuring out the problem.  We took the "easy" way out and just removed
> that option from the portal page in our production build.

It's probably using xulG.new_tab/xulG.set_tab instead of
xulG.new_patron_tab/xulG.set_patron_tab.

The patron tab functions don't take the first URL param from the
regular _tab functions.

Revision history for this message
Thomas Berezansky (tsbere) wrote :

The find_patron function in the portal page loads the vertical interface without checking the setting. It would need to be updated to check the setting and load the horizontal one where appropriate.

For a "you use only the horizontal one" quick-fix you could quickly hack in "use just the horizontal version" into that function, at least until someone makes it check the setting.

Changed in evergreen:
milestone: 2.2.0beta2 → 2.2.0rc1
Changed in evergreen:
assignee: nobody → Jason Etheridge (phasefx)
Revision history for this message
Jason Etheridge (phasefx) wrote :

collab/phasefx/portal_patron_search @ working/Evergreen.git
http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/collab/phasefx/portal_patron_search

Based off master but should be suitable for 2.1 as well

tags: added: pullrequest
no longer affects: evergreen/2.2
Revision history for this message
Dan Scott (denials) wrote :

Confirmed that Jason's branch fixes what it said it would fix, signed off and pushed to master, rel_2_2, and rel_2_1.

Also experienced the horizontal patron display for the first time.... interesting.

Thanks Jason!

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.