# get only the columns of aqbudgetperiods in budget_period_hashref
my @columns = Koha::Database->new()->schema->source('Aqbudgetperiod')->columns;
-my $budget_period_hashref = { map { join(' ',@columns) =~ /$_/ ? ( $_ => $input->param($_) ) : () } keys( %{$input->Vars()} ) } ;
-$budget_period_hashref->{budget_period_startdate} = dt_from_string( $input->param('budget_period_startdate') );
-$budget_period_hashref->{budget_period_enddate} = dt_from_string( $input->param('budget_period_enddate') );
+my $budget_period_hashref = { map { join(' ',@columns) =~ /$_/ ? ( $_ => scalar $input->param($_) ) : () } keys( %{$input->Vars()} ) } ;
+$budget_period_hashref->{budget_period_startdate} = dt_from_string( scalar $input->param('budget_period_startdate') );
+$budget_period_hashref->{budget_period_enddate} = dt_from_string( scalar $input->param('budget_period_enddate') );
-my $activepagesize = 20;
-my $inactivepagesize = 20;
$searchfield =~ s/\,//g;
my ($template, $borrowernumber, $cookie, $staff_flags ) = get_template_and_user(
# This is used in incbudgets-active-currency.inc
my $active_currency = Koha::Acquisition::Currencies->get_active;
-$template->param( symbol => $active_currency->symbol,
- currency => $active_currency->currency
- );
+if ( $active_currency ) {
+ $template->param( symbol => $active_currency->symbol,
+ currency => $active_currency->currency
+ );
+}
# ADD OR MODIFY A BUDGET PERIOD - BUILD SCREEN
if ( $op eq 'add_form' ) {
#--------------------------------------------------
elsif ( $op eq 'delete_confirm' ) {
## delete a budget period (preparation)
- my $dbh = C4::Context->dbh;
- ## $total = number of records linked to the record that must be deleted
+ my $funds = GetBudgets({ budget_period_id => $budget_period_id });
+ my $fund_count = scalar @$funds;
+ if ( $fund_count > 0 ) {
+ $template->param( funds_exist => 1 );
+ }
+
+ #$total = number of records linked to the record that must be deleted
my $total = 0;
my $data = GetBudgetPeriod( $budget_period_id);
-
$template->param(
- %$data
+ %$data
);
}
elsif ( $op eq 'delete_confirmed' ) {
-## delete the budget period record
-
- my $data = GetBudgetPeriod( $budget_period_id);
- DelBudgetPeriod($budget_period_id);
+ ## confirm no funds have been added to budget
+ my $funds = GetBudgets({ budget_period_id => $budget_period_id });
+ my $fund_count = scalar @$funds;
+ if ( $fund_count > 0 ) {
+ $template->param( failed_delete_funds_exist => 1 );
+ } else {
+ ## delete the budget period record
+ my $data = GetBudgetPeriod( $budget_period_id);
+ DelBudgetPeriod($budget_period_id);
+ }
$op='else';
}
elsif ( $op eq 'duplicate_budget' ){
die "please specify a budget period id\n" if( !defined $budget_period_id || $budget_period_id eq '' );
- my $budget_period_startdate = dt_from_string $input->param('budget_period_startdate');
- my $budget_period_enddate = dt_from_string $input->param('budget_period_enddate');
+ my $budget_period_startdate = dt_from_string scalar $input->param('budget_period_startdate');
+ my $budget_period_enddate = dt_from_string scalar $input->param('budget_period_enddate');
my $budget_period_description = $input->param('budget_period_description');
my $amount_change_percentage = $input->param('amount_change_percentage');
my $amount_change_round_increment = $input->param('amount_change_round_increment');
my $budgets_to_move = GetBudgetHierarchy($budget_period_id);
- # C4::Context->userenv->{branchcode}, $show_mine ? $borrower_id : '')
-
my $number_of_unreceived_orders = 0;
for my $budget (@$budgets_to_move) {
{ budget_period_active => 1 },
{ -asc => 'budget_period_description' },
);
-my $first = ( $activepage - 1 ) * $activepagesize;
-my $last = min( $first + $activepagesize - 1, scalar @{$results} - 1, );
+
my @period_active_loop;
-foreach my $result ( @{$results}[ $first .. $last ] ) {
+foreach my $result ( @{$results} ) {
my $budgetperiod = $result;
$budgetperiod->{budget_active} = 1;
+ my $funds = GetBudgets({ budget_period_id => $budgetperiod->{budget_period_id} });
+ $budgetperiod->{count} = scalar @$funds;
push( @period_active_loop, $budgetperiod );
}
-my $url = "aqbudgetperiods.pl";
-$url .= "?ipage=$inactivepage" if($inactivepage != 1);
-my $active_pagination_bar = pagination_bar ($url, getnbpages( scalar(@$results), $activepagesize), $activepage, "apage");
# Get inactive budget periods
$results = GetBudgetPeriods(
{ -desc => 'budget_period_enddate' },
);
-$first = ( $inactivepage - 1 ) * $inactivepagesize;
-$last = min( $first + $inactivepagesize - 1, scalar @{$results} - 1, );
my @period_inactive_loop;
-foreach my $result ( @{$results}[ $first .. $last ] ) {
+foreach my $result ( @{$results} ) {
my $budgetperiod = $result;
$budgetperiod->{budget_active} = 1;
+ my $funds = GetBudgets({ budget_period_id => $budgetperiod->{budget_period_id} });
+ $budgetperiod->{count} = scalar @$funds;
push( @period_inactive_loop, $budgetperiod );
}
-$url = "aqbudgetperiods.pl?tab=2";
-$url .= "&apage=$activepage" if($activepage != 1);
-my $inactive_pagination_bar = pagination_bar ($url, getnbpages( scalar(@$results), $inactivepagesize), $inactivepage, "ipage");
-
-my $branchloop = C4::Branch::GetBranchesLoop();
my $tab = $input->param('tab') ? $input->param('tab') - 1 : 0;
$template->param(
period_active_loop => \@period_active_loop,
period_inactive_loop => \@period_inactive_loop,
- active_pagination_bar => $active_pagination_bar,
- inactive_pagination_bar => $inactive_pagination_bar,
tab => $tab,
- branchloop => $branchloop,
);
$template->param($op=>1);