Updating INSTALL.opensuse file with the changes by Freek de Kruijf
[koha.git] / admin / aqbudgetperiods.pl
index 08d3a36..724c7e5 100755 (executable)
@@ -14,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 admin/aqbudgetperiods.pl
 
@@ -42,6 +42,7 @@ script to administer the budget periods table
 
 ## modules
 use strict;
+#use warnings; FIXME - Bug 2505
 use Number::Format qw(format_price);
 use CGI;
 use List::Util qw/min/;
@@ -60,14 +61,13 @@ my $input       = new CGI;
 
 my $searchfield          = $input->param('searchfield');
 my $budget_period_id     = $input->param('budget_period_id');
-my $budget_period_active = $input->param('budget_period_active');
-my $budget_period_locked = $input->param('budget_period_locked');
-my $op                   = $input->param('op');
+my $op                   = $input->param('op')||"else";
 
+my $budget_period_hashref= $input->Vars;
 #my $sort1_authcat = $input->param('sort1_authcat');
 #my $sort2_authcat = $input->param('sort2_authcat');
 
-my $pagesize    = 10;
+my $pagesize    = 20;
 $searchfield =~ s/\,//g;
 
 my ($template, $borrowernumber, $cookie, $staff_flags ) = get_template_and_user(
@@ -80,14 +80,11 @@ my ($template, $borrowernumber, $cookie, $staff_flags ) = get_template_and_user(
        }
 );
 
-my $script_name = "/cgi-bin/koha/admin/aqbudgetperiods.pl";  # ???
-
-my ( $count, $results ) = GetBudgetPeriods();
-### $count
-$template->param( period_button_only => 1 ) if ($count == 0) ;
 
 my $cur = GetCurrency();
-$template->param( cur => $cur->{symbol} );
+$template->param( symbol => $cur->{symbol},
+                  currency => $cur->{currency}
+               );
 my $cur_format = C4::Context->preference("CurrencyFormat");
 my $num;
 
@@ -108,96 +105,45 @@ if ( $cur_format eq 'US' ) {
     );
 }
 
-if   ($op) { $template->param( $op    => 1 ); }
-else       { $template->param( 'else' => 1 ); }
 
 # ADD OR MODIFY A BUDGET PERIOD - BUILD SCREEN
 if ( $op eq 'add_form' ) {
     ## add or modify a budget period (preparation)
     ## get information about the budget period that must be modified
 
-#    my ( $default, $sort1_authcat_dropbox, $sort1_default, $sort2_default );
-#    my ( $default, t );
 
     if ($budget_period_id) {    # MOD
-        my $data;
-        my $dbh = C4::Context->dbh;
-        my $sth = $dbh->prepare(qq|
-                                        SELECT * FROM aqbudgetperiods
-                                        WHERE budget_period_id=?    | );
-        $sth->execute($budget_period_id);
-        $data = $sth->fetchrow_hashref;
-        $sth->finish;
-
+               my $budgetperiod_hash=GetBudgetPeriod($budget_period_id);
         # get dropboxes
+               FormatData($budgetperiod_hash);
+
+        my $editnum = new Number::Format(
+            'int_curr_symbol'   => '',
+            'thousands_sep'     => '',
+            'mon_thousands_sep' => '',
+            'mon_decimal_point' => '.'
+        );
+
+        $$budgetperiod_hash{budget_period_total}= $editnum->format_price($$budgetperiod_hash{'budget_period_total'});
         $template->param(
-            budget_period_id          => $budget_period_id,
-            budget_period_startdate   => format_date( $data->{'budget_period_startdate'} ),
-            budget_period_enddate     => format_date( $data->{'budget_period_enddate'} ),
-            budget_period_description => $data->{'budget_period_description'},
-            budget_period_total       => sprintf ("%.2f",  $data->{'budget_period_total'} ),
-            budget_period_active      => $data->{'budget_period_active'},
-            budget_period_locked      => $data->{'budget_period_locked'},
+                       %$budgetperiod_hash
         );
     } # IF-MOD
-    $template->param(              DHTMLcalendar_dateformat => C4::Dates->DHTMLcalendar(),     );
+    $template->param( DHTMLcalendar_dateformat         => C4::Dates->DHTMLcalendar(),);
 }
 
 elsif ( $op eq 'add_validate' ) {
 ## add or modify a budget period (confirmation)
 
-    ## update budget period data
-    if ( $budget_period_id ne '' ) {
-        my $query = '
-                UPDATE aqbudgetperiods
-                SET    budget_period_startdate  = ?
-                    , budget_period_enddate     = ?
-                    , budget_period_description = ?
-                    , budget_period_total       = ?
-                    , budget_period_locked      = ?
-                WHERE budget_period_id          = ?
-            ';
-
-        my $sth = $dbh->prepare($query);
-        $sth->execute(
-            $input->param('budget_period_startdate')   ? format_date_in_iso( $input->param('budget_period_startdate') ) : undef,
-            $input->param('budget_period_enddate')     ? format_date_in_iso( $input->param('budget_period_enddate') )   : undef,
-            $input->param('budget_period_description') ? $input->param('budget_period_description')                     : undef,
-            $input->param('budget_period_total')       ? $input->param('budget_period_total')                           : undef,
-            $input->param('budget_period_locked')      ? $input->param('budget_period_locked')                          : undef,
-            $input->param('budget_period_id'),
-        );
-
-        # IF PASSED ACTIVE - THEN SET IT IN DB TOO.
-        set_active($budget_period_id) if ( $budget_period_active == 1 );
-
-    } else {    # ELSE ITS AN ADD
-        my $query = "
-                INSERT INTO aqbudgetperiods (
-                    budget_period_id
-                    , budget_period_startdate
-                    , budget_period_enddate
-                    , budget_period_total
-                    , budget_period_description
-                    , budget_period_locked )
-                VALUES  (?,?,?,?,?,? );
-            ";
-        my $sth = $dbh->prepare($query);
-        $sth->execute(
-            $budget_period_id,
-            $input->param('budget_period_startdate')   ? format_date_in_iso( $input->param('budget_period_startdate') ) : undef,
-            $input->param('budget_period_enddate')     ? format_date_in_iso( $input->param('budget_period_enddate') )   : undef,
-            $input->param('budget_period_total')       ? $input->param('budget_period_total')                           : undef,
-            $input->param('budget_period_description') ? $input->param('budget_period_description')                     : undef,
-            $input->param('budget_period_locked')      ? $input->param('budget_period_locked')                          : undef,
-        );
-        $budget_period_id = $dbh->last_insert_id( undef, undef, 'aqbudgetperiods', undef );
-        set_active($budget_period_id) if ( $budget_period_active == 1 );
-    }
-
-    print "Content-Type: text/html\n\n<META HTTP-EQUIV=Refresh CONTENT=\"0; URL=aqbudgetperiods.pl\"></html>";    #YUCK
-    #    output_html_with_http_headers $input, $cookie, $template->output;   # FIXME: THIS WOULD BE NICER THAN THE PREVIOUS PRINT
-    exit;
+       ## update budget period data
+       if ( $budget_period_id ne '' ) {
+               $$budget_period_hashref{$_}||=0 for qw(budget_period_active budget_period_locked);
+               my $status=ModBudgetPeriod($budget_period_hashref);
+       } 
+       else {    # ELSE ITS AN ADD
+               my $budget_period_id=AddBudgetPeriod($budget_period_hashref);
+       }
+       $op='else';
 }
 
 #--------------------------------------------------
@@ -208,39 +154,28 @@ elsif ( $op eq 'delete_confirm' ) {
     my $total = 0;
     my $data = GetBudgetPeriod( $budget_period_id);
 
+       FormatData($data);
+       $$data{'budget_period_total'}=$num->format_price(  $data->{'budget_period_total'});
     $template->param(
-            budget_period_id            => $budget_period_id,
-            budget_period_startdate     => format_date($data->{'budget_period_startdate'}),
-            budget_period_enddate       => format_date($data->{'budget_period_enddate'}),
-            budget_period_total         => $num->format_price(  $data->{'budget_period_total'}   )
-
-#            budget_period_active            => $data->{'budget_period_active'},
-#            budget_period_description    => $data->{'budget_period_description'},
-#            template                    => C4::Context->preference('template'),  ##  ??!?
+               %$data
     );
 }
 
 elsif ( $op eq 'delete_confirmed' ) {
 ## delete the budget period record
 
-    my $dbh              = C4::Context->dbh;
-    my $budget_period_id = uc( $input->param('budget_period_id') );
-    my $sth              = $dbh->prepare("DELETE FROM aqbudgetperiods WHERE budget_period_id=?");
-    $sth->execute($budget_period_id);
-    $sth->finish;
-    print "Content-Type: text/html\n\n<META HTTP-EQUIV=Refresh CONTENT=\"0; URL=aqbudgetperiods.pl\"></html>";
-    exit;
+    my $data = GetBudgetPeriod( $budget_period_id);
+    DelBudgetPeriod($budget_period_id);
+       $op='else';
 }
 
-else {
-
 # DEFAULT - DISPLAY AQPERIODS TABLE
 # -------------------------------------------------------------------
 # display the list of budget periods
-    my ( $count, $results ) = GetBudgetPeriods();
+    my $results = GetBudgetPeriods();
+       $template->param( period_button_only => 1 ) unless (@$results) ;
     my $page = $input->param('page') || 1;
     my $first = ( $page - 1 ) * $pagesize;
-
     # if we are on the last page, the number of the last word to display
     # must not exceed the length of the results array
     my $last = min( $first + $pagesize - 1, scalar @{$results} - 1, );
@@ -248,10 +183,8 @@ else {
     my @period_loop;
     foreach my $result ( @{$results}[ $first .. $last ] ) {
         my $budgetperiod = $result;
-        $budgetperiod->{'budget_period_startdate'} = format_date( $budgetperiod->{'budget_period_startdate'} );
-        $budgetperiod->{'budget_period_enddate'}   = format_date( $budgetperiod->{'budget_period_enddate'} );
+               FormatData($budgetperiod);
         $budgetperiod->{'budget_period_total'}     = $num->format_price( $budgetperiod->{'budget_period_total'} );
-        $budgetperiod->{toggle} = ( $toggle++ % 2 eq 0 ? 1 : 0 );
         $budgetperiod->{budget_active} = 1;
         push( @period_loop, $budgetperiod );
     }
@@ -260,22 +193,8 @@ else {
     $template->param(
         budget_period_dropbox => $budget_period_dropbox,
         period_loop           => \@period_loop,
-#        pagination_bar        => pagination_bar( $script_name, 
-#                                                getnbpages( scalar @{$results}, 
-#                                                $pagesize ), $page, 'page' )
+               pagination_bar            => pagination_bar("aqbudgetperiods.pl",getnbpages(scalar(@$results),$pagesize),$page),
     );
-}
 
+$template->param($op=>1);
 output_html_with_http_headers $input, $cookie, $template->output;
-
-sub set_active {
-                               my $sth = $dbh->do(
-                                       "UPDATE aqbudgetperiods
-                     SET budget_period_active = 0 "
-                               );
-                               my $sth = $dbh->do(
-                                       "UPDATE aqbudgetperiods
-                     SET budget_period_active = 1
-                     WHERE budget_period_id  =  $budget_period_id"
-                               );
-}