Bug 13162 - Label batch editor title sorting by default, rather then label number
Summary: Label batch editor title sorting by default, rather then label number
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Tools (show other bugs)
Version: unspecified
Hardware: All All
: P5 - low minor (vote)
Assignee: Galen Charlton
QA Contact: Testopia
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-10-29 17:49 UTC by Kyle M Hall
Modified: 2015-06-04 23:33 UTC (History)
3 users (show)

See Also:
Change sponsored?: ---
Patch complexity: Trivial patch
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:


Attachments
Bug 13162 - Label batch editor title sorting by default, rather then label number (1.41 KB, patch)
2014-10-29 17:52 UTC, Kyle M Hall
Details | Diff | Splinter Review
[Signed-off] Bug 13162 - Label batch editor title sorting by default, rather then label number (1.60 KB, patch)
2014-10-29 20:22 UTC, Marc Véron
Details | Diff | Splinter Review
[PASSED QA] Bug 13162 - Label batch editor title sorting by default, rather then label number (1.68 KB, patch)
2014-10-30 01:09 UTC, Katrin Fischer
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Kyle M Hall 2014-10-29 17:49:16 UTC
The label batches editor has always sorted by the order the items were added to a batch. However, it is now sorting by title. This should be remedied.
Comment 1 Kyle M Hall 2014-10-29 17:52:31 UTC Comment hidden (obsolete)
Comment 2 Marc Véron 2014-10-29 20:22:50 UTC Comment hidden (obsolete)
Comment 3 Katrin Fischer 2014-10-30 01:09:14 UTC
Created attachment 32958 [details] [review]
[PASSED QA] Bug 13162 - Label batch editor title sorting by default, rather then label number

The label batches editor has always sorted by the order the items were
added to a batch. However, it is now sorting by title. This should be
remedied.

Test Plan:
1) Create a label batch
2) Note the items are in order of title ( summary )
3) Apply this patch
4) Refresh the page, not the items are now sorted by label number

Tiny patch. Works as expected.
Signed-off-by: Marc Véron <veron@veron.ch>

Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Passes tests, works as described.
Comment 4 Tomás Cohen Arazi 2014-11-21 22:53:43 UTC
Patch pushed to master.

Thanks Kyle!