use C4::Output;
use C4::Reserves2;
use C4::Auth;
+use HTML::Template;
###############################################
# constants
my $branch = $query->param("branch");
my $printer = $query->param("printer");
-
($branch) || ($branch=$query->cookie('branch')) ;
($printer) || ($printer=$query->cookie('printer')) ;
($branches->{$branch}) || ($branch=(keys %$branches)[0]);
($printers->{$printer}) || ($printer=(keys %$printers)[0]);
-my $request=$query->param('request');
+my $genbrname = $branches->{$branch}->{'branchname'} ;
+my $genprname = $printers->{$printer}->{'printername'};
+my $messages;
+my $found;
+my $reserved;
+my $waiting;
+my $reqmessage;
+my $cancelled;
+my $setwaiting;
+my $reqbrchname;
+
+my $request=$query->param('request');
+my $borrnum = $query->param('borrowernumber');
my $tobranchcd=$query->param('tobranchcd');
my $frbranchcd='';
-# set up the branchselect options....
-my $tobranchoptions;
-foreach my $br (keys %$branches) {
- (next) unless $branches->{$br}->{'CU'};
- my $selected='';
- ($selected='selected') if ($br eq $tobranchcd);
- $tobranchoptions.="<option value=$br $selected>$branches->{$br}->{'branchname'}\n";
-}
-
-# collect the stack of books already transfered so they can printed...
-my %transfereditems;
-my $ritext = '';
-my %frbranchcds;
-my %tobranchcds;
-foreach ($query->param){
- (next) unless (/bc-(\d*)/);
- my $counter=$1;
- my $barcode=$query->param("bc-$counter");
- my $frbcd=$query->param("fb-$counter");
- my $tobcd=$query->param("tb-$counter");
- $counter++;
- $transfereditems{$counter}=$barcode;
- $frbranchcds{$counter}=$frbcd;
- $tobranchcds{$counter}=$tobcd;
- $ritext.="<input type=hidden name=bc-$counter value=$barcode>\n";
- $ritext.="<input type=hidden name=fb-$counter value=$frbcd>\n";
- $ritext.="<input type=hidden name=tb-$counter value=$tobcd>\n";
- }
-
-
-# Collect a few messages here...
-my $messagetext='';
-
############
# Deal with the requests....
if ($request eq "KillWaiting") {
my $item = $query->param('itemnumber');
- my $borrnum = $query->param('borrowernumber');
CancelReserve(0, $item, $borrnum);
- $messagetext .= "Reserve Cancelled<br>";
+ $cancelled = 1;
+ $reqmessage =1;
}
+
my $ignoreRs = 0;
if ($request eq "SetWaiting") {
my $item = $query->param('itemnumber');
- my $borrnum = $query->param('borrowernumber');
$tobranchcd = ReserveWaiting($item, $borrnum);
+ $reqbrchname = $branches->{$tobranchcd}->{'branchname'};
$ignoreRs = 1;
- $messagetext .= "Item should now be waiting at branch: $branches->{$tobranchcd}->{'branchname'}<br>";
+ $setwaiting = 1;
+ $reqmessage =1;
}
if ($request eq 'KillReserved'){
my $biblio = $query->param('biblionumber');
- my $borrnum = $query->param('borrowernumber');
CancelReserve($biblio, 0, $borrnum);
- $messagetext .= "Reserve Cancelled<br>";
+ $cancelled = 1;
+ $reqmessage =1;
}
-
-# Warnings etc that get displayed at top of next page....
-my $messages;
-# if the barcode has been entered action that and write a message
-# and put onto the top of the stack...
-my $iteminformation;
-my $barcode = $query->param('barcode');
-if ($barcode) {
- my $transfered;
- my $iteminformation;
- ($transfered, $messages, $iteminformation)
- = transferbook($tobranchcd, $barcode, $ignoreRs);
- if ($transfered) {
- my $frbranchcd = $iteminformation->{'holdingbranch'};
- $ritext.="<input type=hidden name=bc-0 value=$barcode>\n";
- $ritext.="<input type=hidden name=fb-0 value=$frbranchcd>\n";
- $ritext.="<input type=hidden name=tb-0 value=$tobranchcd>\n";
- $transfereditems{0}=$barcode;
- $frbranchcds{0}=$frbranchcd;
- $tobranchcds{0}=$tobranchcd;
- }
+# set up the branchselect options....
+my @branchoptionloop;
+foreach my $br (keys %$branches) {
+ (next) unless $branches->{$br}->{'CU'};
+ my %branch;
+ $branch{selected}=($br eq $tobranchcd);
+ $branch{code}=$br;
+ $branch{name}=$branches->{$br}->{'branchname'};
+ push (@branchoptionloop, \%branch);
}
-#################################################################################
-# Html code....
-# collect together the various elements...
+# collect the stack of books already transfered so they can printed...
+my @trsfitemloop;
+my %transfereditems;
+my %frbranchcds;
+my %tobranchcds;
+my $color=$linecolor2;
-my $entrytext= << "EOF";
-<form method=post action=/cgi-bin/koha/circ/branchtransfers.pl>
-<table border=1 cellpadding=5 cellspacing=0 align=left>
-<tr><td colspan=2 align=center background=$backgroundimage>
-<font color=black><b>Select Branch</b></font></td></tr>
-<tr><td>Destination Branch:</td>
- <td><select name=tobranchcd> $tobranchoptions </select></td></tr></table>
+my $barcode = $query->param('barcode');
+if ($barcode) {
+ my $transfered;
+ my $iteminformation;
+ ($transfered, $messages, $iteminformation)
+ = transferbook($tobranchcd, $barcode, $ignoreRs);
+ $found = $messages->{'ResFound'};
+ if ($transfered) {
+ my %item;
+ my $frbranchcd = $iteminformation->{'holdingbranch'};
+ if (not ($found)) {
+ ($color eq $linecolor1) ? ($color=$linecolor2) : ($color=$linecolor1);
+ $item{'color'}=$color;
+ $item{'biblionumber'}=$iteminformation->{'biblionumber'};
+ $item{'title'}=$iteminformation->{'title'};
+ $item{'author'}=$iteminformation->{'author'};
+ $item{'itemtype'}=$iteminformation->{'itemtype'};
+ $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);
+ }
+}
-<img src="/images/holder.gif" width=24 height=50 align=left>
+foreach ($query->param){
+ (next) unless (/bc-(\d*)/);
+ my $counter=$1;
+ my %item;
+ my $bc=$query->param("bc-$counter");
+ my $frbcd=$query->param("fb-$counter");
+ my $tobcd=$query->param("tb-$counter");
+ $counter++;
+ $item{counter}=$counter;
+ $item{barcode}=$bc;
+ $item{frombrcd}=$frbcd;
+ $item{tobrcd}=$tobcd;
+ my ($iteminformation) = getiteminformation(\%env, 0, $bc);
+ ($color eq $linecolor1) ? ($color=$linecolor2) : ($color=$linecolor1);
+ $item{'color'}=$color;
+ $item{'biblionumber'}=$iteminformation->{'biblionumber'};
+ $item{'title'}=$iteminformation->{'title'};
+ $item{'author'}=$iteminformation->{'author'};
+ $item{'itemtype'}=$iteminformation->{'itemtype'};
+ $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);
+}
-<table border=1 cellpadding=5 cellspacing=0 ><tr>
-<td colspan=2 align=center background=$backgroundimage>
-<font color=black><b>Enter Book Barcode</b></font></td></tr>
-<tr><td>Item Barcode:</td><td><input name=barcode size=10></td></tr>
-</table>
-<input type=hidden name=tobranchcd value=$tobranchcd>
-$ritext
-</form>
-EOF
+my $name;
+my $bornum;
+my $borcnum;
+my $itemnumber;
+my $biblionum;
+my $branchname;
#####################
-my $reservefoundtext;
-if ($messages->{'ResFound'}) {
+if ($found) {
my $res = $messages->{'ResFound'};
- my $reservetext;
- my $branchname = $branches->{$res->{'branchcode'}}->{'branchname'};
- my ($borr) = getpatroninformation(\%env, $res->{'borrowernumber'}, 0);
- my $name = $borr->{'surname'}." ".$borr->{'title'}." ".$borr->{'firstname'};
- my $number = "<a href=/cgi-bin/koha/moremember.pl?bornum=$borr->{'borrowernumber'} onClick='openWindow(this,'Member', 480, 640)'>$borr->{'cardnumber'}</a>";
- if ($res->{'ResFound'} eq "Waiting") {
- $reservetext = <<"EOF";
-<font color='red' size='+2'>Item marked Waiting:</font><br>
- Item is marked waiting at $branchname for $name ($number).<br>
-<table border=1 cellpadding=5 cellspacing=0>
-<tr><td>Cancel reservation and then attempt transfer: </td>
-<td>
-<form method=post action='branchtransfers.pl'>
-$ritext
-<input type=hidden name=itemnumber value=$res->{'itemnumber'}>
-<input type=hidden name=borrowernumber value=$res->{'borrowernumber'}>
-<input type=hidden name=tobranchcd value=$tobranchcd>
-<input type=hidden name=barcode value=$barcode>
-<input type=hidden name=request value='KillWaiting'>
-<input type=submit value="Cancel">
-</form>
-</td></tr>
-<tr><td>Ignore and return to transfers: </td>
-<td>
-<form method=post action='branchtransfers.pl'>
-$ritext
-<input type=hidden name=tobranchcd value=$tobranchcd>
-<input type=hidden name=barcode value=0>
-<input type=submit value="Ignore">
-</form>
-</td></tr></table>
-EOF
- }
- if ($res->{'ResFound'} eq "Reserved") {
- $reservetext = <<"EOF";
-<font color='red' size='+2'>Reserved:</font> reserve found for $name ($number).
-<table border=1 cellpadding=5 cellspacing=0>
-<tr><td>Set reserve to waiting and transfer book to <b>$branchname </b>: </td>
-<td>
-<form method=post action='branchtransfers.pl'>
-$ritext
-<input type=hidden name=itemnumber value=$res->{'itemnumber'}>
-<input type=hidden name=borrowernumber value=$res->{'borrowernumber'}>
-<input type=hidden name=barcode value=$barcode>
-<input type=hidden name=request value='SetWaiting'>
-<input type=submit value="Waiting">
-</form>
-</td></tr>
-<tr><td>Cancel reservation and then attempt transfer: </td>
-<td>
-<form method=post action='branchtransfers.pl'>
-$ritext
-<input type=hidden name=biblionumber value=$res->{'biblionumber'}>
-<input type=hidden name=borrowernumber value=$res->{'borrowernumber'}>
-<input type=hidden name=tobranchcd value=$tobranchcd>
-<input type=hidden name=barcode value=$barcode>
-<input type=hidden name=request value='KillReserved'>
-<input type=submit value="Cancel">
-</form>
-</td></tr><tr><td>Ignore and return to transfers: </td>
-<td>
-<form method=post action='branchtransfers.pl'>
-<input type=hidden name=tobranchcd value=$tobranchcd>
-<input type=hidden name=barcode value=0>
-$ritext
-<input type=submit value="Ignore">
-</form>
-</td></tr></table>
-EOF
- }
- $reservefoundtext = <<"EOF";
-<table border=1 cellpadding=5 cellspacing=0 bgcolor='#dddddd'>
-<tr><th bgcolor=$headerbackgroundcolor background=$backgroundimage><font>Reserve Found</font></th></tr>
-<tr><td> $reservetext </td></tr></table>
-EOF
+ $branchname = $branches->{$res->{'branchcode'}}->{'branchname'};
+ my ($borr) = getpatroninformation(\%env, $res->{'borrowernumber'}, 0);
+ $name = name($borr);
+ $bornum = $borr->{'borrowernumber'}; #Hopefully, borr->{borrowernumber}=res->{borrowernumber}
+ $borcnum = $borr->{'cardnumber'};
+ $itemnumber = $res->{'itemnumber'};
+
+ if ($res->{'ResFound'} eq "Waiting") {
+ $waiting = 1;
+ }
+ if ($res->{'ResFound'} eq "Reserved") {
+ $reserved = 1;
+ $biblionum = $res->{'biblionumber'};
+ }
}
#####################
+my @errmsgloop;
foreach my $code (keys %$messages) {
- if ($code eq 'BadBarcode'){
- $messagetext .= "<font color='red' size='+2'> No Item with barcode: $messages->{'BadBarcode'} </font> <br>";
- }
+ my %err;
+ $err{errbadcode} = ($code eq 'BadBarcode');
+ if ($code eq 'BadBarcode') {
+ $err{msg}=$messages->{'BadBarcode'};
+ }
+
+ $err{errispermanent} = ($code eq 'IsPermanent');
if ($code eq 'IsPermanent'){
- my $braname = $branches->{$messages->{'IsPermanent'}}->{'branchname'};
- $messagetext .= "<font color='red' size='+2'> Please return item to home branch: $braname </font> <br>";
+ $err{msg} = $branches->{$messages->{'IsPermanent'}}->{'branchname'};
+ # Here, msg contains the branchname
+ # Not so satisfied with this... But should work
}
- if ($code eq 'DestinationEqualsHolding'){
- $messagetext .= "<font color='red' size='+2'> Item is already at destination branch. </font> <br>";
- }
- if ($code eq 'WasReturned') {
- my ($borrowerinfo) = getpatroninformation(\%env, $messages->{'WasReturned'}, 0);
-
- my $binfo = <<"EOF";
-<a href=/cgi-bin/koha/moremember.pl?bornum=$borrowerinfo->{'borrowernumber'}
-onClick="openWindow(this,'Member', 480, 640)">$borrowerinfo->{'cardnumber'}</a>
-$borrowerinfo->{'surname'}, $borrowerinfo->{'title'} $borrowerinfo->{'firstname'}
-EOF
- $messagetext .= "Item was on loan to $binfo and has been returned. <br>";
+ $err{errdesteqholding} = ($code eq 'DestinationEqualsHolding');
+
+ $err{errwasreturned} = ($code eq 'WasReturned');
+ if ($code eq 'WasReturned') {
+ my ($borrowerinfo) = getpatroninformation(\%env, $messages->{'WasReturned'}, 0);
+ $name =name($borrowerinfo);
+ $bornum =$borrowerinfo->{'borrowernumber'};
+ $borcnum =$borrowerinfo->{'cardnumber'};
}
if ($code eq 'WasTransfered'){
# Put code here if you want to notify the user that item was transfered...
}
+ push (@errmsgloop, \%err);
}
-$messagetext = substr($messagetext, 0, -4);
-
-my $messagetable;
-if ($messagetext) {
- $messagetable = << "EOF";
-<table border=1 cellpadding=5 cellspacing=0 bgcolor='#dddddd'>
-<tr><th bgcolor=$headerbackgroundcolor background=$backgroundimage><font>Messages</font></th></tr>
-<tr><td> $messagetext </td></tr></table>
-<img src="/images/holder.gif" width=24 height=24>
-EOF
-}
+
#######################################################################################
# Make the page .....
-
-print $query->header(-cookie=>$sessioncookie);
-print startpage;
-#print startmenu('circulation');
-my @inp=startmenu('circulation');
-my $count=@inp;
-for (my $i=0;$i<$count;$i++){
- $inp[$i]=~ s/document.forms\[0\].elements\[0\]/document\.forms\[0\]\.elements\[1\]/;
- }
-print @inp;
-
-#foreach my $key (%$messages) {
-# print $key." : ".$messages->{$key}."<br>";
-#}
-
-print <<"EOF";
-<table align=right><tr><td>
-<a href=circulation.pl>
-<img src="/images/button-issues.gif" width="99" height="42" border="0" alt="Next Borrower"></a>  
-<a href=returns.pl>
-<img src="/images/button-returns.gif" width="110" height="42" border="0" alt="Returns"></a>
-</td></tr></table>
-
-<FONT SIZE=6><em>Circulation: Transfers</em></FONT><br>
-<b>Branch:</b> $branches->{$branch}->{'branchname'}  
-<b>Printer:</b> $printers->{$printer}->{'printername'}<br>
-<a href=selectbranchprinter.pl>Change Settings</a>
-<input type=hidden name=branch value=$branch>
-<input type=hidden name=printer value=$printer>
-<p>
-EOF
-
-if ($reservefoundtext) {
- print $reservefoundtext;
-} else {
- print $messagetable;
- print $entrytext;
-
- if (%transfereditems) {
- print << "EOF";
-<p>
-<table border=1 cellpadding=5 cellspacing=0 bgcolor=#dddddd>
-<tr><th colspan=6 bgcolor=$headerbackgroundcolor background=$backgroundimage><font color=black>Transfered Items</font></th></tr>
-<tr><th>Bar Code</th><th>Title</th><th>Author</th><th>Type</th><th>From</th><th>To</th></tr>
-EOF
- my $color='';
- foreach (sort keys %transfereditems) {
- ($color eq $linecolor1) ? ($color=$linecolor2) : ($color=$linecolor1);
- my $barcode=$transfereditems{$_};
- my $frbcd=$frbranchcds{$_};
- my $tobcd=$tobranchcds{$_};
- my ($iteminformation) = getiteminformation(\%env, 0, $barcode);
- print << "EOF";
-<tr><td bgcolor=$color align=center>
-<a href=/cgi-bin/koha/detail.pl?bib=$iteminformation->{'biblionumber'}
-&type=intra onClick=\"openWindow(this, 'Item', 480, 640)\">$barcode</a></td>
-<td bgcolor=$color>$iteminformation->{'title'}</td>
-<td bgcolor=$color>$iteminformation->{'author'}</td>
-<td bgcolor=$color align=center>$iteminformation->{'itemtype'}</td>
-<td bgcolor=$color align=center>$branches->{$frbcd}->{'branchname'}</td>
-<td bgcolor=$color align=center>$branches->{$tobcd}->{'branchname'}</td>
-</tr>\n
-EOF
- }
- print "</table>\n";
- }
+my ($template, $borrowernumber, $cookie)
+ = get_template_and_user({template_name => "circ/branchtransfers.tmpl",
+ query => $query,
+ type => "intranet",
+ authnotrequired => 0,
+ flagsrequired => {parameters => 1},
+ });
+$template->param( genbrname => $genbrname,
+ genprname => $genprname,
+ branch => $branch,
+ printer => $printer,
+ found => $found,
+ hdrbckgdcolor => $headerbackgroundcolor,
+ bckgdimg => $backgroundimage,
+ reserved => $reserved,
+ waiting => $waiting,
+ name => $name,
+ bornum => $bornum,
+ borcnum => $borcnum,
+ branchname => $branchname,
+ itemnumber => $itemnumber,
+ barcode => $barcode,
+ biblionumber => $biblionum,
+ tobranchcd => $tobranchcd,
+ reqmessage => $reqmessage,
+ cancelled => $cancelled,
+ setwaiting => $setwaiting,
+ trsfitemloop => \@trsfitemloop,
+ branchoptionloop => \@branchoptionloop,
+ errmsgloop => \@errmsgloop
+ );
+print $query->header(-cookie=>$sessioncookie), $template->output;
+
+
+sub name {
+ my ($borinfo) = @_;
+ return $borinfo->{'surname'}." ".$borinfo->{'title'}." ".$borinfo->{'firstname'};
}
-
-print endmenu('circulation');
-print endpage;
-
-