X-Git-Url: http://git.rot13.org/?a=blobdiff_plain;f=members%2Fmoremember.pl;h=6198d87c466b04bf912816ed1203ad6b55c248fa;hb=ff1579de6d2e95158cf4f9e9261068bdeef31f53;hp=4ccba007165c853e1f843258aa81a8b4db4d6c8d;hpb=840a907b86c054aa1c216a8cf353b1a9ef3c44fa;p=koha.git

diff --git a/members/moremember.pl b/members/moremember.pl
index 4ccba00716..6198d87c46 100755
--- a/members/moremember.pl
+++ b/members/moremember.pl
@@ -2,21 +2,22 @@
 
 # Copyright 2000-2002 Katipo Communications
 # Copyright 2010 BibLibre
+# Copyright 2014 ByWater Solutions
 #
 # This file is part of Koha.
 #
-# Koha is free software; you can redistribute it and/or modify it under the
-# terms of the GNU General Public License as published by the Free Software
-# Foundation; either version 2 of the License, or (at your option) any later
-# version.
+# Koha is free software; you can redistribute it and/or modify it
+# under the terms of the GNU General Public License as published by
+# the Free Software Foundation; either version 3 of the License, or
+# (at your option) any later version.
 #
-# Koha is distributed in the hope that it will be useful, but WITHOUT ANY
-# WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS FOR
-# A PARTICULAR PURPOSE.  See the GNU General Public License for more details.
+# Koha is distributed in the hope that it will be useful, but
+# WITHOUT ANY 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.,
-# 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
+# You should have received a copy of the GNU General Public License
+# along with Koha; if not, see <http://www.gnu.org/licenses>.
 
 
 =head1 moremember.pl
@@ -34,30 +35,32 @@
 
 use strict;
 #use warnings; FIXME - Bug 2505
-use CGI;
+use CGI qw ( -utf8 );
 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;
 use C4::Koha;
 use C4::Letters;
 use C4::Biblio;
-use C4::Reserves;
 use C4::Branch; # GetBranchName
-use C4::Overdues qw/CheckBorrowerDebarred/;
 use C4::Form::MessagingPreferences;
 use List::MoreUtils qw/uniq/;
 use C4::Members::Attributes qw(GetBorrowerAttributes);
-
+use Koha::Borrower::Debarments qw(GetDebarments IsDebarred);
+use Module::Load;
+if ( C4::Context->preference('NorwegianPatronDBEnable') && C4::Context->preference('NorwegianPatronDBEnable') == 1 ) {
+    load Koha::NorwegianPatronDB, qw( NLGetSyncDataFromBorrowernumber );
+}
 #use Smart::Comments;
 #use Data::Dumper;
 use DateTime;
 use Koha::DateUtils;
+use Koha::Database;
 
 use vars qw($debug);
 
@@ -70,38 +73,30 @@ my $dbh = C4::Context->dbh;
 my $input = CGI->new;
 $debug or $debug = $input->param('debug') || 0;
 my $print = $input->param('print');
-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; }
 
 my $template_name;
 my $quickslip = 0;
 
 my $flagsrequired;
-if ($print eq "page") {
-    $template_name = "members/moremember-print.tmpl";
+if (defined $print and $print eq "page") {
+    $template_name = "members/moremember-print.tt";
     # circ staff who process checkouts but can't edit
     # patrons still need to be able to access print view
     $flagsrequired = { circulate => "circulate_remaining_permissions" };
-} elsif ($print eq "slip") {
-    $template_name = "members/moremember-receipt.tmpl";
+} elsif (defined $print and $print eq "slip") {
+    $template_name = "members/moremember-receipt.tt";
     # 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";
+} elsif (defined $print and $print eq "qslip") {
+    $template_name = "members/moremember-receipt.tt";
     $quickslip = 1;
     $flagsrequired =  { circulate => "circulate_remaining_permissions" };
-} elsif ($print eq "brief") {
-    $template_name = "members/moremember-brief.tmpl";
+} elsif (defined $print and $print eq "brief") {
+    $template_name = "members/moremember-brief.tt";
     $flagsrequired = { borrowers => 1 };
 } else {
-    $template_name = "members/moremember.tmpl";
+    $template_name = "members/moremember.tt";
     $flagsrequired = { borrowers => 1 };
 }
 
@@ -116,10 +111,13 @@ my ( $template, $loggedinuser, $cookie ) = get_template_and_user(
     }
 );
 my $borrowernumber = $input->param('borrowernumber');
+my $error = $input->param('error');
+$template->param( error => $error ) if ( $error );
 
-#start the page and read in includes
-my $data           = GetMember( 'borrowernumber' => $borrowernumber );
-my $reregistration = $input->param('reregistration');
+my ( $od, $issue, $fines ) = GetMemberIssuesAndFines($borrowernumber);
+$template->param( issuecount => $issue );
+
+my $data = GetMember( 'borrowernumber' => $borrowernumber );
 
 if ( not defined $data ) {
     $template->param (unknowuser => 1);
@@ -127,26 +125,17 @@ if ( not defined $data ) {
     exit;
 }
 
-# re-reregistration function to automatic calcul of date expiry
-if ( $reregistration eq 'y' ) {
-	$data->{'dateexpiry'} = ExtendMemberSubscriptionTo( $borrowernumber );
-}
-
 my $category_type = $data->{'category_type'};
 
-### $category_type
-
 $debug and printf STDERR "dates (enrolled,expiry,birthdate) raw: (%s, %s, %s)\n", map {$data->{$_}} qw(dateenrolled dateexpiry dateofbirth);
 foreach (qw(dateenrolled dateexpiry dateofbirth)) {
-		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 );
+    my $userdate = $data->{$_};
+    unless ($userdate) {
+        $debug and warn sprintf "Empty \$data{%12s}", $_;
+        $data->{$_} = '';
+        next;
+    }
+    $template->param( $_ => dt_from_string( $userdate ) );
 }
 $data->{'IS_ADULT'} = ( $data->{'categorycode'} ne 'I' );
 
@@ -154,19 +143,17 @@ for (qw(gonenoaddress lost borrowernotes)) {
 	 $data->{$_} and $template->param(flagged => 1) and last;
 }
 
-my $debar = CheckBorrowerDebarred($borrowernumber);
-if ($debar) {
+if ( IsDebarred($borrowernumber) ) {
     $template->param( 'userdebarred' => 1, 'flagged' => 1 );
+    my $debar = $data->{'debarred'};
     if ( $debar ne "9999-12-31" ) {
-        $template->param( 'userdebarreddate' => C4::Dates::format_date($debar) );
+        $template->param( 'userdebarreddate' => output_pref( { dt => dt_from_string( $debar ), dateonly => 1 } ) );
         $template->param( 'debarredcomment'  => $data->{debarredcomment} );
     }
 }
 
-$data->{'ethnicity'} = fixEthnicity( $data->{'ethnicity'} );
-$data->{ "sex_".$data->{'sex'}."_p" } = 1;
+$data->{ "sex_".$data->{'sex'}."_p" } = 1 if defined $data->{sex};
 
-my $catcode;
 if ( $category_type eq 'C') {
    my  ( $catcodes, $labels ) =  GetborCatFromCatType( 'A', 'WHERE category_type = ?' );
    my $cnt = scalar(@$catcodes);
@@ -175,18 +162,14 @@ if ( $category_type eq 'C') {
    $template->param( 'catcode' =>    $catcodes->[0])  if $cnt == 1;
 }
 
-
-if ( $data->{'ethnicity'} || $data->{'ethnotes'} ) {
-    $template->param( printethnicityline => 1 );
-}
-if ( $category_type eq 'A' || $category_type eq 'I') {
+my ( $count, $guarantees ) = GetGuarantees( $data->{'borrowernumber'} );
+if ( $count ) {
     $template->param( isguarantee => 1 );
 
     # FIXME
     # It looks like the $i is only being returned to handle walking through
     # the array, which is probably better done as a foreach loop.
     #
-    my ( $count, $guarantees ) = GetGuarantees( $data->{'borrowernumber'} );
     my @guaranteedata;
     for ( my $i = 0 ; $i < $count ; $i++ ) {
         push(@guaranteedata,
@@ -199,7 +182,6 @@ if ( $category_type eq 'A' || $category_type eq 'I') {
         );
     }
     $template->param( guaranteeloop => \@guaranteedata );
-    ( $template->param( adultborrower => 1 ) ) if ( $category_type eq 'A' || $category_type eq 'I' );
 }
 else {
     if ($data->{'guarantorid'}){
@@ -214,18 +196,23 @@ else {
 	}
 }
 
+$template->param( adultborrower => 1 ) if ( $category_type eq 'A' || $category_type eq 'I' );
+
 my %bor;
 $bor{'borrowernumber'} = $borrowernumber;
 
 # Converts the branchcode to the branch name
 my $samebranch;
-if ( C4::Context->preference("IndependantBranches") ) {
+if ( C4::Context->preference("IndependentBranches") ) {
     my $userenv = C4::Context->userenv;
-    unless ( $userenv->{flags} % 2 == 1 ) {
+    if ( C4::Context->IsSuperLibrarian() ) {
+        $samebranch = 1;
+    }
+    else {
         $samebranch = ( $data->{'branchcode'} eq $userenv->{branch} );
     }
-    $samebranch = 1 if ( $userenv->{flags} % 2 == 1 );
-}else{
+}
+else {
     $samebranch = 1;
 }
 my $branchdetail = GetBranchDetail( $data->{'branchcode'});
@@ -237,101 +224,43 @@ my $lib2 = &GetSortDetails( "Bsort2", $data->{'sort2'} );
 $template->param( lib1 => $lib1 ) if ($lib1);
 $template->param( lib2 => $lib2 ) if ($lib2);
 
+# If printing a page, send the account informations to the template
+if ($print eq "page") {
+    foreach my $accountline (@$accts) {
+        $accountline->{amount} = sprintf '%.2f', $accountline->{amount};
+        $accountline->{amountoutstanding} = sprintf '%.2f', $accountline->{amountoutstanding};
+
+        if ($accountline->{accounttype} ne 'F' && $accountline->{accounttype} ne 'FU'){
+            $accountline->{printtitle} = 1;
+        }
+    }
+    $template->param( accounts => $accts );
+}
+
 # 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 @borrowernumbers = GetMemberRelatives($borrowernumber);
-my $issue       = GetPendingIssues($borrowernumber);
-my $relissue    = [];
-if ( @borrowernumbers ) {
-    $relissue    = GetPendingIssues(@borrowernumbers);
-}
-my $roaddetails = &GetRoadTypeDetails( $data->{'streettype'} );
+my @relatives = GetMemberRelatives($borrowernumber);
+my $relatives_issues_count =
+  Koha::Database->new()->schema()->resultset('Issue')
+  ->count( { borrowernumber => \@relatives } );
+
+my $roadtype = C4::Koha::GetAuthorisedValueByCode( 'ROADTYPE', $data->{streettype} );
 my $today       = DateTime->now( time_zone => C4::Context->tz);
 $today->truncate(to => 'day');
-my @borrowers_with_issues;
 my $overdues_exist = 0;
 my $totalprice = 0;
 
-my @issuedata = build_issue_data($issue);
-my @relissuedata = build_issue_data($relissue);
-
-
 ### ###############################################################################
 # BUILD HTML
 # show all reserves of this borrower, and the position of the reservation ....
 if ($borrowernumber) {
-
-    # new op dev
-    # now we show the status of the borrower's reservations
-    my @borrowerreserv = GetReservesFromBorrowernumber($borrowernumber );
-    my @reservloop;
-    foreach my $num_res (@borrowerreserv) {
-        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'},'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
-        if ( $num_res->{'found'} eq 'W' ) {
-            $getreserv{color}   = 'reserved';
-            $getreserv{waiting} = 1;
-        }
-
-        # 		check transfers with the itemnumber foud in th reservation loop
-        if ($transfertwhen) {
-            $getreserv{color}      = 'transfered';
-            $getreserv{transfered} = 1;
-            $getreserv{datesent}   = C4::Dates->new($transfertwhen, 'iso')->output('syspref') or die "Cannot get new($transfertwhen, 'iso') from C4::Dates";
-            $getreserv{frombranch} = GetBranchName($transfertfrom);
-        }
-
-        if ( ( $getiteminfo->{'holdingbranch'} ne $num_res->{'branchcode'} )
-            and not $transfertwhen )
-        {
-            $getreserv{nottransfered}   = 1;
-            $getreserv{nottransferedby} =
-                GetBranchName( $getiteminfo->{'holdingbranch'} );
-        }
-
-# 		if we don't have a reserv on item, we put the biblio infos and the waiting position
-        if ( $getiteminfo->{'title'} eq '' ) {
-            my $getbibinfo = GetBiblioData( $num_res->{'biblionumber'} );
-            my $getbibtype = getitemtypeinfo( $getbibinfo->{'itemtype'} );
-            $getreserv{color}           = 'inwait';
-            $getreserv{title}           = $getbibinfo->{'title'};
-            $getreserv{nottransfered}   = 0;
-            $getreserv{itemtype}        = $getbibtype->{'description'};
-            $getreserv{author}          = $getbibinfo->{'author'};
-            $getreserv{biblionumber}  = $num_res->{'biblionumber'};	
-        }
-        $getreserv{waitingposition} = $num_res->{'priority'};
-        $getreserv{suspend} = $num_res->{'suspend'};
-        $getreserv{suspend_until} = $num_res->{'suspend_until'};
-
-        push( @reservloop, \%getreserv );
-    }
-
-    # return result to the template
-    $template->param( reservloop => \@reservloop,
-        countreserv => scalar @reservloop,
-	 );
+    $template->param(
+        holds_count => Koha::Database->new()->schema()->resultset('Reserve')
+          ->count( { borrowernumber => $borrowernumber } ) );
 }
 
 # current alert subscriptions
@@ -341,24 +270,20 @@ foreach (@$alerts) {
     $_->{relatedto} = findrelatedto( $_->{type}, $_->{externalid} );
 }
 
-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;
+# Add sync data to the user data
+if ( C4::Context->preference('NorwegianPatronDBEnable') && C4::Context->preference('NorwegianPatronDBEnable') == 1 ) {
+    my $sync = NLGetSyncDataFromBorrowernumber( $borrowernumber );
+    if ( $sync ) {
+        $data->{'sync'}       = $sync->sync;
+        $data->{'syncstatus'} = $sync->syncstatus;
+        $data->{'lastsync'}   = $sync->lastsync;
     }
 }
 
 # 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'});
+my ($picture, $dberror) = GetPatronImage($data->{'borrowernumber'});
 $template->param( picture => 1 ) if $picture;
 
 my $branch=C4::Context->userenv->{'branch'};
@@ -408,117 +333,29 @@ $template->param( $data->{'categorycode'} => 1 );
 $template->param(
     detailview => 1,
     AllowRenewalLimitOverride => C4::Context->preference("AllowRenewalLimitOverride"),
-    CANDELETEUSER    => $candeleteuser,
-    roaddetails     => $roaddetails,
+    roadtype        => $roadtype,
     borrowernumber  => $borrowernumber,
     othernames      => $data->{'othernames'},
     categoryname    => $data->{'description'},
-    reregistration  => $reregistration,
+    was_renewed     => $input->param('was_renewed') ? 1 : 0,
     branch          => $branch,
-    todaysdate      => C4::Dates->today(),
+    todaysdate      => output_pref({ dt => dt_from_string, dateformat => 'iso', dateonly => 1 }),
     totalprice      => sprintf("%.2f", $totalprice),
     totaldue        => sprintf("%.2f", $total),
     totaldue_raw    => $total,
-    issueloop       => @issuedata,
-    relissueloop    => @relissuedata,
     overdues_exist  => $overdues_exist,
-    error           => $error,
     StaffMember     => ($category_type eq 'S'),
     is_child        => ($category_type eq 'C'),
-#   reserveloop     => \@reservedata,
-    samebranch     => $samebranch,
-    quickslip		  => $quickslip,
+    samebranch      => $samebranch,
+    quickslip       => $quickslip,
     activeBorrowerRelationship => (C4::Context->preference('borrowerRelationship') ne ''),
     AutoResumeSuspendedHolds => C4::Context->preference('AutoResumeSuspendedHolds'),
     SuspendHoldsIntranet => C4::Context->preference('SuspendHoldsIntranet'),
+    RoutingSerials => C4::Context->preference('RoutingSerials'),
+    debarments => GetDebarments({ borrowernumber => $borrowernumber }),
+    PatronsPerPage => C4::Context->preference("PatronsPerPage") || 20,
+    relatives_issues_count => $relatives_issues_count,
+    relatives_borrowernumbers => \@relatives,
 );
-$template->param( $error => 1 ) if $error;
 
 output_html_with_http_headers $input, $cookie, $template->output;
-
-sub build_issue_data {
-    my $issues = shift;
-
-    my $localissue;
-
-    foreach my $issue ( @{$issues} ) {
-
-        # Getting borrower details
-        my $memberdetails = GetMemberDetails( $issue->{borrowernumber} );
-        $issue->{borrowername} =
-          $memberdetails->{firstname} . ' ' . $memberdetails->{surname};
-        $issue->{cardnumber} = $memberdetails->{cardnumber};
-        my $issuedate;
-        if ($issue->{issuedate} ) {
-           $issuedate = $issue->{issuedate}->clone();
-        }
-
-        $issue->{date_due}  = output_pref( $issue->{date_due} );
-        $issue->{issuedate} = output_pref( $issue->{issuedate} ) if defined $issue->{issuedate};
-        my $biblionumber = $issue->{biblionumber};
-        $issue->{issuingbranchname} = GetBranchName($issue->{branchcode});
-        my %row          = %{$issue};
-        $totalprice += $issue->{replacementprice};
-
-        # item lost, damaged loops
-        if ( $row{'itemlost'} ) {
-            my $fw       = GetFrameworkCode( $issue->{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->{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 ( $issue->{overdue} ) {
-            $overdues_exist = 1;
-            $row{red} = 1;
-        }
-        if ($issuedate) {
-            $issuedate->truncate( to => 'day' );
-            if ( DateTime->compare( $issuedate, $today ) == 0 ) {
-                $row{today} = 1;
-            }
-        }
-
-        #find the charge for an item
-        my ( $charge, $itemtype ) =
-          GetIssuingCharges( $issue->{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->{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->{itemnumber} };
-        $row{return_failed} = $return_failed{ $issue->{barcode} };
-        push( @{$localissue}, \%row );
-    }
-    return $localissue;
-}