use List::Util qw/min/;
use Number::Format qw(format_price);
+use Koha::Database;
use C4::Auth qw/get_user_subpermissions/;
use C4::Branch; # GetBranches
use C4::Dates qw/format_date format_date_in_iso/;
use C4::Auth;
use C4::Acquisition;
-use C4::Budgets; #
+use C4::Budgets;
use C4::Members; # calls GetSortDetails()
use C4::Context;
use C4::Output;
use C4::Koha;
use C4::Debug;
-#use POSIX qw(locale_h);
my $input = new CGI;
my $dbh = C4::Context->dbh;
my ($template, $borrowernumber, $cookie, $staffflags ) = get_template_and_user(
- { template_name => "admin/aqbudgets.tmpl",
+ { template_name => "admin/aqbudgets.tt",
query => $input,
type => "intranet",
authnotrequired => 0,
currency => $cur->{currency}
);
-my $op = $input->param('op') // '';
+my $op = $input->param('op') || 'list';
-# see if the user want to see all budgets or only owned ones
-my $show_mine = 1; #SHOW BY DEFAULT
-my $show = $input->param('show') // 0; # SET TO 1, BY A FORM SUMBIT
-$show_mine = $input->param('show_mine') if $show == 1;
+# see if the user want to see all budgets or only owned ones by default
+my $show_mine = $input->param('show_mine') // 1;
# IF USER DOESNT HAVE PERM FOR AN 'ADD', THEN REDIRECT TO THE DEFAULT VIEW...
if (not defined $template->{VARS}->{'CAN_user_acquisition_budget_add_del'}
and $op eq 'add_form')
{
- $op = '';
+ $op = 'list';
}
my $num=FormatNumber;
-my $script_name = "/cgi-bin/koha/admin/aqbudgets.pl";
-my $budget_hash = $input->Vars;
-my $budget_id = $$budget_hash{budget_id};
+# get only the columns of aqbudgets in budget_hash
+my @columns = Koha::Database->new()->schema->source('Aqbudget')->columns;
+my $budget_hash = { map { join(' ',@columns) =~ /$_/ ? ( $_ => $input->param($_) ) : () } keys( %{$input->Vars()}) } ;
+
+my $budget_id = $input->param('budget_id');
+my $budget_period_id = $input->param('budget_period_id');
my $budget_permission = $input->param('budget_permission');
+my $budget_users_ids = $input->param('budget_users_ids');
my $filter_budgetbranch = $input->param('filter_budgetbranch') // '';
my $filter_budgetname = $input->param('filter_budgetname');
-#filtering non budget keys
-delete $$budget_hash{$_} foreach grep {/filter|^op$|show/} keys %$budget_hash;
-$template->param(
- notree => ($filter_budgetbranch or $show_mine)
-);
+
# ' ------- get periods stuff ------------------'
# IF PERIODID IS DEFINED, GET THE PERIOD - ELSE JUST GET THE ACTIVE PERIOD BY DEFAULT
-my $period = GetBudgetPeriod($$budget_hash{budget_period_id});
+my $period;
+if ( $budget_period_id ) {
+ $period = GetBudgetPeriod( $budget_period_id );
+}
-$template->param(
- %$period
-);
# ------- get periods stuff ------------------
# USED FOR PERMISSION COMPARISON LATER
my $user_branchcode = $user->{'branchcode'};
$template->param(
- action => $script_name,
- script_name => $script_name,
show_mine => $show_mine,
- $op || else => 1,
+ op => $op,
);
-
# retrieve branches
my ( $budget, );
push @branchloop2, \%row;
}
-$template->param(auth_cats_loop => GetBudgetAuthCats($$period{budget_period_id}) );
+$template->param(auth_cats_loop => GetBudgetAuthCats( $budget_period_id ))
+ if $budget_period_id;
# Used to create form to add or modify a record
if ($op eq 'add_form') {
# if no buget_id is passed then its an add
$template->param(
- add_validate => 1,
budget_parent_id => $budget_parent->{'budget_id'},
budget_parent_name => $budget_parent->{'budget_name'},
branchloop_select => \@branchloop_select,
);
# END $OP eq DELETE_CONFIRM
# called by delete_confirm, used to effectively confirm deletion of data in DB
-} else{
- if ( $op eq 'delete_confirmed' ) {
- my $rc = DelBudget($budget_id);
- }elsif( $op eq 'add_validate' ) {
- my @budgetusersid;
- if (defined $$budget_hash{'budget_users_ids'}){
- @budgetusersid = split(':', $budget_hash->{'budget_users_ids'});
- }
+} elsif ( $op eq 'delete_confirmed' ) {
+ my $rc = DelBudget($budget_id);
+ $op = 'list';
+} elsif( $op eq 'add_validate' ) {
+ my @budgetusersid;
+ if (defined $budget_users_ids){
+ @budgetusersid = split(':', $budget_users_ids);
+ }
- if ( defined $$budget_hash{budget_id} ) {
- if (CanUserModifyBudget($borrowernumber, $budget_hash->{budget_id},
- $staffflags)
- ) {
- ModBudget( $budget_hash );
- ModBudgetUsers($budget_hash->{budget_id}, @budgetusersid);
- }
- else {
- $template->param(error_not_authorised_to_modify => 1);
- }
- } else {
- AddBudget( $budget_hash );
+ if (defined $budget_id) {
+ if (CanUserModifyBudget($borrowernumber, $budget_hash->{budget_id},
+ $staffflags)
+ ) {
+ ModBudget( $budget_hash );
ModBudgetUsers($budget_hash->{budget_id}, @budgetusersid);
}
+ else {
+ $template->param(error_not_authorised_to_modify => 1);
+ }
+ } else {
+ AddBudget( $budget_hash );
+ ModBudgetUsers($budget_hash->{budget_id}, @budgetusersid);
}
+ $op = 'list';
+}
+
+if ( $op eq 'list' ) {
my $branches = GetBranches();
$template->param(
- budget_id => $budget_id,
+ budget_id => $budget_id,
%$period,
);
C4::Context->userenv->{branchcode}, $show_mine ? $borrower_id : '')
};
- my $toggle = 0;
- my @loop;
my $period_total = 0;
my ($period_alloc_total, $spent_total, $ordered_total, $available_total) = (0,0,0,0);
#This Looks WEIRD to me : should budgets be filtered in such a way ppl who donot own it would not see the amount spent on the budget by others ?
foreach my $budget (@budgets) {
- #Level and sublevels total spent and ordered
- $budget->{total_spent} = $budget->{budget_spent_sublevels} + $budget->{budget_spent};
- $budget->{total_ordered} = $budget->{budget_ordered_sublevels} + $budget->{budget_ordered};
# PERMISSIONS
unless(CanUserModifyBudget($borrowernumber, $budget, $staffflags)) {
$budget->{'budget_lock'} = 1;
push @budget_hierarchy, { element_name => $period->{"budget_period_description"} };
@budget_hierarchy = reverse(@budget_hierarchy);
- push( @loop, { %{$budget},
- branchname => $branches->{ $budget->{branchcode} }->{branchname},
- budget_hierarchy => \@budget_hierarchy,
- }
- );
+ $budget->{branchname} = $branches->{ $budget->{branchcode} }->{branchname};
+ $budget->{budget_hierarchy} = \@budget_hierarchy;
}
my $budget_period_total = $period->{budget_period_total};
$_ = $num->format_price($_);
}
+ my $periods = GetBudgetPeriods();
+
$template->param(
- else => 1,
- budget => \@loop,
+ op => 'list',
+ budgets => \@budgets,
+ periods => $periods,
budget_period_total => $budget_period_total,
period_alloc_total => $period_alloc_total,
spent_total => $spent_total,
branchloop => \@branchloop2,
);
-} #---- END $OP eq DEFAULT
+} #---- END list
output_html_with_http_headers $input, $cookie, $template->output;