[Bug 5199] New: Renewals do not add branchcode to statistics table
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5199 Summary: Renewals do not add branchcode to statistics table Change sponsored?: --- Product: Koha Version: rel_3_2 Platform: All OS/Version: All Status: NEW Severity: enhancement Priority: P5 Component: Circulation AssignedTo: kyle.m.hall@gmail.com ReportedBy: ian.walls@bywatersolutions.com QAContact: koha-bugs@lists.koha-community.org CC: gmcharlt@gmail.com Estimated Hours: 0.0 The AddRenewal() function of Circulation.pm is defined to take $branch as the third parameter, but most calls to this function, including OPAC renewal and Self Checkout, only take the first two, $borrowernumber and $itemnumber. This leaves the statistics table full of NULL values in the $branch field for renewal statistics. The $branch value passed should be the branch of the currently logged in user. This would be the current branch selected in the staff client, the patron's home branch in the OPAC, and the homebranch of the Self Checkout user for SCO. -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA contact for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5199 --- Comment #1 from Owen Leonard <oleonard@myacpl.org> 2010-09-07 12:53:32 UTC --- This problem has plagued us (and our statistics preparation) for a long time. I don't know how SCO determines the user's branch--does it credit the patron's home branch with the circulation rather than the location at which the checkout is being made? My preference for crediting the branch for online renewals would be to give the circ statistic to the branch which originated the checkout rather than to the patron's home branch. In my multi-library system the patron's home branch doesn't necessarily have much to do with the libraries they actually use. -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA contact for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5199 Nicole C. Engard <nengard@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |nengard@gmail.com --- Comment #2 from Nicole C. Engard <nengard@gmail.com> 2010-09-07 12:57:20 UTC --- I haven't tested this - but does your SCO staff user have a branch associated with them? I would think this is the branch that should be credited - just like when the staff log in their branch is credited with the check outs. This of course means you need a SCO user for each branch (which is also a good idea). -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA contact for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5199 --- Comment #3 from Ian Walls <ian.walls@bywatersolutions.com> 2010-09-10 18:50:46 UTC --- For SCO, the homebranch of the SCO staff account should be used, since that can be customized by the library on a per-SCO-terminal basis. For opac renewals, there seem to be a few options: -Credit the library that originated the circ -Credit the patron's home branch -Credit the library that owns the item -Use CircControlBranch to determine which of the above three to use -Create a new system preference just like CircControlBranch, but for OPAC renewal Other options include: -Use a code like "OPAC" (would break if there were an 'OPAC' branchcode) -Define a new statistic type "Self Renewal" -Leave it as null Thoughts? -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA contact for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5199 --- Comment #4 from Owen Leonard <oleonard@myacpl.org> 2010-09-10 18:57:04 UTC ---
-Use a code like "OPAC" (would break if there were an 'OPAC' branchcode)
This creates difficulties if you're designing reports around your branch codes, i.e. creating a table of circ stats with one branch in each column.
-Define a new statistic type "Self Renewal"
I don't think this helps the situation. We're still left with wanting to be able to aggregate circulation stats for each branch. At best this makes explicit what is now only implied by a null value. My vote is definitely to credit the library which originated the checkout. -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA contact for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5199 --- Comment #5 from Nicole C. Engard <nengard@gmail.com> 2010-09-10 21:36:21 UTC --- I'm with crediting the library that initiated the checkout - since that's how it's done at the circ desk. What I'm wondering is is there any other way to know that it was done at the self checkout terminal? Can that be done by querying against the self checkout user that is logged into the terminal? If not then there should be a way to get numbers of how many people used the self checkout versus how many items were checked out from the circ desk. -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA contact for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5199 --- Comment #6 from Nicole C. Engard <nengard@gmail.com> 2011-04-25 14:07:22 UTC --- Just want to give this bug a little bump, since it's effecting statistics reports for librarians. -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA Contact for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5199 Chris Cormack <chris@bigballofwax.co.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED CC| |chris@bigballofwax.co.nz --- Comment #7 from Chris Cormack <chris@bigballofwax.co.nz> 2011-05-05 21:12:39 UTC --- This was discussed today on irc and on the mailing list. I think for opac renewals the best option is a sys pref where the behaviour can be chosen. Will work on a patch -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA Contact for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5199 --- Comment #8 from Chris Cormack <chris@bigballofwax.co.nz> 2011-05-05 23:16:43 UTC --- I think I have it working now, changing the system preference means it records the appropriate branch select * from statistics; +---------------------+-----------+----------+--------+-------+-------+----------+------------+----------+----------------+--------------------+ | datetime | branch | proccode | value | type | other | usercode | itemnumber | itemtype | borrowernumber | associatedborrower | +---------------------+-----------+----------+--------+-------+-------+----------+------------+----------+----------------+--------------------+ | 2011-05-06 11:12:59 | CPL | NULL | 5.0000 | issue | | NULL | 1 | BK | 51 | NULL | | 2011-05-06 11:14:47 | CPL | NULL | 5.0000 | renew | | NULL | 1 | BK | 51 | NULL | | 2011-05-06 11:15:48 | OPACRenew | NULL | 5.0000 | renew | | NULL | 1 | BK | 51 | NULL | +---------------------+-----------+----------+--------+-------+-------+----------+------------+----------+----------------+--------------------+ Patch to follow -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA Contact for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5199 Chris Cormack <chris@bigballofwax.co.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Priority|P5 |PATCH-Sent Version|rel_3_2 |rel_3_6 Patch Status|--- |Needs Signoff --- Comment #9 from Chris Cormack <chris@bigballofwax.co.nz> 2011-05-05 23:19:04 UTC --- The patch does not change the self checkout part, only OPAC renewals. -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA Contact for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5199 --- Comment #10 from Chris Cormack <chris@bigballofwax.co.nz> 2011-05-05 23:19:30 UTC --- Created attachment 4045 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=4045 Proposed Patch -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA Contact for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5199 Nicole C. Engard <nengard@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Patch Status|Needs Signoff |Failed QA --- Comment #11 from Nicole C. Engard <nengard@gmail.com> 2011-05-06 15:09:42 UTC --- Chris, this works beautifully unless you choose NULL after having it set to another value. I had it set to OPACRenew and then I set it to items branch and they both worked then I set it to NULL and it put OPACRenew in the table. We might need someone else to test to confirm my results. -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA Contact for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5199 Chris Cormack <chris@bigballofwax.co.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Patch Status|Failed QA |Needs Signoff --- Comment #12 from Chris Cormack <chris@bigballofwax.co.nz> 2011-05-06 19:21:46 UTC --- I see the problem, follow up patch sent -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA Contact for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5199 --- Comment #13 from Chris Cormack <chris@bigballofwax.co.nz> 2011-05-06 19:23:11 UTC --- Created attachment 4052 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=4052 Follow up patch -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA Contact for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5199 Nicole C. Engard <nengard@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Patch Status|Needs Signoff |Signed Off -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA Contact for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5199 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |katrin.fischer@bsz-bw.de AssignedTo|kyle.m.hall@gmail.com |chris@bigballofwax.co.nz -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA Contact for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5199 Ian Walls <ian.walls@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Patch Status|Signed Off |Passed QA --- Comment #14 from Ian Walls <ian.walls@bywatersolutions.com> 2011-06-25 21:02:08 UTC --- Verified that "OpacRenew" option should work, even if it's not a valid branchcode in the system. Patch covers all languages syspref files, updatedatabase, preference editor, and code itself. Followup corrects comparison error. Marking as Passed QA -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA Contact for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5199 Chris Cormack <chris@bigballofwax.co.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Patch Status|Passed QA |Patch Pushed --- Comment #15 from Chris Cormack <chris@bigballofwax.co.nz> 2011-07-02 20:53:42 UTC --- Pushed to master, please test -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA Contact for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5199 --- Comment #16 from Ian Walls <ian.walls@bywatersolutions.com> 2011-08-12 20:32:04 UTC --- While this patch fixes the issue going forward, it does not repair lost statistics for systems that ran with the bug in effect. A separate cleanup script should be written so folks can restore their OPAC renewal stats, based on the current system preference's setting. -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA Contact for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5199 Jared Camins-Esakov <jcamins@cpbibliography.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Pushed to Master |RESOLVED Resolution|--- |FIXED --- Comment #17 from Jared Camins-Esakov <jcamins@cpbibliography.com> --- This appears to be working. Closing. -- You are receiving this mail because: You are the QA Contact for the bug. You are watching all bug changes.
participants (1)
-
bugzilla-daemon＠bugs.koha-community.org