Funds Go Back Option Takes User Back to the First Fund Screen

Bug #1287370 reported by Erica Rohlfs
26
This bug affects 5 people
Affects Status Importance Assigned to Milestone
Evergreen
Fix Released
Low
Unassigned
2.6
Fix Released
Undecided
Unassigned
2.7
Fix Released
Undecided
Unassigned

Bug Description

Evergreen Version 2.4.6 and 2.5.2

When users select a Fund (Admin -> Server Administration -> Acquisitions -> Funds -> then select a fund) and are ready to go back to their Funds listing, they can select the Go Back button. Selecting the Go Back button takes the users too far back. It takes them to the initial Funds page. So, if the user has selected a Context Org Unit and / or Year, that information is not retained. The user must start over again from the beginning.

Instead of returning to the beginning Fund screen, the Go Back option should return the user back to the last results page that he just navigated away from to view a selected fund.

Thank you!

Erica Rohlfs (erohlfs)
description: updated
Kathy Lussier (klussier)
Changed in evergreen:
status: New → Confirmed
importance: Undecided → Low
Revision history for this message
Kathy Lussier (klussier) wrote :

Just adding a note that the same thing happens on acq search screens. If you retrieve a multi-page list of results, click to view a PO/invoice/selection list, and then click the Go Back button, the user is brought back to the initial funds page. IMO, this makes the bug even more troublesome as it gets in the way of everyday workflow.

Galen Charlton (gmc)
Changed in evergreen:
assignee: nobody → Galen Charlton (gmc)
milestone: none → 2.next
Revision history for this message
Galen Charlton (gmc) wrote :

Three patches for the fund search page is available at the tip of the user/gmcharlt/lp1287370_fund_search_filter_offset_persist branch in the working/Evergreen repository:

http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/gmcharlt/lp1287370_fund_search_filter_offset_persist

The AutoGrid work establishes a general mechanism that could be applied to other forms; separate bugs should probably be opened for this.

tags: added: pullrequest
Changed in evergreen:
assignee: Galen Charlton (gmc) → nobody
Galen Charlton (gmc)
Changed in evergreen:
milestone: 2.next → 2.8-beta
Revision history for this message
Martha Crawley (mcrawley) wrote :

Enhancement OK'd by Cumberland.

Revision history for this message
Erica Rohlfs (erohlfs) wrote :

Tested as part of Bug Squashing Day (March 2, 2015). The patch is working for me on the test database. Clicking Go Back will take me back to my previous fund screen, where I've selected a different Context Org Unit and Year.

Revision history for this message
Erica Rohlfs (erohlfs) wrote :

I have tested this code and consent to signing off on it with my name, Erica Rohlfs, and my email address, <email address hidden>

Revision history for this message
Mike Rylander (mrylander) wrote :
tags: added: signedoff
Revision history for this message
Ben Shum (bshum) wrote :

Thanks everyone, pushing the changes to master, rel_2_7, and rel_2_6.

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.