use CGI;
use C4::Circulation;
use C4::Output;
-use C4::Reserves2;
-use C4::Auth;
-use C4::Interface::CGI::Output;
+use C4::Reserves;
+use C4::Biblio;
+use C4::Items;
+use C4::Auth qw/:DEFAULT get_session/;
use C4::Branch; # GetBranches
use C4::Koha;
-
-###############################################
-# constants
-
-my %env;
-my $branches = GetBranches;
-my $printers = GetPrinters( \%env );
+use C4::Members;
###############################################
# Getting state
my $query = new CGI;
-my $branch = GetBranch( $query, $branches );
-my $printer = GetPrinter( $query, $printers );
+if (!C4::Context->userenv){
+ my $sessionID = $query->cookie("CGISESSID");
+ my $session;
+ $session = get_session($sessionID) if $sessionID;
+ if (!$session or $session->param('branch') eq 'NO_LIBRARY_SET'){
+ # no branch set we can't transfer
+ print $query->redirect("/cgi-bin/koha/circ/selectbranchprinter.pl");
+ exit;
+ }
+}
+
+
+#######################################################################################
+# Make the page .....
+my ( $template, $cookie );
+my $user;
+( $template, $user, $cookie ) = get_template_and_user(
+ {
+ template_name => "circ/branchtransfers.tmpl",
+ query => $query,
+ type => "intranet",
+ authnotrequired => 0,
+ flagsrequired => { circulate => "circulate_remaining_permissions" },
+ }
+);
-my $genbrname = $branches->{$branch}->{'branchname'};
-my $genprname = $printers->{$printer}->{'printername'};
+my $branches = GetBranches;
+my $branch = GetBranch( $query, $branches );
my $messages;
my $found;
my $reqmessage;
my $cancelled;
my $setwaiting;
-my $reqbrchname;
-my $allmessages;
my $request = $query->param('request');
my $borrowernumber = $query->param('borrowernumber');
-
-my $tobranchcd = $query->param('tobranchcd');
-my $frbranchcd = '';
+my $tobranchcd = $query->param('tobranchcd');
############
# Deal with the requests....
CancelReserve( 0, $item, $borrowernumber );
$cancelled = 1;
$reqmessage = 1;
- $allmessages = 1;
}
my $ignoreRs = 0;
if ( $request eq "SetWaiting" ) {
my $item = $query->param('itemnumber');
- $tobranchcd = ReserveWaiting( $item, $borrowernumber );
- $reqbrchname = $branches->{$tobranchcd}->{'branchname'};
+ ModReserveAffect( $item, $borrowernumber );
$ignoreRs = 1;
$setwaiting = 1;
$reqmessage = 1;
- $allmessages = 1;
}
if ( $request eq 'KillReserved' ) {
my $biblio = $query->param('biblionumber');
CancelReserve( $biblio, 0, $borrowernumber );
$cancelled = 1;
$reqmessage = 1;
- $allmessages = 1;
}
# set up the branchselect options....
my @branchoptionloop;
-foreach my $br ( keys %$branches ) {
-
- #(next) unless $branches->{$br}->{'CU'}; #FIXME disabled to fix bug 202
+foreach my $br (sort {$branches->{$a}->{branchname} cmp $branches->{$b}->{branchname}} keys %$branches ) {
my %branch;
$branch{selected} = ( $br eq $tobranchcd );
$branch{code} = $br;
# collect the stack of books already transfered so they can printed...
my @trsfitemloop;
my %transfereditems;
-my %frbranchcds;
-my %tobranchcds;
my $transfered;
my $barcode = $query->param('barcode');
+# strip whitespace
+$barcode =~ s/\s*//g;
+# warn "barcode : $barcode";
if ($barcode) {
my $iteminformation;
( $transfered, $messages, $iteminformation ) =
transferbook( $tobranchcd, $barcode, $ignoreRs );
+# use Data::Dumper;
+# warn "Transfered : $transfered / ".Dumper($messages);
$found = $messages->{'ResFound'};
if ($transfered) {
my %item;
- my $frbranchcd = $iteminformation->{'frbranchcd'};
- if ( not($found) ) {
- $item{'biblionumber'} = $iteminformation->{'biblionumber'};
- $item{'title'} = $iteminformation->{'title'};
- $item{'author'} = $iteminformation->{'author'};
- $item{'itemtype'} = $iteminformation->{'itemtype'};
- $item{'ccode'} = $iteminformation->{'ccode'};
- $item{'frbrname'} = $branches->{$frbranchcd}->{'branchname'};
- $item{'tobrname'} = $branches->{$tobranchcd}->{'branchname'};
- }
+ my $frbranchcd = C4::Context->userenv->{'branch'};
+# if ( not($found) ) {
+ $item{'biblionumber'} = $iteminformation->{'biblionumber'};
+ $item{'title'} = $iteminformation->{'title'};
+ $item{'author'} = $iteminformation->{'author'};
+ $item{'itemtype'} = $iteminformation->{'itemtype'};
+ $item{'ccode'} = $iteminformation->{'ccode'};
+ $item{'frbrname'} = $branches->{$frbranchcd}->{'branchname'};
+ $item{'tobrname'} = $branches->{$tobranchcd}->{'branchname'};
+# }
$item{counter} = 0;
$item{barcode} = $barcode;
$item{frombrcd} = $frbranchcd;
$item{tobrcd} = $tobranchcd;
-##########
- #Are these lines still useful ???
- $transfereditems{0} = $barcode;
- $frbranchcds{0} = $frbranchcd;
- $tobranchcds{0} = $tobranchcd;
-##########
push( @trsfitemloop, \%item );
+# warn Dumper(@trsfitemloop);
}
}
$item{barcode} = $bc;
$item{frombrcd} = $frbcd;
$item{tobrcd} = $tobcd;
- my ($iteminformation) = GetBiblioFromItemNumer( 0, $bc );
+ my ($iteminformation) = GetBiblioFromItemNumber( GetItemnumberFromBarcode($bc) );
$item{'biblionumber'} = $iteminformation->{'biblionumber'};
$item{'title'} = $iteminformation->{'title'};
$item{'author'} = $iteminformation->{'author'};
$item{'ccode'} = $iteminformation->{'ccode'};
$item{'frbrname'} = $branches->{$frbcd}->{'branchname'};
$item{'tobrname'} = $branches->{$tobcd}->{'branchname'};
-##########
- #Are these lines still useful ???
- $transfereditems{$counter} = $bc;
- $frbranchcds{$counter} = $frbcd;
- $tobranchcds{$counter} = $tobcd;
-#########
push( @trsfitemloop, \%item );
}
-my $title;
-my $surname;
-my $firstname;
-my $borphone;
-my $borstraddress;
-my $borcity;
-my $borzip;
-my $boremail;
-my $borcnum;
my $itemnumber;
-my $biblionum;
-my $branchname;
-my $wastransferred;
+my $biblionumber;
#####################
if ($found) {
my $res = $messages->{'ResFound'};
- $branchname = $branches->{ $res->{'branchcode'} }->{'branchname'};
- my ($borr) = GetMemberDetails( $res->{'borrowernumber'}, 0 );
- $title = $borr->{'title'};
- $surname = $borr->{'surname'};
- $firstname = $borr->{'firstname'};
- $borrowernumber = $borr->{'borrowernumber'};
- $borphone = $borr->{'phone'};
- $borstraddress = $borr->{'streetaddress'};
- $borcity = $borr->{'city'};
- $borzip = $borr->{'zipcode'};
- $boremail = $borr->{'emailadress'};
-
- #Hopefully, borr->{borrowernumber}=res->{borrowernumber}
- $borcnum = $borr->{'cardnumber'};
$itemnumber = $res->{'itemnumber'};
if ( $res->{'ResFound'} eq "Waiting" ) {
}
if ( $res->{'ResFound'} eq "Reserved" ) {
$reserved = 1;
- $biblionum = $res->{'biblionumber'};
+ $biblionumber = $res->{'biblionumber'};
}
}
if ( $code eq 'BadBarcode' ) {
$err{msg} = $messages->{'BadBarcode'};
$err{errbadcode} = 1;
- $allmessages = 1;
}
if ( $code eq 'IsPermanent' ) {
$err{errispermanent} = 1;
$err{msg} = $branches->{ $messages->{'IsPermanent'} }->{'branchname'};
-
- # Here, msg contains the branchname
- # Not so satisfied with this... But should work
- $allmessages = 1;
}
$err{errdesteqholding} = ( $code eq 'DestinationEqualsHolding' );
if ( $code eq 'WasReturned' ) {
$err{errwasreturned} = 1;
- $allmessages = 1;
- my ($borrowerinfo) =
- GetMemberDetails( $messages->{'WasReturned'}, 0 );
- $title = $borrowerinfo->{'title'};
- $surname = $borrowerinfo->{'surname'};
- $firstname = $borrowerinfo->{'firstname'};
- $borrowernumber = $borrowerinfo->{'borrowernumber'};
- $borcnum = $borrowerinfo->{'cardnumber'};
+ $err{borrowernumber}=$messages->{'WasReturned'};
+ my $borrower = GetMember($messages->{'WasReturned'},'borrowernumber');
+ $err{title}=$borrower->{'title'};
+ $err{firstname}=$borrower->{'firstname'};
+ $err{surname}=$borrower->{'surname'};
+ $err{cardnumber} =$borrower->{'cardnumber'};
}
-
- # if ($code eq 'WasTransfered'){
- # Put code here if you want to notify the user that item was transfered...
- # $wastransferred = 1;
- # }
push( @errmsgloop, \%err );
}
-#######################################################################################
-# Make the page .....
-my ( $template, $cookie );
-( $template, $borrowernumber, $cookie ) = get_template_and_user(
- {
- template_name => "circ/branchtransfers.tmpl",
- query => $query,
- type => "intranet",
- authnotrequired => 0,
- flagsrequired => { circulate => 1 },
- }
-);
-if ($allmessages) {
- $template->param( allmessages => 1 );
-}
-
+# use Data::Dumper;
+# warn "FINAL ============= ".Dumper(@trsfitemloop);
$template->param(
- genbrname => $genbrname,
- genprname => $genprname,
- branch => $branch,
- printer => $printer,
found => $found,
reserved => $reserved,
waiting => $waiting,
- title => $title,
- surname => $surname,
- firstname => $firstname,
- borphone => $borphone,
- borstraddress => $borstraddress,
- borcity => $borcity,
- borzip => $borzip,
- boremail => $boremail,
borrowernumber => $borrowernumber,
- borcnum => $borcnum,
- branchname => $branchname,
itemnumber => $itemnumber,
barcode => $barcode,
- biblionumber => $biblionum,
+ biblionumber => $biblionumber,
tobranchcd => $tobranchcd,
reqmessage => $reqmessage,
cancelled => $cancelled,
setwaiting => $setwaiting,
- wastransferred => $wastransferred,
trsfitemloop => \@trsfitemloop,
branchoptionloop => \@branchoptionloop,
errmsgloop => \@errmsgloop,
- intranetcolorstylesheet =>
- C4::Context->preference("intranetcolorstylesheet"),
- intranetstylesheet => C4::Context->preference("intranetstylesheet"),
- IntranetNav => C4::Context->preference("IntranetNav"),
+ CircAutocompl => C4::Context->preference("CircAutocompl")
);
output_html_with_http_headers $query, $cookie, $template->output;