Follow-up Bug 5634: Ordering branches should be case independent
authorNadia Nicolaides <nanunic@gmail.com>
Wed, 19 Dec 2012 18:52:34 +0000 (15:52 -0300)
committerJared Camins-Esakov <jcamins@cpbibliography.com>
Fri, 21 Dec 2012 12:42:18 +0000 (07:42 -0500)
commit4d644c6930bc200595fcaa2ed07999945ec10cd5
tree24a63a6d5d86300edf68782ecdf669a028fb7779
parentdd754c3108dd71885e8af3ebcaa34028855bdfee
Follow-up Bug 5634: Ordering branches should be case independent

This patch fix the order of branches in the log-in page,
on Branch.pm we added the variable branchcode to the
hash returned by GetBranchesLoop, and this function is used
on Auth.pm to get a list of branches ordered by branchname

To test
1) Use an installation with some branches
2) On login screen the branches are ordered by branchcode
3) apply the patch
4) On login screen the branches are now ordered by branchname

Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
This patch works as expected. Before applying the patch the branches
with lower case was at the end of the list. Now they are well ordered.
Signed-off-by: Jared Camins-Esakov <jcamins@cpbibliography.com>
C4/Auth.pm
C4/Branch.pm