Bug 15106

Created attachment 44529 [details] [review]
Bug 15106:: Batch Patron Modification Performance Improvement

Batch Patron Modification has a fairly long run/load time. I have attempted
to improve the runtime of the Batch Patron Modification tool by removing
unecessary reporting detail after large batch patron changes and altering
some logic to avoid fetching and checking data that won't be needed.

Test Plan:
1) Prepare NTYprof (http://wiki.koha-community.org/wiki/Profiling_with_Devel::NYTProf)
   or procure a stopwatch (stopwatch will be simpler but less accurate).
2) Ensure you have a decent number of patrons in your Koha system (>500) if you are
   lacking patrons to modify in your database you can import 1000 using the
   PatronDataCSV.csv attachement and the Import Patrons tool before moving to
   step 3.
3) Prepare a file with the cardnumber of borrowers to modify. If you imported
   patrons using PatronDataCSV.csv you can use the attached PatronCardNums.txt
4) Navigate to Home > Tools > Batch Patrons Modification in the Koha Intranet.
5) Click the "Browse..." button and select PatronCardNums.txt
6) Click the "Continue" button and scroll to the bottom of the page.
7) Enter text into the "Country" field box.
8) Enter text into the "Circulation note" field box.
9) If you are using a stopwatch, prepare your stopwatch so that you will
   start counting seconds from the point you click the button in the next
   step.
10) Click the "Save" button (simultaneously start your stopwatch if using one)
11) When the page appears showing completion of the change, stop your stopwatch
    and check the time or navigate to the folder you have set NYTProf to output.
12) Record the runtime. This is the pre-optimisation time.
13) If this is your pre-patch test, the success page will show all the patrons
    with the data changed, if this is your post-patch test it should only show
    a "Successfully changed <number> patrons" message.
13) Apply this patch.
14) Repeat steps 2-12 of this testplan with the patch applied. This will yield
    the post-optimisation time.
15) Compare the pre-optimisation time and post optimisation time. The second
    post-optimisation time should be faster.

Created attachment 44530 [details]
CSV file containing patron data for import

Created attachment 44531 [details]
Card Numbers txt file for identifying patrons to modify