[Koha-bugs] [Bug 18882] Add location code to statistics table for checkouts and renewals
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18882 Jonathan Druart changed: What|Removed |Added Status|Passed QA |Pushed to Master --- Comment #10 from Jonathan Druart --- Pushed to master for 17.11, thanks to everybody involved! -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 18882] Add location code to statistics table for checkouts and renewals
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18882 Marcel de Rooy changed: What|Removed |Added Attachment #64759|0 |1 is obsolete|| --- Comment #9 from Marcel de Rooy --- Created attachment 68912 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=68912&action=edit Bug 18882 [QA Followup] - Ensure stats with no or undef location are set to NULL in db Confirm that only NULL or a string are in the location field of the statistics table. Test with two items, one with shelving location set, and one without it set. Signed-off-by: Mark Tompsett Signed-off-by: Marcel de Rooy -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 18882] Add location code to statistics table for checkouts and renewals
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18882 Marcel de Rooy changed: What|Removed |Added QA Contact|testo...@bugs.koha-communit |m.de.r...@rijksmuseum.nl |y.org | -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 18882] Add location code to statistics table for checkouts and renewals
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18882 Marcel de Rooy changed: What|Removed |Added Attachment #64758|0 |1 is obsolete|| --- Comment #8 from Marcel de Rooy --- Created attachment 68911 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=68911&action=edit Bug 18882 - Add location code to statistics table for checkouts and renewals Some libraries wish to track what the current location of items was at the time they were checked out. This will help libraries track which physical locations in the library patrons are more likely to check out a given book from. Test Plan: 1) Apply this patch 2) Run updatedatabase.pl 3) Check out an item that has a location set 4) Renew that item 5) View the checkout and renewal in the statistics table, verify each has the location column populated correctly Signed-off-by: Mark Tompsett Signed-off-by: Marcel de Rooy -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 18882] Add location code to statistics table for checkouts and renewals
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18882 Marcel de Rooy changed: What|Removed |Added Status|Signed Off |Passed QA Patch complexity|--- |Small patch -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 18882] Add location code to statistics table for checkouts and renewals
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18882 Marcel de Rooy changed: What|Removed |Added CC||m.de.r...@rijksmuseum.nl --- Comment #7 from Marcel de Rooy --- (In reply to Kyle M Hall from comment #4) > (In reply to M. Tompsett from comment #2) > > The code puts '' if there is no location key. But if the location key is > > NULL, because I hadn't set my shelving location, there is NULL. Do we want > > NULL's? If so, I'll sign this off. If not, could you tweak that? > > I think we should always use NULL is the location is not passed or is set to > undef. It seems like odd behavior to convert undef values to empty strings. What we are doing now however, is storing undef only for location. Which is inconsistent. You change all lines in the second patch, only set undef for location. No blocker, but could be improved. I think that empty string would have been better now. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 18882] Add location code to statistics table for checkouts and renewals
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18882 M. Tompsett changed: What|Removed |Added Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 18882] Add location code to statistics table for checkouts and renewals
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18882 M. Tompsett changed: What|Removed |Added Attachment #64756|0 |1 is obsolete|| --- Comment #6 from M. Tompsett --- Created attachment 64759 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=64759&action=edit Bug 18882 [QA Followup] - Ensure stats with no or undef location are set to NULL in db Confirm that only NULL or a string are in the location field of the statistics table. Test with two items, one with shelving location set, and one without it set. Signed-off-by: Mark Tompsett -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 18882] Add location code to statistics table for checkouts and renewals
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18882 M. Tompsett changed: What|Removed |Added Attachment #64727|0 |1 is obsolete|| --- Comment #5 from M. Tompsett --- Created attachment 64758 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=64758&action=edit Bug 18882 - Add location code to statistics table for checkouts and renewals Some libraries wish to track what the current location of items was at the time they were checked out. This will help libraries track which physical locations in the library patrons are more likely to check out a given book from. Test Plan: 1) Apply this patch 2) Run updatedatabase.pl 3) Check out an item that has a location set 4) Renew that item 5) View the checkout and renewal in the statistics table, verify each has the location column populated correctly Signed-off-by: Mark Tompsett -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 18882] Add location code to statistics table for checkouts and renewals
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18882 Kyle M Hall changed: What|Removed |Added Status|In Discussion |Needs Signoff -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 18882] Add location code to statistics table for checkouts and renewals
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18882 Kyle M Hall changed: What|Removed |Added Status|Needs Signoff |In Discussion --- Comment #4 from Kyle M Hall --- (In reply to M. Tompsett from comment #2) > The code puts '' if there is no location key. But if the location key is > NULL, because I hadn't set my shelving location, there is NULL. Do we want > NULL's? If so, I'll sign this off. If not, could you tweak that? I think we should always use NULL is the location is not passed or is set to undef. It seems like odd behavior to convert undef values to empty strings. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 18882] Add location code to statistics table for checkouts and renewals
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18882 --- Comment #3 from Kyle M Hall --- Created attachment 64756 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=64756&action=edit Bug 18882 [QA Followup] - Ensure stats with no or undef location are set to NULL in db -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 18882] Add location code to statistics table for checkouts and renewals
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18882 Kyle M Hall changed: What|Removed |Added Status|In Discussion |Needs Signoff -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 18882] Add location code to statistics table for checkouts and renewals
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18882 M. Tompsett changed: What|Removed |Added CC||mtomp...@hotmail.com Status|Needs Signoff |In Discussion --- Comment #2 from M. Tompsett --- The code puts '' if there is no location key. But if the location key is NULL, because I hadn't set my shelving location, there is NULL. Do we want NULL's? If so, I'll sign this off. If not, could you tweak that? -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 18882] Add location code to statistics table for checkouts and renewals
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18882 --- Comment #1 from Kyle M Hall --- Created attachment 64727 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=64727&action=edit Bug 18882 - Add location code to statistics table for checkouts and renewals Some libraries wish to track what the current location of items was at the time they were checked out. This will help libraries track which physical locations in the library patrons are more likely to check out a given book from. Test Plan: 1) Apply this patch 2) Run updatedatabase.pl 3) Check out an item that has a location set 4) Renew that item 5) View the checkout and renewal in the statistics table, verify each has the location column populated correctly -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 18882] Add location code to statistics table for checkouts and renewals
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18882 Kyle M Hall changed: What|Removed |Added Status|NEW |Needs Signoff -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 18882] Add location code to statistics table for checkouts and renewals
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18882 Kyle M Hall changed: What|Removed |Added Assignee|koha-b...@lists.koha-commun |k...@bywatersolutions.com |ity.org | -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
