X-Git-Url: http://koha-dev.rot13.org:8081/gitweb/?a=blobdiff_plain;f=reports%2Fitemtypes.plugin;h=813435440beff3cc5589311db34715dc8d9efed5;hb=1d882495c2cd28f0ffd3d492aa99611a82ca1825;hp=d541429932b66678bf5c9a59f3513ae1c2d712b7;hpb=6e7a4d3a3da8db196dddb502c09817dd291f3590;p=koha_gimpoz

diff --git a/reports/itemtypes.plugin b/reports/itemtypes.plugin
index d541429932..813435440b 100755
--- a/reports/itemtypes.plugin
+++ b/reports/itemtypes.plugin
@@ -22,7 +22,6 @@ use strict;
 use C4::Auth;
 use CGI;
 use C4::Context;
-use HTML::Template::Pro;
 use C4::Search;
 use C4::Output;
 use C4::Koha;
@@ -36,20 +35,23 @@ sub set_parameters {
 	my $dbh = C4::Context->dbh;
 	my $branches=GetBranches();
 	my @branches;
+	my $default;
 	my @select_branch;
 	my %select_branches;
 	push @select_branch,"";
 	$select_branches{""} = "";
-	foreach my $branch (keys %$branches) {
+	for my $branch (sort { $branches->{$a}->{branchname} cmp $branches->{$b}->{branchname} } keys %$branches) {
 		push @select_branch, $branch;
 		$select_branches{$branch} = $branches->{$branch}->{'branchname'};
+		$default = C4::Context->userenv->{'branch'} if (C4::Context->userenv && C4::Context->userenv->{'branch'});
 	}
 	my $CGIbranch=CGI::scrolling_list( -name     => 'value',
 				-id => 'value',
 				-values   => \@select_branch,
 				-labels   => \%select_branches,
 				-size     => 1,
-				-multiple => 0 );
+				-multiple => 0,
+				-default => $default, );
 	$template->param(CGIbranch => $CGIbranch);
 	return $template;
 }
@@ -66,7 +68,8 @@ sub calculate {
 			FROM itemtypes,items         
 		WHERE items.itype=itemtypes.itemtype         
 		AND items.holdingbranch=?            
-		GROUP BY  items.itype");
+		GROUP BY items.itype
+		ORDER BY itemtypes.description");
 
 		}
 		else {
@@ -76,7 +79,8 @@ sub calculate {
 		WHERE biblioitems.itemtype=itemtypes.itemtype 
 		AND items.biblioitemnumber=biblioitems.biblioitemnumber
 		AND items.holdingbranch=?
-			GROUP BY  biblioitems.itemtype");
+			GROUP BY  biblioitems.itemtype
+			ORDER BY itemtypes.description");
 		}
 		$sth->execute($branch);
 	} else {
@@ -85,14 +89,16 @@ sub calculate {
 		SELECT description,items.itype AS itemtype, COUNT(*) AS total 
 			FROM itemtypes,items
 		WHERE items.itype=itemtypes.itemtype
-			GROUP BY items.itype");
+			GROUP BY items.itype
+			ORDER BY itemtypes.description");
 		}
 		else {
 		$sth = $dbh->prepare("SELECT description, biblioitems.itemtype, COUNT(*) AS total
 			FROM itemtypes, biblioitems,items 
 		WHERE biblioitems.itemtype=itemtypes.itemtype 
 		AND biblioitems.biblioitemnumber = items.biblioitemnumber
-			GROUP BY biblioitems.itemtype");
+			GROUP BY biblioitems.itemtype
+			ORDER BY itemtypes.description");
 		}
 		$sth->execute;
 	}
@@ -117,6 +123,7 @@ sub calculate {
 	$globalline{loopitemtype} = \@results;
 	$globalline{total} = $grantotal;
 	$globalline{branch} = $branch;
+	$globalline{branchname} = GetBranchName($branch);
 	push @mainloop,\%globalline;
 	return \@mainloop;
 }