[Koha-patches] [PATCH] Bug 2553: Fixes location drop-down alphabetization in Serials Statistics wizard form.

Jesse pianohacker at gmail.com
Sat Jul 18 07:44:14 CEST 2009


2009/7/17 Garry Collum <gcollum at gmail.com>

> ---
>  reports/serials_stats.pl |    4 ++--
>  1 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/reports/serials_stats.pl b/reports/serials_stats.pl
> index a42fc70..ff98f34 100755
> --- a/reports/serials_stats.pl
> +++ b/reports/serials_stats.pl
> @@ -145,7 +145,7 @@ if($do_it){
>     ## We generate branchlist
>     my $branches=GetBranches();
>        my @branchloop;
> -       foreach (keys %$branches) {
> +       foreach (sort {$branches->{$a}->{'branchname'} cmp
> $branches->{$b}->{'branchname'}} keys %$branches) {
>                my $thisbranch = ''; # FIXME: populate $thisbranch to
> preselect one
>                my %row = (branchcode => $_,
>                        selected => ($thisbranch eq $_ ? 1 : 0),
> @@ -170,4 +170,4 @@ if($do_it){
>         branches     => \@branchloop);
>  }
>
> -output_html_with_http_headers $input, $cookie, $template->output;
> \ No newline at end of file
> +output_html_with_http_headers $input, $cookie, $template->output;
> --
> 1.5.6.5
>
> _______________________________________________
> Koha-patches mailing list
> Koha-patches at lists.koha.org
> http://lists.koha.org/mailman/listinfo/koha-patches
>

I noticed that you've been sending off a lot of patches that fix
alphabetization issues. I think that the GetBranchesLoop function would make
your life a lot easier. There might be an equivalent for itemtypes; not
sure.

-- 
Jesse Weaver
-------------- next part --------------
An HTML attachment was scrubbed...
URL: </pipermail/koha-patches/attachments/20090717/010d9470/attachment-0002.htm>


More information about the Koha-patches mailing list