---- Reported by nahuel.angelinetti@biblibre.com 2010-01-19 10:59:46 ---- 2 more columns are needed in csv export of overdues : * item callnumber * issue date ---- Additional Comments From nahuel.angelinetti@biblibre.com 2010-01-19 13:18:08 ---- first name and last name must be separated in two columns. ---- Additional Comments From gmcharlt@gmail.com 2010-02-08 17:40:12 ---- The patches that were sent to patches@koha.org do not apply against HEAD and never did. Please resubmit. --- Bug imported by chris@bigballofwax.co.nz 2010-05-21 01:22 UTC --- This bug was previously known as _bug_ 4051 at http://bugs.koha.org/cgi-bin/bugzilla3/show_bug.cgi?id=4051 Actual time not defined. Setting to 0.0 Setting qa contact to the default for this product. This bug either had no qa contact or an invalid one.
Patches: http://lists.koha-community.org/pipermail/koha-patches/2010-January/010844.html http://lists.koha-community.org/pipermail/koha-patches/2010-January/010848.html http://lists.koha-community.org/pipermail/koha-patches/2010-January/010871.html http://lists.koha-community.org/pipermail/koha-patches/2010-January/010891.html Patches were applied to 3.0.x but never resubmitted for HEAD.
Do we need these for master?
Paul, is this something you think we need in master?
Created attachment 5242 [details] [review] proposed patch Yes, I think it's worth having date due (as well as callnumber, was not specified in previous comments) in what is exported in CSV
Created attachment 5243 [details] [review] proposed patch oups, typo in comment = it's issuedate and not date_due (that was already here) (Only the patch comment has been updated)
Comment on attachment 5242 [details] [review] proposed patch As far as I can tell, this is obsoleted by Attachment #5243 [details].
itemcallnumber is already included in the CSV export, see: http://git.koha-community.org/gitweb/?p=koha.git;a=blob;f=circ/overdue.pl;h=2bdafe1a0bb969fd2e80ae86d12fc02ec08bb2ef;hb=HEAD#l256 http://git.koha-community.org/gitweb/?p=koha.git;a=blob;f=circ/overdue.pl;h=2bdafe1a0bb969fd2e80ae86d12fc02ec08bb2ef;hb=HEAD#l324 After applying this patch it is included twice in the SQL query. It only shows up once in the generated CSV, but I'm marking this as failed QA...
Created attachment 6520 [details] [review] patch adding issuedate Patch adding the issuedate column only (itemcallnumber being already present).
Created attachment 6556 [details] [review] Bug 4051: adds issuedate column to overdues export Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
QA comment: small patch, easy to read, passed QA Patch pushed, please test chris_n = I think it's an enhancement, and suggest not to backport this bug to 3.6: ppl that have automated scripts with CSV may be confused by their scripts not working anymore. But, as usual, it's up to you