X-Git-Url: http://koha-dev.rot13.org:8081/gitweb/?a=blobdiff_plain;f=admin%2Fbranch_transfer_limits.pl;h=1d8593790ba67ba17f6491dd4686f7b9e6265695;hb=7a71d2c5e8f5a6685b0382217f83c98f850ef269;hp=344b70cdf01e7dd2598784adbd358544a9cc0c14;hpb=73dd78ce1cb9ee58d036290294834afd97cfbeeb;p=koha_gimpoz diff --git a/admin/branch_transfer_limits.pl b/admin/branch_transfer_limits.pl index 344b70cdf0..1d8593790b 100755 --- a/admin/branch_transfer_limits.pl +++ b/admin/branch_transfer_limits.pl @@ -1,6 +1,7 @@ #!/usr/bin/perl # Copyright 2000-2002 Katipo Communications +# copyright 2010 BibLibre # # This file is part of Koha. # @@ -13,9 +14,9 @@ # WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS FOR # A PARTICULAR PURPOSE. See the GNU General Public License for more details. # -# You should have received a copy of the GNU General Public License along with -# Koha; if not, write to the Free Software Foundation, Inc., 59 Temple Place, -# Suite 330, Boston, MA 02111-1307 USA +# You should have received a copy of the GNU General Public License along +# with Koha; if not, write to the Free Software Foundation, Inc., +# 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA. use strict; use warnings; @@ -39,6 +40,28 @@ my ($template, $loggedinuser, $cookie) }); my $dbh = C4::Context->dbh; +my $branchcode; +if((!defined($input->param('branchcode'))) & mybranch() ne '') +{ + $branchcode = mybranch(); +} +else +{ + $branchcode = $input->param('branchcode'); +} + +my $branchname = GetBranchName($branchcode); + +# Getting the branches for user selection +my $branches = GetBranches(); +my @branch_loop; +for my $thisbranch (sort { $branches->{$a}->{branchname} cmp $branches->{$b}->{branchname} } keys %$branches) { + my %row =(value => $thisbranch, + branchname => $branches->{$thisbranch}->{'branchname'}, + ); + push @branch_loop, \%row; +} + # Set the template language for the correct limit type my $limit_phrase = 'Collection Code'; @@ -69,15 +92,13 @@ while ( my $row = $sth->fetchrow_hashref ) { ## If Form Data Passed, Update the Database if ( $input->param('updateLimits') ) { - DeleteBranchTransferLimits(); + DeleteBranchTransferLimits(); foreach my $code ( @codes ) { foreach my $toBranch ( @branchcodes ) { - foreach my $fromBranch ( @branchcodes ) { - my $isSet = $input->param( $code . "_" . $toBranch . "_" . $fromBranch ); - if ( $isSet ) { - CreateBranchTransferLimit( $toBranch, $fromBranch, $code ); - } + my $isSet = not $input->param( $code . "_" . $toBranch); + if ( $isSet ) { + CreateBranchTransferLimit( $toBranch, $branchcode, $code ); } } } @@ -100,23 +121,12 @@ foreach my $code ( @codes ) { $row_data{ code } = $code; $row_data{ to_branch_loop } = \@to_branch_loop; foreach my $toBranch ( @branchcodes ) { - my @from_branch_loop; my %row_data; - $row_data{ code } = $code; - $row_data{ toBranch } = $toBranch; - $row_data{ from_branch_loop } = \@from_branch_loop; - - foreach my $fromBranch ( @branchcodes ) { - my %row_data; - my $isChecked = ! IsBranchTransferAllowed( $toBranch, $fromBranch, $code ); - $row_data{ code } = $code; - $row_data{ toBranch } = $toBranch; - $row_data{ fromBranch } = $fromBranch; - $row_data{ isChecked } = $isChecked; - - push( @from_branch_loop, \%row_data ); - } - + my $isChecked = IsBranchTransferAllowed( $toBranch, $branchcode, $code ); + $row_data{ code } = $code; + $row_data{ toBranch } = $toBranch; + $row_data{ isChecked } = $isChecked; + $row_data{ toBranchname } = GetBranchName($toBranch); push( @to_branch_loop, \%row_data ); } @@ -127,7 +137,10 @@ foreach my $code ( @codes ) { $template->param( branchcount => $branchcount, codes_loop => \@codes_loop, + branch_loop => \@branch_loop, branchcode_loop => \@branchcode_loop, + branchcode => $branchcode, + branchname => $branchname, limit_phrase => $limit_phrase, );