Bug 10038: fix doubled 'default' entry when creating a new authority type

Test plan:
- Create a new authority type
- Click MARC structure
- Verify the pull down has only 1 entry for Default
- Go on the authority type home (admin/authtypes.pl)
- Click on the "MARC structure" link for the default type
- Verify the pull down has only 1 entry for Default

This patch adds a sort (on the authtypecode) for these 2 lists.

Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel@gmail.com>
Comment: Works as described. No koha-qa errors.

When creating a new framework it would be better to have Default
on top, but one is way better than two :)

Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Agreed, one is better than two :)
All tests and QA script pass.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>

diff --git a/admin/auth_tag_structure.pl b/admin/auth_tag_structure.pl
index 906f8d3..e84b459 100755 (executable)
--- a/admin/auth_tag_structure.pl
+++ b/admin/auth_tag_structure.pl
@@ -57,7 +57,7 @@ my ($template, $loggedinuser, $cookie)
 # get authtype list
 my $authtypes     = getauthtypes;
 my @authtypesloop = ();
-foreach my $thisauthtype ( keys %{$authtypes} ) {
+foreach my $thisauthtype ( sort keys %{$authtypes} ) {
     push @authtypesloop,
       { value        => $thisauthtype,
         selected     => $thisauthtype eq $authtypecode,
@@ -209,6 +209,7 @@ if ($op eq 'add_form') {
                        push @existingauthtypeloop,\%line;
                }
        }
+    @existingauthtypeloop = sort { lc($a->{authtypetext}) cmp lc($b->{authtypetext}) }@existingauthtypeloop;
        $template->param(existingauthtypeloop => \@existingauthtypeloop,
                                        authtypecode => $authtypecode,
                                        );

diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/auth_tag_structure.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/auth_tag_structure.tt
index aa14e13..aed41b6 100644 (file)
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/auth_tag_structure.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/auth_tag_structure.tt
@@ -190,9 +190,12 @@ return false;
         <input type="hidden" name="authtypecode" value="[% authtypecode %]" />
         Create authority framework for [% authtypecode %] using
         <select name="existingauthtypecode">
-            <option value="">Default</option>
         [% FOREACH existingauthtypeloo IN existingauthtypeloop %]
-            <option value="[% existingauthtypeloo.value %]">[% existingauthtypeloo.authtypetext %]</option>
+            [% IF existingauthtypeloo.value == "" %]
+              <option value="" selected="selected">[% existingauthtypeloo.authtypetext %]</option>
+            [% ELSE %]
+              <option value="[% existingauthtypeloo.value %]">[% existingauthtypeloo.authtypetext %]</option>
+            [% END %]
         [% END %]
         </select>
         <input type="submit" value="OK" class="submit" />
@@ -209,7 +212,6 @@ return false;
 <h2>Select an authority framework</h2>
 <form action="[% script_name %]" method="post">
     <select name="authtypecode">
-        <option value="">Default</option>
     [% FOREACH authtypeloo IN authtypeloop %]
         [% IF ( authtypeloo.selected ) %]
         <option value="[% authtypeloo.value %]" selected="selected">[% authtypeloo.authtypetext %]</option>