The vendor search results page shows tables of baskets for each of those vendors. In those tables the "item count" and "biblio count" columns will be sorted incorrectly if some of the cells contain textual data (ex: "(2 cancelled)"). These columns should be configured to have a numeric sort.
Created attachment 32058 [details] [review] Bug 13039 - Vendor search: sorting of "item count" and "biblio count" columns can be incorrect On the vendor search results page if some cells contain textual data the "item count" and "biblio count" columns will sort incorrectly. This patch sets an explicit numeric sort on these columns. In doing so this patch also changes the existing column sorting configuration to use table header cell classes instead of sorting based on index. This patch also corrects two instances of unescaped ampersands. To test, search for a vendor which has multiple baskets, at least one of which should contain canceled orders. Confirm that sorting by item count, biblio count, date, and closed all work correctly.
Created attachment 32299 [details] [review] Bug 13039 - Vendor search: sorting of "item count" and "biblio count" columns can be incorrect On the vendor search results page if some cells contain textual data the "item count" and "biblio count" columns will sort incorrectly. This patch sets an explicit numeric sort on these columns. In doing so this patch also changes the existing column sorting configuration to use table header cell classes instead of sorting based on index. This patch also corrects two instances of unescaped ampersands. To test, search for a vendor which has multiple baskets, at least one of which should contain canceled orders. Confirm that sorting by item count, biblio count, date, and closed all work correctly. Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
Created attachment 32329 [details] [review] [PASSED QA] Bug 13039 - Vendor search: sorting of "item count" and "biblio count" columns can be incorrect On the vendor search results page if some cells contain textual data the "item count" and "biblio count" columns will sort incorrectly. This patch sets an explicit numeric sort on these columns. In doing so this patch also changes the existing column sorting configuration to use table header cell classes instead of sorting based on index. This patch also corrects two instances of unescaped ampersands. To test, search for a vendor which has multiple baskets, at least one of which should contain canceled orders. Confirm that sorting by item count, biblio count, date, and closed all work correctly. Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz> Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de> I have not been able to see the sorting problem, but the patch causes no regression and everything seems to work nicely. Passes QA script and tests. http://bugs.koha-community.org/show_bug.cgi?id=12039
Patch pushed to master. Thanks Owen!