[Bug 13168] New: "Today's checkouts" sort improperly because issuedate lacks seconds.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13168 Bug ID: 13168 Summary: "Today's checkouts" sort improperly because issuedate lacks seconds. Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Circulation Assignee: koha-bugs@lists.koha-community.org Reporter: olli-antti.kivilahti@jns.fi QA Contact: testopia@bugs.koha-community.org CC: gmcharlt@gmail.com, kyle.m.hall@gmail.com TO REPLICATE: Prepare a bunch of Items (6+) for checking out, or have a set of barcodes ready for copy-pasting. Check-out those items quickly within one minute and observe that the sorting order is not always from the first checkout to the last. This is because the issuedate doesn't have seconds defined. AFTER THIS The bunch of Items is sorted properly. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13168 Olli-Antti Kivilahti <olli-antti.kivilahti@jns.fi> changed: What |Removed |Added ---------------------------------------------------------------------------- Assignee|koha-bugs@lists.koha-commun |olli-antti.kivilahti@jns.fi |ity.org | -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13168 Olli-Antti Kivilahti <olli-antti.kivilahti@jns.fi> changed: What |Removed |Added ---------------------------------------------------------------------------- Depends on| |11703 -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13168 --- Comment #1 from Olli-Antti Kivilahti <olli-antti.kivilahti@jns.fi> --- Created attachment 32970 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=32970&action=edit Bug 13168 - "Today's checkouts" sort improperly because issuedate lacks seconds. TO REPLICATE: Prepare a bunch of Items (6+) for checking out, or have a set of barcodes ready for copy-pasting. Check-out those items quickly within one minute and observe that the sorting order is not always from the first checkout to the last. This is because the issuedate doesn't have seconds defined. AFTER THIS The bunch of Items is sorted properly. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13168 Olli-Antti Kivilahti <olli-antti.kivilahti@jns.fi> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13168 --- Comment #2 from Marc Véron <veron@veron.ch> --- Created attachment 33021 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=33021&action=edit [Signed-off] Bug 13168 - "Today's checkouts" sort improperly because issuedate lacks seconds. TO REPLICATE: Prepare a bunch of Items (6+) for checking out, or have a set of barcodes ready for copy-pasting. Check-out those items quickly within one minute and observe that the sorting order is not always from the first checkout to the last. This is because the issuedate doesn't have seconds defined. AFTER THIS The bunch of Items is sorted properly. Tiny patch, works as expected. Passed QA script. Signed-off-by: Marc Véron <veron@veron.ch> -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13168 Marc Véron <veron@veron.ch> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #32970|0 |1 is obsolete| | CC| |veron@veron.ch -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13168 Marc Véron <veron@veron.ch> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13168 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |katrin.fischer@bsz-bw.de Severity|enhancement |normal -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13168 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13168 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #33021|0 |1 is obsolete| | --- Comment #3 from Kyle M Hall <kyle@bywatersolutions.com> --- Created attachment 33082 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=33082&action=edit [PASSED QA] Bug 13168 - "Today's checkouts" sort improperly because issuedate lacks seconds. TO REPLICATE: Prepare a bunch of Items (6+) for checking out, or have a set of barcodes ready for copy-pasting. Check-out those items quickly within one minute and observe that the sorting order is not always from the first checkout to the last. This is because the issuedate doesn't have seconds defined. AFTER THIS The bunch of Items is sorted properly. Tiny patch, works as expected. Passed QA script. Signed-off-by: Marc Veron <veron@veron.ch> Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13168 Helen Linda <helen.linda@goddard.edu> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |helen.linda@goddard.edu -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13168 Fred P <fred.pierre@smfpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |fred.pierre@smfpl.org --- Comment #4 from Fred P <fred.pierre@smfpl.org> --- I like it, but shouldn't the order be newest checkout to oldest? That way the circulation staff sees the most recent item at the top of the list. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13168 --- Comment #5 from Fred P <fred.pierre@smfpl.org> --- Excuse me, I see that is covered in bug #13020. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13168 Tomás Cohen Arazi <tomascohen@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Passed QA |Pushed to Master CC| |tomascohen@gmail.com --- Comment #6 from Tomás Cohen Arazi <tomascohen@gmail.com> --- Patch pushed to master. Thanks Owen! -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13168 --- Comment #7 from Tomás Cohen Arazi <tomascohen@gmail.com> --- I mean, Olli :-D -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13168 --- Comment #8 from Olli-Antti Kivilahti <olli-antti.kivilahti@jns.fi> --- I aim to please! IMHO the biggest challenge is not to write the code. Thie biggest challenge is to find all the awesome people who make the final push! Thank you all who made this happen! -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13168 Mason James <mtj@kohaaloha.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords| |rel_3_16_candidate CC| |mtj@kohaaloha.com --- Comment #9 from Mason James <mtj@kohaaloha.com> --- (In reply to Tomás Cohen Arazi from comment #6)
Patch pushed to master.
Thanks Owen!
Pushed to 3.16.x, will be in 3.16.7 -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13168 Mason James <mtj@kohaaloha.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords|rel_3_16_candidate | Status|Pushed to Master |Pushed to Stable -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon@bugs.koha-community.org