Bug 11065

Summary: List of imported biblios is too small - increase to a larger number
Product: Koha Reporter: Doug Kingston <dpk>
Component: ToolsAssignee: Galen Charlton <gmcharlt>
Status: RESOLVED DUPLICATE QA Contact: Testopia <testopia>
Severity: enhancement    
Priority: P5 - low CC: martin.renvoize
Version: 3.12   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: Patch to increase size of results list for manage-marc-import page

Description Doug Kingston 2013-10-16 22:02:17 UTC
Usability fix to MARC importing:

When importing biblios using the MARC importing tools, the list of imported MARC records are display in pages of 25 biblios each.  This means on larger imports (100s or 1000s of biblios) you have to page through way too many pages to find import issues.  This process can be facilitated by returning much larger lists.  We have changed this to 500 entries and its much easier to work with large imports with this size of returned results.

Steps to reproduce:
1. Create a MARC import file with 1000 unique MARC records
2. Upload the file
3. Import the file to the reservoir.
4. You are now looking at the import results page displayed by manage-marc-import.pl with 25 results displayed, and a total of 40 pages to view to see all the import success/failures.

Desired
Reviewing all these pages is tedious.  Less pages will be much better.
 
Fixing this is a one line change to 
library:/usr/share/koha/intranet/cgi-bin/tools# diff tools/manage-marc-import.pl*
50c52
< my $results_per_page = $input->param('results_per_page') || 25; 
---
> my $results_per_page = $input->param('results_per_page') || 500;

(This was also the consensus of the Cataloging roundtable at KohaCon2013.  I volunteered to submit the bug and a formal fix.)
Comment 1 Doug Kingston 2013-10-17 22:43:39 UTC
Created attachment 22041 [details] [review]
Patch to increase size of results list for manage-marc-import page
Comment 2 Martin Renvoize 2013-10-17 23:04:53 UTC
There's a nicer fix for this in bug 10558 that uses Datatables and Ajax to dynamically load the content.  I'de love to see that patch go in in preference to this... though this is a pretty good quick fix.
Comment 3 Martin Renvoize 2013-10-17 23:05:31 UTC

*** This bug has been marked as a duplicate of bug 10558 ***