use C4::Koha;
use C4::Branch; # GetBranches
use C4::Letters;
+use C4::Members;
my $input = new CGI;
my $dbh = C4::Context->dbh;
+my @categories = @{$dbh->selectall_arrayref(
+ 'SELECT description, categorycode FROM categories WHERE overduenoticerequired > 0',
+ { Slice => {} }
+)};
+my @category_codes = map { $_->{categorycode} } @categories;
+my @rule_params = qw(delay letter debarred);
+
+# blank_row($category_code) - return true if the entire row is blank.
+sub blank_row {
+ my ($category_code) = @_;
+ for my $rp (@rule_params) {
+ for my $n (1 .. 3) {
+ my $key = "${rp}${n}-$category_code";
+ my $value = $input->param($key);
+ if ($value) {
+ return 0;
+ }
+ }
+ }
+ return 1;
+}
+
my $type=$input->param('type');
my $branch = $input->param('branch');
$branch="" unless $branch;
query => $input,
type => "intranet",
authnotrequired => 0,
- flagsrequired => {parameters => 1, tools => 'edit_notice_status_triggers'},
+ flagsrequired => { tools => 'edit_notice_status_triggers'},
debug => 1,
});
my $err=0;
# save the values entered into tables
my %temphash;
+my $input_saved = 0;
if ($op eq 'save') {
my @names=$input->param();
my $sth_search = $dbh->prepare("SELECT count(*) AS total FROM overduerules WHERE branchcode=? AND categorycode=?");
$temphash{$bor}->{"$type$num"}=$input->param("$key") if (($input->param("$key") ne "") or ($input->param("$key")>0));
}
}
+
+ # figure out which rows need to be deleted
+ my @rows_to_delete = grep { blank_row($_) } @category_codes;
+
foreach my $bor (keys %temphash){
+ # get category name if we need it for an error message
+ my $bor_category = GetBorrowercategory($bor);
+ my $bor_category_name = defined($bor_category) ? $bor_category->{description} : $bor;
+
# Do some Checking here : delay1 < delay2 <delay3 all of them being numbers
# Raise error if not true
if ($temphash{$bor}->{delay1}=~/[^0-9]/ and $temphash{$bor}->{delay1} ne ""){
- $template->param("ERROR"=>1,"ERRORDELAY"=>"delay1","BORERR"=>$bor);
+ $template->param("ERROR"=>1,"ERRORDELAY"=>"delay1","BORERR"=>$bor_category_name);
$err=1;
} elsif ($temphash{$bor}->{delay2}=~/[^0-9]/ and $temphash{$bor}->{delay2} ne ""){
- $template->param("ERROR"=>1,"ERRORDELAY"=>"delay2","BORERR"=>$bor);
+ $template->param("ERROR"=>1,"ERRORDELAY"=>"delay2","BORERR"=>$bor_category_name);
$err=1;
} elsif ($temphash{$bor}->{delay3}=~/[^0-9]/ and $temphash{$bor}->{delay3} ne ""){
- $template->param("ERROR"=>1,"ERRORDELAY"=>"delay3","BORERR"=>$bor);
+ $template->param("ERROR"=>1,"ERRORDELAY"=>"delay3","BORERR"=>$bor_category_name);
+ $err=1;
+ } elsif ($temphash{$bor}->{delay1} and not ($temphash{$bor}->{"letter1"} or $temphash{$bor}->{"debarred1"})) {
+ $template->param("ERROR"=>1,"ERRORUSELESSDELAY"=>"delay1","BORERR"=>$bor_category_name);
+ $err=1;
+ } elsif ($temphash{$bor}->{delay2} and not ($temphash{$bor}->{"letter2"} or $temphash{$bor}->{"debarred2"})) {
+ $template->param("ERROR"=>1,"ERRORUSELESSDELAY"=>"delay2","BORERR"=>$bor_category_name);
+ $err=1;
+ } elsif ($temphash{$bor}->{delay3} and not ($temphash{$bor}->{"letter3"} or $temphash{$bor}->{"debarred3"})) {
+ $template->param("ERROR"=>1,"ERRORUSELESSDELAY"=>"delay3","BORERR"=>$bor_category_name);
$err=1;
}elsif ($temphash{$bor}->{delay3} and
($temphash{$bor}->{delay3}<=$temphash{$bor}->{delay2} or $temphash{$bor}->{delay3}<=$temphash{$bor}->{delay1})
or $temphash{$bor}->{delay2} and ($temphash{$bor}->{delay2}<=$temphash{$bor}->{delay1})){
- $template->param("ERROR"=>1,"ERRORORDER"=>1,"BORERR"=>$bor);
+ $template->param("ERROR"=>1,"ERRORORDER"=>1,"BORERR"=>$bor_category_name);
$err=1;
}
unless ($err){
}
}
}
- unless ($err) {$template->param(datasaved=>1);}
+ unless ($err) {
+ for my $category_code (@rows_to_delete) {
+ $sth_delete->execute($branch, $category_code);
+ }
+ $template->param(datasaved => 1);
+ $input_saved = 1;
+ }
}
my $branches = GetBranches();
my @branchloop;
-foreach my $thisbranch (keys %$branches) {
+foreach my $thisbranch (sort { $branches->{$a}->{branchname} cmp $branches->{$b}->{branchname} } keys %$branches) {
my $selected = 1 if $thisbranch eq $branch;
my %row =(value => $thisbranch,
selected => $selected,
my $countletters = scalar $letters;
-my $sth=$dbh->prepare("SELECT description,categorycode FROM categories WHERE overduenoticerequired>0 ORDER BY description");
-$sth->execute;
my @line_loop;
-my $toggle= 1;
-# my $i=0;
-while (my $data=$sth->fetchrow_hashref){
+my $toggle = 1;
+
+for my $data (@categories) {
if ( $toggle eq 1 ) {
$toggle = 0;
} else {
$toggle = 1;
}
- my %row = ( overduename => $data->{'categorycode'},
- toggle => $toggle,
- line => $data->{'description'}
- );
- if (%temphash){
+ my %row = (
+ overduename => $data->{'categorycode'},
+ toggle => $toggle,
+ line => $data->{'description'}
+ );
+ if (%temphash and not $input_saved){
+ # if we managed to save the form submission, don't
+ # reuse %temphash, but take the values from the
+ # database - this makes it easier to identify
+ # bugs where the form submission was not correctly saved
for (my $i=1;$i<=3;$i++){
$row{"delay$i"}=$temphash{$data->{'categorycode'}}->{"delay$i"};
$row{"debarred$i"}=$temphash{$data->{'categorycode'}}->{"debarred$i"};
if ($countletters){
my @letterloop;
- foreach my $thisletter (keys %$letters) {
+ foreach my $thisletter (sort { $letters->{$a} cmp $letters->{$b} } keys %$letters) {
my $selected = 1 if $thisletter eq $temphash{$data->{'categorycode'}}->{"letter$i"};
my %letterrow =(value => $thisletter,
selected => $selected,
for (my $i=1;$i<=3;$i++){
if ($countletters){
my @letterloop;
- foreach my $thisletter (keys %$letters) {
+ foreach my $thisletter (sort { $letters->{$a} cmp $letters->{$b} } keys %$letters) {
my $selected = 1 if $thisletter eq $dat->{"letter$i"};
my %letterrow =(value => $thisletter,
selected => $selected,
}
push @line_loop,\%row;
}
-$sth->finish;
$template->param(table=> \@line_loop,
branchloop => \@branchloop,