Bug 4331 - Slow Response Times Switching To Acquisitions
Summary: Slow Response Times Switching To Acquisitions
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Acquisitions (show other bugs)
Version: rel_3_2
Hardware: PC All
: PATCH-Sent (DO NOT USE) major (vote)
Assignee: Henri-Damien LAURENT
QA Contact: Bugs List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-03-23 08:42 UTC by Colin Campbell
Modified: 2012-10-25 22:52 UTC (History)
1 user (show)

See Also:
Change sponsored?: ---
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:


Attachments
Patch adding index (2.38 KB, patch)
2010-03-23 10:05 UTC, Chris Cormack
Details | Diff | Splinter Review
Patch removing useless processing from acq-home.pl (9.49 KB, patch)
2010-03-23 10:06 UTC, Chris Cormack
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Chris Cormack 2010-05-21 01:25:55 UTC


---- Reported by colin.campbell@ptfs-europe.com 2010-03-23 08:42:01 ----

Selecting acquisitions from elsewhere in the system results in a perceivable delay before the screen is displayed



---- Additional Comments From colin.campbell@ptfs-europe.com 2010-03-23 10:04:07 ----

The acq-home page displays the current commited and spent totals for budgets. Both these operations require a calculation on the aqorders tables keyed by budget_id these operations were slow as there is no index of aqorders by budget. Patch adds one which removes the delay time.

Inspection of acqui-home.pl revealed that the script was making numerous db calls and calculations to produce data which was then discarded ( budgets were actually retrieved from the database twice!) Removed pointless processing. Renamed anonymous variables. Separate patch cleans the cruft from acqui-home.pl

   



---- Additional Comments From colin.campbell@ptfs-europe.com 2010-03-23 10:05:34 ----

Created an attachment
Patch adding index





---- Additional Comments From colin.campbell@ptfs-europe.com 2010-03-23 10:06:48 ----

Created an attachment
Patch removing useless processing from acq-home.pl 





---- Additional Comments From gmcharlt@gmail.com 2010-04-07 15:17:30 ----

Patches pushed to HEAD.  Please test and close.



--- Bug imported by chris@bigballofwax.co.nz 2010-05-21 01:25 UTC  ---

This bug was previously known as _bug_ 4331 at http://bugs.koha.org/cgi-bin/bugzilla3/show_bug.cgi?id=4331
Imported an attachment (id=2017)
Imported an attachment (id=2018)

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.
The original submitter of attachment 2017 [details] [review] is unknown.
   Reassigning to the person who moved it here: chris@bigballofwax.co.nz.
The original submitter of attachment 2018 [details] [review] is unknown.
   Reassigning to the person who moved it here: chris@bigballofwax.co.nz.