Evergreen

tpac my list download csv only downloads first 10 entries

Bug #1029433 reported by Jason Stephenson
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Evergreen
Status tracked in Main
2.2
Won't Fix
Low
Unassigned
Evergreen 2.2.5
2.3
Fix Released
Low
Unassigned
Evergreen 2.3.0-beta2
Main
Fix Released
Low
Unassigned
Evergreen 2.3.0-beta2

Bug Description

EG version: Master + a load of other development branches
OpenSRF: 2.1.0
Postgres: 9.1.4
Ubuntu Precise

When the download csv button is clicked in the tpac my list interface, the resulting csv file only contains the first 10 list entries. Also, the publication date field should be added if bug 1029394 is merged.

I'll do two branches and throw them on this bug:

1. to fix the number of entries in the csv.
2. to add the publication date to the csv.

Jason Stephenson (jstephenson)
Changed in evergreen:
importance: Undecided → Wishlist
importance: Wishlist → Undecided
Revision history for this message
Jason Stephenson (jstephenson) wrote :

user/dyrcona/lp1029433_fix on the working repository addresses #1.

It simply adds an args hashref in Trigger/Reactor/ContainerCSV.pm that limits the search results to 1,000.

It should cherry-pick back to 2.2 with no trouble.

#2 might not be so easy to address without affecting other parts of Evergreen.

Revision history for this message
Jason Stephenson (jstephenson) wrote :

#2 is going to be trickier than I thought and will deserve its own, probably wishlist, bug if it gets addressed.

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

I was able to export more than 10 things, so yay, seems to be working so far.

Sign-off: user/bshum/lp1029433_fix

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

Revision history for this message
Dan Scott (denials) wrote :

Thanks to both of you; I tested it out and confirmed it works too :) Pushed to master and rel_2_3.

Jason Stephenson (jstephenson)
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.
Subscribing...

Other bug subscribers

Remote bug watches

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