Bug 14621

Summary: Messaging preferences table needs to be sorted
Product: Koha Reporter: Lari Taskula <lari.taskula>
Component: TemplatesAssignee: Lari Taskula <lari.taskula>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: normal    
Priority: P5 - low CC: chris, gwilliams, jonathan.druart, joonas.kylmala, tomascohen
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Trivial patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Bug Depends on:    
Bug Blocks: 14590    
Attachments: Bug 14621 - Messaging preferences table needs to be sorted
Bug 14621 - Messaging preferences table needs to be sorted
Bug 14621 - Messaging preferences table needs to be sorted

Description Lari Taskula 2015-07-30 15:23:57 UTC
I have been working with messaging preferences and I noticed a weird issue in Firefox on Ubuntu.

On messaging preferences page, the table is unsorted and the content in rows are generated randomly on every page refresh. When you select/deselect checkboxes and refresh the page (without posting the changes), Firefox will remember your choices. Now the issue is that when the table is unsorted and the rows keep changing on page refresh, Firefox has trouble remembering your choices. This makes it appear as if the checkboxes are magically changing values on each page refresh.

We need to have the items sorted in the table to prevent this from happening.
Comment 1 Lari Taskula 2015-08-14 11:51:03 UTC Comment hidden (obsolete)
Comment 2 Chris Cormack 2015-09-03 04:17:18 UTC Comment hidden (obsolete)
Comment 3 Jonathan Druart 2015-09-03 08:28:41 UTC
Created attachment 42273 [details] [review]
Bug 14621 - Messaging preferences table needs to be sorted

I have been working with messaging preferences and I noticed a weird issue in Firefox on Ubuntu.

On messaging preferences page, the table is unsorted and the content in rows are generated randomly
on every page refresh. When you select/deselect checkboxes and refresh the page (without posting the changes),
Firefox will remember your choices. Now the issue is that when the table is unsorted and the rows keep
changing on page refresh, Firefox has trouble remembering your choices. This makes it appear as if the
checkboxes are magically changing values on each page refresh.

Here is a patch that prevents this problem by sorting the messaging settings.

Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 4 Jonathan Druart 2015-09-03 08:28:53 UTC
Very useful patch, thanks!
Comment 5 Tomás Cohen Arazi 2015-09-07 15:27:02 UTC
Patch pushed to master.

Thanks Lari! And congratulations for your commit!
Comment 6 Chris Cormack 2015-09-22 16:57:19 UTC
Pushed to 3.20.x will be in 3.20.4