Bug 7684: QA Followup and bugfixes

[koha.git] / members / mancredit.pl

diff --git a/members/mancredit.pl b/members/mancredit.pl
index 4f2fe50..a2c2258 100755 (executable)
--- a/members/mancredit.pl
+++ b/members/mancredit.pl
@@ -1,10 +1,11 @@
 #!/usr/bin/perl
 
-#wrriten 11/1/2000 by chris@katipo.oc.nz
+#written 11/1/2000 by chris@katipo.oc.nz
 #script to display borrowers account details
 
 
 # Copyright 2000-2002 Katipo Communications
+# Copyright 2010 BibLibre
 #
 # This file is part of Koha.
 #
@@ -17,50 +18,97 @@
 # 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 C4::Auth;
 use C4::Output;
 use CGI;
 
 use C4::Members;
+use C4::Branch;
 use C4::Accounts;
+use C4::Items;
+use C4::Members::Attributes qw(GetBorrowerAttributes);
+
 my $input=new CGI;
+my $flagsrequired = { borrowers => 1, updatecharges => 1 };
 
 my $borrowernumber=$input->param('borrowernumber');
 
 #get borrower details
-my $data=GetMember($borrowernumber,'borrowernumber');
+my $data=GetMember('borrowernumber' => $borrowernumber);
 my $add=$input->param('add');
 
 if ($add){
-    my $itemnum=$input->param('itemnum');
-    my $desc=$input->param('desc');
-    my $amount=$input->param('amount');
-    $amount = -$amount;
-    my $type=$input->param('type');
-    manualinvoice($borrowernumber,$itemnum,$desc,$type,$amount);
-    print $input->redirect("/cgi-bin/koha/members/boraccount.pl?borrowernumber=$borrowernumber");
+    if ( checkauth( $input, 0, $flagsrequired, 'intranet' ) ) {
+        my $barcode = $input->param('barcode');
+        my $itemnum;
+        if ($barcode) {
+            $itemnum = GetItemnumberFromBarcode($barcode);
+        }
+        my $desc    = $input->param('desc');
+        my $note    = $input->param('note');
+        my $amount  = $input->param('amount') || 0;
+        $amount = -$amount;
+        my $type = $input->param('type');
+        manualinvoice( $borrowernumber, $itemnum, $desc, $type, $amount, $note );
+        print $input->redirect("/cgi-bin/koha/members/boraccount.pl?borrowernumber=$borrowernumber");
+    }
 } else {
        my ($template, $loggedinuser, $cookie)
-       = get_template_and_user({template_name => "members/mancredit.tmpl",
-                                       query => $input,
-                                       type => "intranet",
-                                       authnotrequired => 0,
-                                       flagsrequired => {borrowers => 1},
-                                       debug => 1,
-                                       });
+         = get_template_and_user({template_name => "members/mancredit.tmpl",
+                                         query => $input,
+                                         type => "intranet",
+                                         authnotrequired => 0,
+                      flagsrequired => $flagsrequired,
+                                         debug => 1,
+                                         });
+                                         
+    if ( $data->{'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;
+    }
+
+    $template->param( adultborrower => 1 ) if ( $data->{category_type} eq 'A' );
+    my ($picture, $dberror) = GetPatronImage($data->{'borrowernumber'});
+    $template->param( picture => 1 ) if $picture;
+
+if (C4::Context->preference('ExtendedPatronAttributes')) {
+    my $attributes = GetBorrowerAttributes($borrowernumber);
     $template->param(
-                    borrowernumber => $borrowernumber,
-                    firstname => $data->{'firstname'},
-                    surname  => $data->{'surname'},
+        ExtendedPatronAttributes => 1,
+        extendedattributes => $attributes
     );
-       print $input->header(
-           -type => 'utf-8',
-           -cookie => $cookie
-       ),$template->output;
-
+}
+    
+    $template->param(
+        borrowernumber => $borrowernumber,
+        firstname => $data->{'firstname'},
+        surname  => $data->{'surname'},
+                   cardnumber => $data->{'cardnumber'},
+                   categorycode => $data->{'categorycode'},
+                   category_type => $data->{'category_type'},
+                   categoryname  => $data->{'description'},
+                   address => $data->{'address'},
+                   address2 => $data->{'address2'},
+                   city => $data->{'city'},
+                   state => $data->{'state'},
+                   zipcode => $data->{'zipcode'},
+                   country => $data->{'country'},
+                   phone => $data->{'phone'},
+                   email => $data->{'email'},
+                   branchcode => $data->{'branchcode'},
+                   branchname => GetBranchName($data->{'branchcode'}),
+                   is_child        => ($data->{'category_type'} eq 'C'),
+                       activeBorrowerRelationship => (C4::Context->preference('borrowerRelationship') ne ''),
+            RoutingSerials => C4::Context->preference('RoutingSerials'),
+        );
+    output_html_with_http_headers $input, $cookie, $template->output;
 }