Summary: | "Today's checkouts" sort improperly because issuedate lacks seconds. | ||
---|---|---|---|
Product: | Koha | Reporter: | Olli-Antti Kivilahti <olli-antti.kivilahti> |
Component: | Circulation | Assignee: | Olli-Antti Kivilahti <olli-antti.kivilahti> |
Status: | CLOSED FIXED | QA Contact: | Testopia <testopia> |
Severity: | normal | ||
Priority: | P5 - low | CC: | fred.pierre, gmcharlt, helen.linda, kyle.m.hall, mtj, tomascohen, veron |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
GIT URL: | Change sponsored?: | --- | |
Patch complexity: | --- | Documentation contact: | |
Documentation submission: | Text to go in the release notes: | ||
Version(s) released in: | Circulation function: | ||
Bug Depends on: | 11703 | ||
Bug Blocks: | |||
Attachments: |
Bug 13168 - "Today's checkouts" sort improperly because issuedate lacks seconds.
[Signed-off] Bug 13168 - "Today's checkouts" sort improperly because issuedate lacks seconds. [PASSED QA] Bug 13168 - "Today's checkouts" sort improperly because issuedate lacks seconds. |
Description
Olli-Antti Kivilahti
2014-10-30 12:54:02 UTC
Created attachment 32970 [details] [review] 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. Created attachment 33021 [details] [review] [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> Created attachment 33082 [details] [review] [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> 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. Excuse me, I see that is covered in bug #13020. Patch pushed to master. Thanks Owen! I mean, Olli :-D 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! (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 |