X-Git-Url: http://git.rot13.org/?a=blobdiff_plain;f=circ%2Ftransferstoreceive.pl;h=522d922841b55c138c29ebccbb8178b989801f78;hb=9b01fd97ea323b00629709d04f50cdd27e45dfa5;hp=697a41a71c96ca615a77b4b89817d29ba4f50e55;hpb=606ecb532aa42562687bf0d05b4d6df6697300d7;p=koha.git diff --git a/circ/transferstoreceive.pl b/circ/transferstoreceive.pl index 697a41a71c..522d922841 100755 --- a/circ/transferstoreceive.pl +++ b/circ/transferstoreceive.pl @@ -14,17 +14,18 @@ # 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; use CGI; use C4::Context; use C4::Output; -use C4::Branch; +use C4::Branch; # GetBranches use C4::Auth; -use C4::Date; +use C4::Dates qw/format_date/; use C4::Biblio; use C4::Circulation; use C4::Members; @@ -38,23 +39,15 @@ use C4::Koha; use C4::Reserves; my $input = new CGI; - -my $theme = $input->param('theme'); # only used if allowthemeoverride is set my $itemnumber = $input->param('itemnumber'); -my $todaysdate = join "-", &Today; - -# if we have a resturn of the form to delete the transfer, we launch the subrroutine -if ($itemnumber) { - C4::Circulation::Circ2::DeleteTransfer($itemnumber); -} my ( $template, $loggedinuser, $cookie ) = get_template_and_user( { - template_name => "circ/transferstoreceive.tmpl", + template_name => "circ/transferstoreceive.tt", query => $input, type => "intranet", authnotrequired => 0, - flagsrequired => { circulate => 1 }, + flagsrequired => { circulate => "circulate_remaining_permissions" }, debug => 1, } ); @@ -65,6 +58,7 @@ my $default = C4::Context->userenv->{'branch'}; # get the all the branches for reference my $branches = GetBranches(); my @branchesloop; +my $latetransfers; foreach my $br ( keys %$branches ) { my @transferloop; my %branchloop; @@ -85,37 +79,34 @@ foreach my $br ( keys %$branches ) { C4::Context->preference('TransfersMaxDaysWarning')); my $calcDate = Date_to_Days( $sent_year, $sent_month, $sent_day ); my $today = Date_to_Days(&Today); - my $warning = ( $today > $calcDate ); + my $diff = $today - $calcDate; - if ( $warning > 0 ) { + if ($today > $calcDate) { + $latetransfers = 1; $getransf{'messcompa'} = 1; + $getransf{'diff'} = $diff; } my $gettitle = GetBiblioFromItemNumber( $num->{'itemnumber'} ); - my $itemtypeinfo = getitemtypeinfo( $gettitle->{'itemtype'} ); - - $getransf{'title'} = $gettitle->{'title'}; - $getransf{'datetransfer'} = format_date( $num->{'datesent'} ); - $getransf{'biblionumber'} = $gettitle->{'biblionumber'}; - $getransf{'itemnumber'} = $gettitle->{'itemnumber'}; - $getransf{'barcode'} = $gettitle->{'barcode'}; - $getransf{'itemtype'} = $itemtypeinfo->{'description'}; - $getransf{'homebranch'} = $gettitle->{'homebranch'}; - $getransf{'holdingbranch'} = $gettitle->{'holdingbranch'}; - $getransf{'itemcallnumber'} = $gettitle->{'itemcallnumber'}; - - # we check if we have a reserv for this transfer - my @checkreserv = GetReservesFromItemnumber($num->{'itemnumber'} ); + my $itemtypeinfo = getitemtypeinfo( (C4::Context->preference('item-level_itypes')) ? $gettitle->{'itype'} : $gettitle->{'itemtype'} ); + + $getransf{'datetransfer'} = $num->{'datesent'}; + $getransf{'itemtype'} = $itemtypeinfo ->{'description'}; + foreach (qw(title author biblionumber itemnumber barcode homebranch holdingbranch itemcallnumber)) { + $getransf{$_} = $gettitle->{$_}; + } + + my $record = GetMarcBiblio($gettitle->{'biblionumber'}); + $getransf{'subtitle'} = GetRecordValue('subtitle', $record, GetFrameworkCode($gettitle->{'biblionumber'})); + + # we check if we have a reserv for this transfer + my @checkreserv = GetReservesFromItemnumber($num->{'itemnumber'}); if ( $checkreserv[0] ) { - my $getborrower = - GetMemberDetails( $checkreserv[1] ); - $getransf{'borrowernum'} = $getborrower->{'borrowernumber'}; - $getransf{'borrowername'} = $getborrower->{'surname'}; + my $getborrower = GetMemberDetails( $checkreserv[1] ); + $getransf{'borrowernum'} = $getborrower->{'borrowernumber'}; + $getransf{'borrowername'} = $getborrower->{'surname'}; $getransf{'borrowerfirstname'} = $getborrower->{'firstname'}; - if ( $getborrower->{'emailaddress'} ) { - $getransf{'borrowermail'} = $getborrower->{'emailaddress'}; - } - $getransf{'borrowerphone'} = $getborrower->{'phone'}; - + $getransf{'borrowermail'} = $getborrower->{'emailaddress'} if $getborrower->{'emailaddress'}; + $getransf{'borrowerphone'} = $getborrower->{'phone'}; } push( @transferloop, \%getransf ); } @@ -128,7 +119,9 @@ foreach my $br ( keys %$branches ) { $template->param( branchesloop => \@branchesloop, - show_date => format_date($todaysdate), + show_date => format_date(C4::Dates->today('iso')), + TransfersMaxDaysWarning => C4::Context->preference('TransfersMaxDaysWarning'), + latetransfers => $latetransfers ? 1 : 0, ); output_html_with_http_headers $input, $cookie, $template->output;