Bug 21456: The 'New authority' button lists authority types inconsistently
authorAndreas Roussos <arouss1980@gmail.com>
Sun, 30 Sep 2018 10:06:28 +0000 (13:06 +0300)
committerFridolin Somers <fridolin.somers@biblibre.com>
Fri, 9 Nov 2018 15:03:19 +0000 (16:03 +0100)
The 'New authority' button does not always list the available
authority types in A-Z order based on their description.

This patch fixes that.

Test plan:
1) Go to Home > Authorities in the staff client; notice how the
   'New authority' button lists auth types sorted by description.
2) Go to an authority's detail page; notice how the same 'New
   authority' button will list auth types sorted by their code.
3) Apply the patch.
4) In an authority's detail page, the 'New authority' button will
   now list the available authority types sorted by description.

Signed-off-by: Michal Denar <black23@gmail.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>

Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
(cherry picked from commit bf7c8303ca7f9e4955f25fc4d1f30d65d8c52dc9)
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
(cherry picked from commit 1f8356d2df19a704a5a12597a9443eb459941aad)
Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com>

authorities/detail.pl

index 8dd4b77..ce5c024 100755 (executable)
@@ -183,7 +183,7 @@ my $authtypecode = $authobj ? $authobj->authtypecode : q{};
 $tagslib = &GetTagsLabels(1,$authtypecode);
 
 # Build list of authtypes for showing them
-my $authority_types = Koha::Authority::Types->search({}, { order_by => ['authtypecode']});
+my $authority_types = Koha::Authority::Types->search({}, { order_by => ['authtypetext']});
 
 my $record=GetAuthority($authid);