From 845486da3cd04e1998531a604e7453138be54664 Mon Sep 17 00:00:00 2001 From: Jonathan Druart Date: Wed, 17 Jul 2013 15:02:43 +0200 Subject: [PATCH] [SIGNED-OFF] Bug 10038: 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 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 :) --- admin/auth_tag_structure.pl | 3 ++- .../prog/en/modules/admin/auth_tag_structure.tt | 8 +++++--- 2 files changed, 7 insertions(+), 4 deletions(-) diff --git a/admin/auth_tag_structure.pl b/admin/auth_tag_structure.pl index 906f8d3..e84b459 100755 --- 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 --- 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; Create authority framework for [% authtypecode %] using @@ -209,7 +212,6 @@ return false;

Select an authority framework