Bug 24170

Summary: sysprefs search result does not have a consistent order
Product: Koha Reporter: Jonathan Druart <jonathan.druart>
Component: System AdministrationAssignee: Jonathan Druart <jonathan.druart>
Status: CLOSED FIXED QA Contact: Marcel de Rooy <m.de.rooy>
Severity: normal    
Priority: P5 - low CC: carnold, gmcharlt, m.de.rooy, martin.renvoize
Version: unspecified   
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:
20.05.00
Attachments: Bug 24170: Fix sysprefs search result order
Bug 24170: Fix sysprefs search result order
Bug 24170: Fix sysprefs search result order

Description Jonathan Druart 2019-12-04 14:44:47 UTC
The same sysprefs search will not always return the results in the same order.
Comment 1 Jonathan Druart 2019-12-04 14:47:36 UTC
Created attachment 95993 [details] [review]
Bug 24170: Fix sysprefs search result order

The same sysprefs search will not always return the results in the same
order.
If you search for 'd' in the "search system preferences" box
  /cgi-bin/koha/admin/preferences.pl?op=search&searchfield=d
you will see that the modules are not ordered consistently.

This patch makes them display alphabetically

Test plan:
Search the same string several times in the syspref list.
The search result should always be ordered the same way
Comment 2 ByWater Sandboxes 2019-12-04 15:43:50 UTC
Created attachment 95995 [details] [review]
Bug 24170: Fix sysprefs search result order

The same sysprefs search will not always return the results in the same
order.
If you search for 'd' in the "search system preferences" box
  /cgi-bin/koha/admin/preferences.pl?op=search&searchfield=d
you will see that the modules are not ordered consistently.

This patch makes them display alphabetically

Test plan:
Search the same string several times in the syspref list.
The search result should always be ordered the same way

Signed-off-by: cori <corilynn.arnold@gmail.com>
Comment 3 Cori Lynn Arnold 2019-12-04 15:44:11 UTC
Works as advertised!
Comment 4 Marcel de Rooy 2019-12-06 09:19:11 UTC
Created attachment 96058 [details] [review]
Bug 24170: Fix sysprefs search result order

The same sysprefs search will not always return the results in the same
order.
If you search for 'd' in the "search system preferences" box
  /cgi-bin/koha/admin/preferences.pl?op=search&searchfield=d
you will see that the modules are not ordered consistently.

This patch makes them display alphabetically

Test plan:
Search the same string several times in the syspref list.
The search result should always be ordered the same way

Signed-off-by: cori <corilynn.arnold@gmail.com>

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Comment 5 Jonathan Druart 2019-12-10 09:55:50 UTC
> Signed-off-by: cori <corilynn.arnold@gmail.com>

Do not you want your real name appears in the release notes?
Comment 6 Martin Renvoize 2019-12-10 10:38:24 UTC
(In reply to Jonathan Druart from comment #5)
> > Signed-off-by: cori <corilynn.arnold@gmail.com>
> 
> Do not you want your real name appears in the release notes?

I'll likely just add 'cori' as an alias for her full name in the release-tools.  But.. it's always nice to have it done right in the commit message in the first place too
Comment 7 Martin Renvoize 2019-12-13 14:07:42 UTC
Nice work everyone!

Pushed to master for 20.05