Summary: | circ/billing.pl needs the more complex code to achieve goals for large data sets | ||
---|---|---|---|
Product: | Koha | Reporter: | Chris Cormack <chris> |
Component: | Circulation | Assignee: | Katrin Fischer <katrin.fischer> |
Status: | CLOSED FIXED | QA Contact: | |
Severity: | normal | ||
Priority: | P3 | CC: | koha.sekjal, paul.poulain |
Version: | 3.8 | ||
Hardware: | All | ||
OS: | All | ||
Change sponsored?: | --- | Patch complexity: | --- |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: | ||
Circulation function: | |||
Attachments: |
Bug 2034: Removing unused scripts billig.pl/billing.tt
Bug 2034: Removing unused scripts billig.pl/billing.tt |
Description
Chris Cormack
2010-05-21 00:37:26 UTC
If this script is to be kept, let's fix it. Otherwise let's just remove it. I vote remove it, it is a pile of spaghetti sql, which can more easily be done in a report. Created attachment 9009 [details] [review] Bug 2034: Removing unused scripts billig.pl/billing.tt - Scripts are currently not linked from anywhere in the templates - Functionality can also be achieved with a custom report - There are unresolved problems with the sql in the existig scripts Created attachment 9032 [details] [review] Bug 2034: Removing unused scripts billig.pl/billing.tt - Scripts are currently not linked from anywhere in the templates - Functionality can also be achieved with a custom report - There are unresolved problems with the sql in the existig scripts Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz> Successfully removes this script from the system; since it was functionally inaccessible, and problematic. Passed QA. This will not be included in 3.6.x, just in case someone is using the script somehow. There have been no further reports of problems so I am marking this bug resolved. |