Acq: Year Drop Down Doesn't Sort Chronologically

Bug #1586567 reported by Jennifer Pringle
12
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Evergreen
Low
Unassigned
2.10
Low
Unassigned
2.11
Low
Unassigned

Bug Description

Evergreen 2.8 and Evergreen 2.10.2

The drop down menu for Year in the Funds interface (Admin -> Acquisitions Administration -> Funds) and for Fiscal Year in the loading MARC records interface (Acquisitions -> Load MARC Order Records) do not list the years chronologically. See attached screenshots.

Ideally the years should be listed chronologically with the most recent year at the top.

Revision history for this message
Jennifer Pringle (jpringle-u) wrote :
Revision history for this message
Jennifer Pringle (jpringle-u) wrote :
Revision history for this message
Dawn Dale (ddale) wrote :

Confirmed in 2.9.1

Changed in evergreen:
status: New → Confirmed
Revision history for this message
Remington Steed (rjs7) wrote :

Here is a branch that fixes the year sorting issue for both of the mentioned Acq interfaces by fixing the underlying perl function:

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

To test, apply the patch and restart the open-ils.acq service.

Does a change like this require a perl test?

tags: added: pullrequest
Remington Steed (rjs7)
Changed in evergreen:
milestone: none → 2.next
importance: Undecided → Low
Dawn Dale (ddale)
Changed in evergreen:
assignee: nobody → Dawn Dale (ddale)
Dawn Dale (ddale)
Changed in evergreen:
assignee: Dawn Dale (ddale) → nobody
tags: added: signedoff
Revision history for this message
Dawn Dale (ddale) wrote :

"I have tested this code and consent to signing off on it with my name, [Dawn Dale] and my email address, <email address hidden>.

Revision history for this message
Kathy Lussier (klussier) wrote :

Thank you Remington and Dawn. In answer to your earlier question, Remington, given that the change is small and that we have three signoffs for the code, I think we can go without the perl test for this one. Merged to master, release 2.11 and 2.10.

Changed in evergreen:
status: Confirmed → Fix Committed
milestone: 2.next → 2.12-rc
Changed in evergreen:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers