X-Git-Url: http://koha-dev.rot13.org:8081/gitweb/?a=blobdiff_plain;f=members%2Fmoremember.pl;h=bfaa4029f1d2711453ba426731f65774738dd1d0;hb=87f60acf9afdecc28603b0a53005230c35afeb89;hp=2542a7042ecc8a8ce3c830d631da4ddf2a0d68f4;hpb=606ecb532aa42562687bf0d05b4d6df6697300d7;p=koha_gimpoz diff --git a/members/moremember.pl b/members/moremember.pl index 2542a7042e..bfaa4029f1 100755 --- a/members/moremember.pl +++ b/members/moremember.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. =head1 moremember.pl @@ -32,11 +33,14 @@ =cut use strict; +#use warnings; FIXME - Bug 2505 use CGI; -use C4::Auth; use C4::Context; +use C4::Auth; use C4::Output; use C4::Members; +use C4::Members::Attributes; +use C4::Members::AttributeTypes; use C4::Dates; use C4::Reserves; use C4::Circulation; @@ -45,16 +49,56 @@ use C4::Letters; use C4::Biblio; use C4::Reserves; use C4::Branch; # GetBranchName +use C4::Form::MessagingPreferences; +use C4::NewsChannels; #get slip news +use List::MoreUtils qw/uniq/; + +#use Smart::Comments; +#use Data::Dumper; + +use vars qw($debug); + +BEGIN { + $debug = $ENV{DEBUG} || 0; +} my $dbh = C4::Context->dbh; my $input = new CGI; +$debug or $debug = $input->param('debug') || 0; my $print = $input->param('print'); -my $template_name; +my $override_limit = $input->param("override_limit") || 0; +my @failedrenews = $input->param('failedrenew'); +my @failedreturns = $input->param('failedreturn'); +my $error = $input->param('error'); +my %renew_failed; +for my $renew (@failedrenews) { $renew_failed{$renew} = 1; } +my %return_failed; +for my $failedret (@failedreturns) { $return_failed{$failedret} = 1; } -if ($print eq "page") { $template_name = "members/moremember-print.tmpl"; } -elsif ($print eq "slip") { $template_name = "members/moremember-receipt.tmpl"; } -else { $template_name = "members/moremember.tmpl"; } +my $template_name; +my $quickslip = 0; + +my $flagsrequired; +if ($print eq "page") { + $template_name = "members/moremember-print.tmpl"; + $flagsrequired = { borrowers => 1 }; +} elsif ($print eq "slip") { + $template_name = "members/moremember-receipt.tmpl"; + # circ staff who process checkouts but can't edit + # patrons still need to be able to print receipts + $flagsrequired = { circulate => "circulate_remaining_permissions" }; +} elsif ($print eq "qslip") { + $template_name = "members/moremember-receipt.tmpl"; + $quickslip = 1; + $flagsrequired = { circulate => "circulate_remaining_permissions" }; +} elsif ($print eq "brief") { + $template_name = "members/moremember-brief.tmpl"; + $flagsrequired = { borrowers => 1 }; +} else { + $template_name = "members/moremember.tmpl"; + $flagsrequired = { borrowers => 1 }; +} my ( $template, $loggedinuser, $cookie ) = get_template_and_user( { @@ -62,38 +106,45 @@ my ( $template, $loggedinuser, $cookie ) = get_template_and_user( query => $input, type => "intranet", authnotrequired => 0, - flagsrequired => { borrowers => 1 }, + flagsrequired => $flagsrequired, debug => 1, } ); my $borrowernumber = $input->param('borrowernumber'); #start the page and read in includes -my $data = GetMember( $borrowernumber ,'borrowernumber'); +my $data = GetMember( 'borrowernumber' => $borrowernumber ); my $reregistration = $input->param('reregistration'); if ( not defined $data ) { $template->param (unknowuser => 1); + output_html_with_http_headers $input, $cookie, $template->output; exit; } # re-reregistration function to automatic calcul of date expiry -( -# $data->{'dateexpiry'} = GetMembeReregistration( - $data->{'dateexpiry'} = ExtendMemberSubscriptionTo( - $data->{'categorycode'}, - $borrowernumber, $data->{'dateenrolled'} - ) -) if ( $reregistration eq 'y' ); -my $borrowercategory = GetBorrowercategory( $data->{'categorycode'} ); -my $category_type = $borrowercategory->{'category_type'}; +if ( $reregistration eq 'y' ) { + $data->{'dateexpiry'} = ExtendMemberSubscriptionTo( $borrowernumber ); +} + +my $category_type = $data->{'category_type'}; + +### $category_type # in template => instutitional (A for Adult& C for children) $template->param( $data->{'categorycode'} => 1 ); +$debug and printf STDERR "dates (enrolled,expiry,birthdate) raw: (%s, %s, %s)\n", map {$data->{$_}} qw(dateenrolled dateexpiry dateofbirth); foreach (qw(dateenrolled dateexpiry dateofbirth)) { - $data->{$_} = C4::Dates->new($data->{$_}, 'iso')->output() - or die ("failed C4::Dates->new(" . $data->{$_} . ", 'iso')->output()"); + my $userdate = $data->{$_}; + unless ($userdate) { + $debug and warn sprintf "Empty \$data{%12s}", $_; + $data->{$_} = ''; + next; + } + $userdate = C4::Dates->new($userdate,'iso')->output('syspref'); + $data->{$_} = $userdate || ''; + $template->param( $_ => $userdate ); } $data->{'IS_ADULT'} = ( $data->{'categorycode'} ne 'I' ); @@ -102,20 +153,22 @@ for (qw(debarred gonenoaddress lost borrowernotes)) { } $data->{'ethnicity'} = fixEthnicity( $data->{'ethnicity'} ); - $data->{ "sex_".$data->{'sex'}."_p" } = 1; -if ( $category_type eq 'C' and $data->{'guarantorid'} ne '0' ) { - my $data2 = GetMember( $data->{'guarantorid'} ,'borrowernumber'); - foreach (qw(address city B_address B_city phone mobilezipcode)) { - $data->{$_} = $data2->{$_}; - } +my $catcode; +if ( $category_type eq 'C') { + my ( $catcodes, $labels ) = GetborCatFromCatType( 'A', 'WHERE category_type = ?' ); + my $cnt = scalar(@$catcodes); + + $template->param( 'CATCODE_MULTI' => 1) if $cnt > 1; + $template->param( 'catcode' => $catcodes->[0]) if $cnt == 1; } + if ( $data->{'ethnicity'} || $data->{'ethnotes'} ) { $template->param( printethnicityline => 1 ); } -if ( $category_type eq 'A' ) { +if ( $category_type eq 'A' || $category_type eq 'I') { $template->param( isguarantee => 1 ); # FIXME @@ -135,24 +188,21 @@ if ( $category_type eq 'A' ) { ); } $template->param( guaranteeloop => \@guaranteedata ); - ( $template->param( adultborrower => 1 ) ) if ( $category_type eq 'A' ); + ( $template->param( adultborrower => 1 ) ) if ( $category_type eq 'A' || $category_type eq 'I' ); } else { if ($data->{'guarantorid'}){ - my ($guarantor) = GetMember( $data->{'guarantorid'},'biblionumber'); - $template->param(guarantor => 1); + my ($guarantor) = GetMember( 'borrowernumber' =>$data->{'guarantorid'}); + $template->param(guarantor => 1); foreach (qw(borrowernumber cardnumber firstname surname)) { $template->param("guarantor$_" => $guarantor->{$_}); } } + if ($category_type eq 'C'){ + $template->param('C' => 1); + } } -#Independant branches management -my $unvalidlibrarian = - ( ( C4::Context->preference("IndependantBranches") ) - && ( C4::Context->userenv->{flags} != 1 ) - && ( $data->{'branchcode'} ne C4::Context->userenv->{branch} ) ); - my %bor; $bor{'borrowernumber'} = $borrowernumber; @@ -160,57 +210,122 @@ $bor{'borrowernumber'} = $borrowernumber; my $samebranch; if ( C4::Context->preference("IndependantBranches") ) { my $userenv = C4::Context->userenv; - unless ( $userenv->{flags} == 1 ) { + unless ( $userenv->{flags} % 2 == 1 ) { $samebranch = ( $data->{'branchcode'} eq $userenv->{branch} ); } - $samebranch = 1 if ( $userenv->{flags} == 1 ); + $samebranch = 1 if ( $userenv->{flags} % 2 == 1 ); +}else{ + $samebranch = 1; } my $branchdetail = GetBranchDetail( $data->{'branchcode'}); -$data->{'branchname'} = $branchdetail->{branchname}; - +@{$data}{keys %$branchdetail} = values %$branchdetail; # merge in all branch columns my ( $total, $accts, $numaccts) = GetMemberAccountRecords( $borrowernumber ); my $lib1 = &GetSortDetails( "Bsort1", $data->{'sort1'} ); my $lib2 = &GetSortDetails( "Bsort2", $data->{'sort2'} ); -( $template->param( lib1 => $lib1 ) ) if ($lib1); -( $template->param( lib2 => $lib2 ) ) if ($lib2); +$template->param( lib1 => $lib1 ) if ($lib1); +$template->param( lib2 => $lib2 ) if ($lib2); + +# Show OPAC privacy preference is system preference is set +if ( C4::Context->preference('OPACPrivacy') ) { + $template->param( OPACPrivacy => 1); + $template->param( "privacy".$data->{'privacy'} => 1); +} # current issues # -my ( $count, $issue ) = GetPendingIssues($borrowernumber); +my @borrowernumbers = GetMemberRelatives($borrowernumber); +my $issue = GetPendingIssues($borrowernumber); +my $relissue = []; +if ( @borrowernumbers ) { + $relissue = GetPendingIssues(@borrowernumbers); +} +my $issuecount = @{$issue}; +my $relissuecount = @{$relissue}; my $roaddetails = &GetRoadTypeDetails( $data->{'streettype'} ); -my $today = POSIX::strftime("%Y%m%d", localtime); # iso format +my $today = POSIX::strftime("%Y-%m-%d", localtime); # iso format my @issuedata; +my @borrowers_with_issues; +my $overdues_exist = 0; my $totalprice = 0; -my $toggle = 0; -for ( my $i = 0 ; $i < $count ; $i++ ) { - my $datedue = $issue->[$i]{'date_due'}; - $issue->[$i]{'date_due'} = C4::Dates->new($issue->[$i]{'date_due'})->output('iso'); - my %row = %{ $issue->[$i] }; - $totalprice += $issue->[$i]{'replacementprice'}; - $row{'replacementprice'} = $issue->[$i]{'replacementprice'}; - if ( $datedue < $today ) { - $row{'red'} = 1; #print ""; - } - $row{toggle} = $toggle++ % 2; - - #find the charge for an item - my ( $charge, $itemtype ) = - GetIssuingCharges( $issue->[$i]{'itemnumber'}, $borrowernumber ); - - my $itemtypeinfo = getitemtypeinfo($itemtype); - $row{'itemtype_description'} = $itemtypeinfo->{description}; - $row{'itemtype_image'} = $itemtypeinfo->{imageurl}; - - $row{'charge'} = sprintf( "%.2f", $charge ); - #check item is not reserved - my ( $restype, $reserves ) = CheckReserves( $issue->[$i]{'itemnumber'} ); - $row{'norenew'} = ($restype) ? 1 : 0; - push( @issuedata, \%row ); +my @issuedata = build_issue_data($issue, $issuecount); +my @relissuedata = build_issue_data($relissue, $relissuecount); + +sub build_issue_data { + my $issue = shift; + my $issuecount = shift; + + my $localissue; + + for ( my $i = 0 ; $i < $issuecount ; $i++ ) { + # Getting borrower details + my $memberdetails = GetMemberDetails($issue->[$i]{'borrowernumber'}); + $issue->[$i]{'borrowername'} = $memberdetails->{'firstname'} . " " . $memberdetails->{'surname'}; + $issue->[$i]{'cardnumber'} = $memberdetails->{'cardnumber'}; + my $datedue = $issue->[$i]{'date_due'}; + my $issuedate = $issue->[$i]{'issuedate'}; + $issue->[$i]{'date_due'} = C4::Dates->new($issue->[$i]{'date_due'}, 'iso')->output('syspref'); + $issue->[$i]{'issuedate'} = C4::Dates->new($issue->[$i]{'issuedate'},'iso')->output('syspref'); + my $biblionumber = $issue->[$i]{'biblionumber'}; + my %row = %{ $issue->[$i] }; + $totalprice += $issue->[$i]{'replacementprice'}; + $row{'replacementprice'} = $issue->[$i]{'replacementprice'}; + # item lost, damaged loops + if ($row{'itemlost'}) { + my $fw = GetFrameworkCode($issue->[$i]{'biblionumber'}); + my $category = GetAuthValCode('items.itemlost',$fw); + my $lostdbh = C4::Context->dbh; + my $sth = $lostdbh->prepare("select lib from authorised_values where category=? and authorised_value =? "); + $sth->execute($category, $row{'itemlost'}); + my $loststat = $sth->fetchrow; + if ($loststat) { + $row{'itemlost'} = $loststat; + } + } + if ($row{'damaged'}) { + my $fw = GetFrameworkCode($issue->[$i]{'biblionumber'}); + my $category = GetAuthValCode('items.damaged',$fw); + my $damageddbh = C4::Context->dbh; + my $sth = $damageddbh->prepare("select lib from authorised_values where category=? and authorised_value =? "); + $sth->execute($category, $row{'damaged'}); + my $damagedstat = $sth->fetchrow; + if ($damagedstat) { + $row{'itemdamaged'} = $damagedstat; + } + } + # end lost, damaged + if ( $datedue lt $today ) { + $overdues_exist = 1; + $row{'red'} = 1; + } + if ( $issuedate eq $today ) { + $row{'today'} = 1; + } + + #find the charge for an item + my ( $charge, $itemtype ) = + GetIssuingCharges( $issue->[$i]{'itemnumber'}, $borrowernumber ); + + my $itemtypeinfo = getitemtypeinfo($itemtype); + $row{'itemtype_description'} = $itemtypeinfo->{description}; + $row{'itemtype_image'} = $itemtypeinfo->{imageurl}; + + $row{'charge'} = sprintf( "%.2f", $charge ); + + my ( $renewokay,$renewerror ) = CanBookBeRenewed( $borrowernumber, $issue->[$i]{'itemnumber'}, $override_limit ); + $row{'norenew'} = !$renewokay; + $row{'can_confirm'} = ( !$renewokay && $renewerror ne 'on_reserve' ); + $row{"norenew_reason_$renewerror"} = 1 if $renewerror; + $row{'renew_failed'} = $renew_failed{ $issue->[$i]{'itemnumber'} }; + $row{'return_failed'} = $return_failed{$issue->[$i]{'barcode'}}; + push( @$localissue, \%row ); + } + return $localissue; } -################################################################################## + +### ############################################################################### # BUILD HTML # show all reserves of this borrower, and the position of the reservation .... if ($borrowernumber) { @@ -220,26 +335,20 @@ if ($borrowernumber) { my @borrowerreserv = GetReservesFromBorrowernumber($borrowernumber ); my @reservloop; foreach my $num_res (@borrowerreserv) { - eval{ - scalar @$num_res; - }; - ($@) and next; # is this eval just a dumb way of checking to see if the arrays have any elements? - my %getreserv; - my $getiteminfo = GetBiblioFromItemNumber( $num_res->{'itemnumber'} ); my $itemtypeinfo = getitemtypeinfo( $getiteminfo->{'itemtype'} ); my ( $transfertwhen, $transfertfrom, $transfertto ) = GetTransfers( $num_res->{'itemnumber'} ); - foreach (qw(waiting transfered nottransfered)) { - $getreserv{$_} = 0; - } - - $getreserv{reservedate} = C4::Dates->new($num_res->{'reservedate'})->output() or die "Cannot get new($num_res->{'reservedate'}) from C4::Dates"; - foreach (qw(biblionumber title author barcodereserv itemcallnumber )) { - $getreserv{$_} = $getiteminfo->{$_}; - } + foreach (qw(waiting transfered nottransfered)) { + $getreserv{$_} = 0; + } + $getreserv{reservedate} = C4::Dates->new($num_res->{'reservedate'},'iso')->output('syspref'); + foreach (qw(biblionumber title author itemcallnumber )) { + $getreserv{$_} = $getiteminfo->{$_}; + } + $getreserv{barcodereserv} = $getiteminfo->{'barcode'}; $getreserv{itemtype} = $itemtypeinfo->{'description'}; # check if we have a waitin status for reservations @@ -252,7 +361,7 @@ if ($borrowernumber) { if ($transfertwhen) { $getreserv{color} = 'transfered'; $getreserv{transfered} = 1; - $getreserv{datesent} = C4::Dates->new($transfertwhen, 'iso')->output() or die "Cannot get new($transfertwhen, 'iso') from C4::Dates"; + $getreserv{datesent} = C4::Dates->new($transfertwhen, 'iso')->output('syspref') or die "Cannot get new($transfertwhen, 'iso') from C4::Dates"; $getreserv{frombranch} = GetBranchName($transfertfrom); } @@ -266,23 +375,24 @@ if ($borrowernumber) { # if we don't have a reserv on item, we put the biblio infos and the waiting position if ( $getiteminfo->{'title'} eq '' ) { - my $getbibinfo = GetBiblioItemData( $num_res->{'biblionumber'} ); + my $getbibinfo = GetBiblioData( $num_res->{'biblionumber'} ); my $getbibtype = getitemtypeinfo( $getbibinfo->{'itemtype'} ); $getreserv{color} = 'inwait'; $getreserv{title} = $getbibinfo->{'title'}; - $getreserv{waitingposition} = $num_res->{'priority'}; $getreserv{nottransfered} = 0; $getreserv{itemtype} = $getbibtype->{'description'}; $getreserv{author} = $getbibinfo->{'author'}; - $getreserv{itemcallnumber} = '----------'; $getreserv{biblionumber} = $num_res->{'biblionumber'}; } + $getreserv{waitingposition} = $num_res->{'priority'}; push( @reservloop, \%getreserv ); } # return result to the template - $template->param( reservloop => \@reservloop ); + $template->param( reservloop => \@reservloop, + countreserv => scalar @reservloop, + ); } # current alert subscriptions @@ -291,27 +401,84 @@ foreach (@$alerts) { $_->{ $_->{type} } = 1; $_->{relatedto} = findrelatedto( $_->{type}, $_->{externalid} ); } -my $picture; -my $htdocs = C4::Context->config('intrahtdocs'); -$picture = "/borrowerimages/" . $borrowernumber . ".jpg"; -if ( -e $htdocs . "$picture" ) { - $template->param( picture => $picture ); + +my $candeleteuser; +my $userenv = C4::Context->userenv; +if($userenv->{flags} % 2 == 1){ + $candeleteuser = 1; +}elsif ( C4::Context->preference("IndependantBranches") ) { + $candeleteuser = ( $data->{'branchcode'} eq $userenv->{branch} ); +}else{ + if( C4::Auth::getuserflags( $userenv->{flags},$userenv->{number})->{borrowers} ) { + $candeleteuser = 1; + }else{ + $candeleteuser = 0; + } } + +# check to see if patron's image exists in the database +# basically this gives us a template var to condition the display of +# patronimage related interface on +my ($picture, $dberror) = GetPatronImage($data->{'cardnumber'}); +$template->param( picture => 1 ) if $picture; + my $branch=C4::Context->userenv->{'branch'}; -$template->param($data); +$template->param(%$data); + +if (C4::Context->preference('ExtendedPatronAttributes')) { + $template->param(ExtendedPatronAttributes => 1); + $template->param(patron_attributes => C4::Members::Attributes::GetBorrowerAttributes($borrowernumber)); + my @types = C4::Members::AttributeTypes::GetAttributeTypes(); + if (scalar(@types) == 0) { + $template->param(no_patron_attribute_types => 1); + } +} + +if (C4::Context->preference('EnhancedMessagingPreferences')) { + C4::Form::MessagingPreferences::set_form_values({ borrowernumber => $borrowernumber }, $template); + $template->param(messaging_form_inactive => 1); + $template->param(SMSSendDriver => C4::Context->preference("SMSSendDriver")); + $template->param(SMSnumber => defined $data->{'smsalertnumber'} ? $data->{'smsalertnumber'} : $data->{'mobile'}); +} + +$template->param( + detailview => 1, + AllowRenewalLimitOverride => C4::Context->preference("AllowRenewalLimitOverride"), + DHTMLcalendar_dateformat => C4::Dates->DHTMLcalendar(), + CANDELETEUSER => $candeleteuser, + roaddetails => $roaddetails, + borrowernumber => $borrowernumber, + categoryname => $data->{'description'}, + reregistration => $reregistration, + branch => $branch, + todaysdate => C4::Dates->today(), + totalprice => sprintf("%.2f", $totalprice), + totaldue => sprintf("%.2f", $total), + totaldue_raw => $total, + issueloop => @issuedata, + relissueloop => @relissuedata, + issuecount => $issuecount, + relissuecount => $relissuecount, + overdues_exist => $overdues_exist, + error => $error, + $error => 1, + StaffMember => ($category_type eq 'S'), + is_child => ($category_type eq 'C'), +# reserveloop => \@reservedata, + dateformat => C4::Context->preference("dateformat"), + "dateformat_" . (C4::Context->preference("dateformat") || '') => 1, + samebranch => $samebranch, + quickslip => $quickslip, +); + +#Get the slip news items +my $all_koha_news = &GetNewsToDisplay("slip"); +my $koha_news_count = scalar @$all_koha_news; $template->param( - roaddetails => $roaddetails, - borrowernumber => $borrowernumber, - reregistration => $reregistration, - branch => $branch, - totalprice => sprintf( "%.2f", $totalprice ), - totaldue => sprintf( "%.2f", $total ), - issueloop => \@issuedata, - unvalidlibrarian => $unvalidlibrarian, - - # reserveloop => \@reservedata, + koha_news => $all_koha_news, + koha_news_count => $koha_news_count ); output_html_with_http_headers $input, $cookie, $template->output;