fix for bug 2359: Returning an item from moremember.pl...
[koha_gimpoz] / admin / aqbookfund.pl
index ea5403b..d5673c8 100755 (executable)
@@ -1,24 +1,6 @@
 #!/usr/bin/perl
 
-#script to administer the aqbudget table
 #written 20/02/2002 by paul.poulain@free.fr
-# This software is placed under the gnu General Public License, v2 (http://www.gnu.org/licenses/gpl.html)
-
-# ALGO :
-# this script use an $op to know what to do.
-# if $op is empty or none of the above values,
-#      - the default screen is build (with all records, or filtered datas).
-#      - the   user can clic on add, modify or delete record.
-# if $op=add_form
-#      - if primkey exists, this is a modification,so we read the $primkey record
-#      - builds the add/modify form
-# if $op=add_validate
-#      - the user has just send datas, so we create/modify the record
-# if $op=delete_form
-#      - we show the record having primkey=$primkey and ask for deletion validation form
-# if $op=delete_confirm
-#      - we delete the record having primkey=$primkey
-
 
 # Copyright 2000-2002 Katipo Communications
 #
 # Koha; if not, write to the Free Software Foundation, Inc., 59 Temple Place,
 # Suite 330, Boston, MA  02111-1307 USA
 
+
+=head1 NAME
+
+aqbookfund.pl
+
+=head1 DESCRIPTION
+
+script to administer the aqbudget table.
+
+=head1 CGI PARAMETERS
+
+=over 4
+
+=item op
+this script use an C<$op> to know what to do.
+C<op> can be equal to:
+* empty or none of the above values, then
+    - the default screen is build (with all records, or filtered datas).
+       - the   user can clic on add, modify or delete record.
+* add_form, then
+       - if primkey exists, this is a modification,so we read the $primkey record
+       - builds the add/modify form
+* add_validate, then
+       - the user has just send datas, so we create/modify the record
+* delete_form, then
+       - we show the record having primkey=$primkey and ask for deletion validation form
+* delete_confirm, then
+    - we delete the record having primkey=$primkey
+
+=cut
+
 use strict;
 use CGI;
-use HTML::Template;
 use List::Util qw/min/;
-
+use C4::Branch; # GetBranches
 use C4::Auth;
 use C4::Koha;
 use C4::Context;
-use C4::Acquisition;
+use C4::Bookfund;
 use C4::Output;
-use C4::Interface::CGI::Output;
-use C4::Search;
-use C4::Date;
+use C4::Dates;
+use C4::Debug;
+
+# use Smart::Comments;
 
 my $dbh = C4::Context->dbh;
 my $input = new CGI;
 my $script_name="/cgi-bin/koha/admin/aqbookfund.pl";
 my $bookfundid=$input->param('bookfundid');
+my $branchcodeid=$input->param('branchcode')|'';
 my $pagesize = 10;
 my $op = $input->param('op') || '';
 
@@ -64,9 +78,9 @@ my ($template, $borrowernumber, $cookie)
          query => $input,
          type => "intranet",
          authnotrequired => 0,
-         flagsrequired => {parameters => 1, management => 1},
+         flagsrequired => {parameters => 1},
          debug => 1,
-     }
+        }
     );
 
 if ($op) {
@@ -81,35 +95,21 @@ else {
 }
 $template->param(action => $script_name);
 
-# my @branches;
-# my @select_branch;
-# my %select_branches;
-
 my $branches = GetBranches;
 
-################## ADD_FORM ##################################
+#-----############# ADD_FORM ##################################
 # called by default. Used to create form to add or  modify a record
 if ($op eq 'add_form') {
        #---- if primkey exists, it's a modify action, so read values to modify...
        my $dataaqbookfund;
        my $header;
        if ($bookfundid) {
-                my $query = '
-SELECT bookfundid,
-       bookfundname,
-       bookfundgroup,
-       branchcode
-  FROM aqbookfund
-  WHERE bookfundid = ?
-';
-               my $sth=$dbh->prepare($query);
-               $sth->execute($bookfundid);
-               $dataaqbookfund = $sth->fetchrow_hashref;
-               $sth->finish;
-           }
+       $dataaqbookfund = GetBookFund($bookfundid,$branchcodeid);
+       }
        if ($bookfundid) {
            $header = "Modify book fund";
            $template->param('header-is-modify-p' => 1);
+           $template->param('current_branch' =>  $branchcodeid);
        } else {
            $header = "Add book fund";
            $template->param('header-is-add-p' => 1);
@@ -144,93 +144,62 @@ SELECT bookfundid,
 
 } # END $OP eq ADD_FORM
 
-################## ADD_VALIDATE ##################################
+#-----############# ADD_VALIDATE ##################################
 # called by add_form, used to insert/modify data in DB
 elsif ($op eq 'add_validate') {
+### add
        my $bookfundid = uc $input->param('bookfundid');
+        my $bookfundname = $input->param('bookfundname');
+        my $branchcode = $input->param('branchcode') || undef;
 
-        my ($query, $sth);
-
-        $query = '
-SELECT COUNT(*) AS counter
-  FROM aqbookfund
-  WHERE bookfundid = ?
-';
-        $sth=$dbh->prepare($query);
-       $sth->execute($bookfundid);
-        my $data = $sth->fetchrow_hashref;
-       $sth->finish;
-        my $bookfund_already_exists = $data->{counter} > 0 ? 1 : 0;
-
-        if ($bookfund_already_exists) {
-            $query = '
-UPDATE aqbookfund
-  SET bookfundname = ?,
-      branchcode = ?
-  WHERE bookfundid = ?
-';
-            $sth=$dbh->prepare($query);
-            $sth->execute(
-                $input->param('bookfundname'),
-                $input->param('branchcode') || undef,
-                $bookfundid,
-            );
-            $sth->finish;
-
-            # budgets depending on a bookfund must have the same branchcode
-            # if the bookfund branchcode is set
-            if (defined $input->param('branchcode')) {
-                $query = '
-UPDATE aqbudget
-  SET branchcode = ?
-';
-                $sth=$dbh->prepare($query);
-                $sth->execute($input->param('branchcode'));
-                $sth->finish;
-            }
-        }
-        else {
-            $query = '
-INSERT
-  INTO aqbookfund
-  (bookfundid, bookfundname, branchcode)
-  VALUES
-  (?, ?, ?)
-';
-            $sth=$dbh->prepare($query);
-            $sth->execute(
-                $bookfundid,
-                $input->param('bookfundname'),
-                $input->param('branchcode') || undef,
-            );
-            $sth->finish;
-        }
+    my $number = Countbookfund($bookfundid,$branchcodeid);
+    if ($number == 0 ) {
+
+        NewBookFund(
+            $bookfundid,
+            $input->param('bookfundname'),
+            $input->param('branchcode')||''
+        );
+    }
+    $input->redirect('aqbookfund.pl');
+# END $OP eq ADD_VALIDATE
+}
 
-        $input->redirect('aqbookfund.pl');
-                                                                               # END $OP eq ADD_VALIDATE
-################## DELETE_CONFIRM ##################################
+#-----############# MOD_VALIDATE ##################################
+# called by add_form, used to insert/modify data in DB
+elsif ($op eq 'mod_validate') {
+       my $bookfundid  = uc $input->param('bookfundid');
+       my $bookfundname   = $input->param('bookfundname');
+       my $branchcode     = $input->param('branchcode'    ) || undef;
+       my $current_branch = $input->param('current_branch') || undef;
+       $debug and warn "$bookfundid, $bookfundname, $branchcode";
+
+       my $number = Countbookfund($bookfundid,$branchcodeid);
+    if ($number < 2)  {
+         $debug and warn "name :$bookfundname branch:$branchcode";
+        ModBookFund($bookfundname,$bookfundid,$current_branch, $branchcode);
+    }
+   $input->redirect('aqbookfund.pl');
+}
+
+#-----############# DELETE_CONFIRM ##################################
 # called by default form, used to confirm deletion of data in DB
-} elsif ($op eq 'delete_confirm') {
-       my $sth=$dbh->prepare("select bookfundid,bookfundname,bookfundgroup from aqbookfund where bookfundid=?");
-       $sth->execute($bookfundid);
-       my $data=$sth->fetchrow_hashref;
-       $sth->finish;
+elsif ($op eq 'delete_confirm') {
+    my $data = GetBookFund($bookfundid,$branchcodeid);
        $template->param(bookfundid => $bookfundid);
        $template->param(bookfundname => $data->{'bookfundname'});
-                                                                                                       # END $OP eq DELETE_CONFIRM
-################## DELETE_CONFIRMED ##################################
+       $template->param(branchcode => $data->{'branchcode'});
+} # END $OP eq DELETE_CONFIRM
+
+#-----############# DELETE_CONFIRMED ##################################
 # called by delete_confirm, used to effectively confirm deletion of data in DB
-} elsif ($op eq 'delete_confirmed') {
-       my $bookfundid=uc($input->param('bookfundid'));
-       my $sth=$dbh->prepare("delete from aqbookfund where bookfundid=?");
-       $sth->execute($bookfundid);
-       $sth->finish;
-       $sth=$dbh->prepare("delete from aqbudget where bookfundid=?");
-       $sth->execute($bookfundid);
-       $sth->finish;
-                                                                                                       # END $OP eq DELETE_CONFIRMED
-################## DEFAULT ##################################
-} else { # DEFAULT
+elsif ($op eq 'delete_confirmed') {
+    DelBookFund(uc($input->param('bookfundid')),$branchcodeid);
+
+}# END $OP eq DELETE_CONFIRMED
+
+#-----############# DEFAULT ##################################
+else { # DEFAULT
     my ($query, $sth);
 
     $template->param(scriptname => $script_name);
@@ -242,31 +211,22 @@ INSERT
             code => $branchcode,
             name => $branches->{$branchcode}->{branchname},
         };
-
         if (defined $input->param('filter_branchcode')
             and $input->param('filter_branchcode') eq $branchcode) {
             $row->{selected} = 1;
         }
-
         push @branchloop, $row;
     }
 
     my @bookfundids_loop;
-    $query = '
-SELECT bookfundid
-  FROM aqbookfund
-';
-    $sth = $dbh->prepare($query);
-    $sth->execute();
+    $sth = GetBookFundsId();
+
     while (my $row = $sth->fetchrow_hashref) {
-        if (defined $input->param('filter_bookfundid')
-            and $input->param('filter_bookfundid') eq $row->{bookfundid}) {
+        if (defined $input->param('filter_bookfundid') and $input->param('filter_bookfundid') eq $row->{bookfundid}){
             $row->{selected} = 1;
         }
-
-        push @bookfundids_loop, $row;
-    }
-    $sth->finish;
+         push @bookfundids_loop, $row;
+     }
 
     $template->param(
         filter_bookfundids => \@bookfundids_loop,
@@ -275,58 +235,24 @@ SELECT bookfundid
     );
 
     # searching the bookfunds corresponding to our filtering rules
-    my @bindings;
-
-    $query = '
-SELECT bookfundid,
-       bookfundname,
-       bookfundgroup,
-       branchcode
-  FROM aqbookfund
-  WHERE 1 = 1';
-    if ($input->param('filter')) {
-        if ($input->param('filter_bookfundid')) {
-            $query.= '
-    AND bookfundid = ?
-';
-            push @bindings, $input->param('filter_bookfundid');
-        }
-        if ($input->param('filter_bookfundname')) {
-            $query.= '
-    AND bookfundname like ?
-';
-            push @bindings, '%'.$input->param('filter_bookfundname').'%';
-        }
-        if ($input->param('filter_branchcode')) {
-            $query.= '
-    AND branchcode = ?
-';
-            push @bindings, $input->param('filter_branchcode');
-        }
-    }
-    $query.= '
-  ORDER BY bookfundid
-';
+    my @results = SearchBookFund(
+        $input->param('filter'),
+        $input->param('filter_bookfundid'),
+        $input->param('filter_bookfundname'),
+        $input->param('filter_branchcode'),
+    );
 
-    $sth = $dbh->prepare($query);
-    $sth->execute(@bindings);
-    my @results;
-    while (my $row = $sth->fetchrow_hashref) {
-        push @results, $row;
+    # does the book funds have budgets?
+    my @loop_id;
+    $sth = GetBookFundsId();
+    while (my $row = $sth->fetchrow){
+        push @loop_id, $row;
     }
 
-    # does the book funds have budgets?
-    $query = '
-SELECT bookfundid,
-       COUNT(*) AS counter
-  FROM aqbudget
-  GROUP BY bookfundid
-';
-    $sth = $dbh->prepare($query);
-    $sth->execute();
-    my %nb_budgets_of;
-    while (my $row = $sth->fetchrow_hashref) {
-        $nb_budgets_of{ $row->{bookfundid} } = $row->{counter};
+    my ($id,%nb_budgets_of);
+    foreach $id (@loop_id){
+        my $number = Countbookfund($id);
+        $nb_budgets_of{$id} = $number;
     }
 
     # pagination informations
@@ -357,17 +283,13 @@ SELECT bookfundid,
     }
 
     $template->param(
-        bookfund => \@loop,
-        pagination_bar => pagination_bar(
-            $script_name,
-            getnbpages(scalar @results, $pagesize),
-            $page,
-            'page'
-        )
-    );
+            bookfund => \@loop,
+            pagination_bar => pagination_bar(
+                        $script_name,
+                        getnbpages(scalar @results, $pagesize),
+                        $page,
+                        'page'
+            )
+        );
 } #---- END $OP eq DEFAULT
-$template->param(intranetcolorstylesheet => C4::Context->preference("intranetcolorstylesheet"),
-               intranetstylesheet => C4::Context->preference("intranetstylesheet"),
-               IntranetNav => C4::Context->preference("IntranetNav"),
-               );
 output_html_with_http_headers $input, $cookie, $template->output;