Bug 12508: adding an error message if a contract cannot be removed
[koha_fer] / tools / overduerules.pl
index e390250..11dd9bd 100755 (executable)
 # 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.
 
 use strict;
+use warnings;
 use CGI;
 use C4::Context;
 use C4::Output;
 use C4::Auth;
 use C4::Koha;
-use C4::Branch; # GetBranches
+use C4::Branch;
 use C4::Letters;
 use C4::Members;
+use C4::Overdues;
 
-my $input = new CGI;
+our $input = new CGI;
 my $dbh = C4::Context->dbh;
 
 my @categories = @{$dbh->selectall_arrayref(
@@ -35,7 +37,7 @@ my @categories = @{$dbh->selectall_arrayref(
     { Slice => {} }
 )};
 my @category_codes  = map { $_->{categorycode} } @categories;
-my @rule_params     = qw(delay letter debarred);
+our @rule_params     = qw(delay letter debarred);
 
 # blank_row($category_code) - return true if the entire row is blank.
 sub blank_row {
@@ -43,6 +45,11 @@ sub blank_row {
     for my $rp (@rule_params) {
         for my $n (1 .. 3) {
             my $key   = "${rp}${n}-$category_code";
+
+            if (utf8::is_utf8($key)) {
+              utf8::encode($key);
+            }
+
             my $value = $input->param($key);
             if ($value) {
                 return 0;
@@ -54,11 +61,10 @@ sub blank_row {
 
 my $type=$input->param('type');
 my $branch = $input->param('branch');
-$branch="" unless $branch;
+$branch ||= q{};
 my $op = $input->param('op');
+$op ||= q{};
 
-# my $flagsrequired;
-# $flagsrequired->{circulation}=1;
 my ($template, $loggedinuser, $cookie)
     = get_template_and_user({template_name => "tools/overduerules.tmpl",
                             query => $input,
@@ -79,13 +85,31 @@ if ($op eq 'save') {
     my $sth_insert = $dbh->prepare("INSERT INTO overduerules (branchcode,categorycode, delay1,letter1,debarred1, delay2,letter2,debarred2, delay3,letter3,debarred3) VALUES (?,?,?,?,?,?,?,?,?,?,?)");
     my $sth_update=$dbh->prepare("UPDATE overduerules SET delay1=?, letter1=?, debarred1=?, delay2=?, letter2=?, debarred2=?, delay3=?, letter3=?, debarred3=? WHERE branchcode=? AND categorycode=?");
     my $sth_delete=$dbh->prepare("DELETE FROM overduerules WHERE branchcode=? AND categorycode=?");
+    my $sth_insert_mtt = $dbh->prepare("
+        INSERT INTO overduerules_transport_types(
+            branchcode, categorycode, letternumber, message_transport_type
+        ) VALUES (
+            ?, ?, ?, ?
+        )
+    ");
+    my $sth_delete_mtt = $dbh->prepare("
+        DELETE FROM overduerules_transport_types
+        WHERE branchcode = ? AND categorycode = ?
+    ");
+
     foreach my $key (@names){
             # ISSUES
-            if ($key =~ /(.*)([1-3])-(.*)/) {
+            if ($key =~ /(delay|letter|debarred)([1-3])-(.*)/) {
                     my $type = $1; # data type
                     my $num = $2; # From 1 to 3
                     my $bor = $3; # borrower category
-                    $temphash{$bor}->{"$type$num"}=$input->param("$key") if (($input->param("$key") ne "") or ($input->param("$key")>0));
+                    my $value = $input->param($key);
+                    if ($type eq 'delay') {
+                        $temphash{$bor}->{"$type$num"} = ($value =~ /^\d+$/ && int($value) > 0) ? int($value) : '';
+                    } else {
+                        # type is letter
+                        $temphash{$bor}->{"$type$num"} = $value if $value ne '';
+                    }
             }
     }
 
@@ -131,13 +155,13 @@ if ($op eq 'save') {
                     my $res = $sth_search->fetchrow_hashref();
                     if ($res->{'total'}>0) {
                         $sth_update->execute(
-                            ($temphash{$bor}->{"delay1"}?$temphash{$bor}->{"delay1"}:0),
+                            ($temphash{$bor}->{"delay1"}?$temphash{$bor}->{"delay1"}:undef),
                             ($temphash{$bor}->{"letter1"}?$temphash{$bor}->{"letter1"}:""),
                             ($temphash{$bor}->{"debarred1"}?$temphash{$bor}->{"debarred1"}:0),
-                            ($temphash{$bor}->{"delay2"}?$temphash{$bor}->{"delay2"}:0),
+                            ($temphash{$bor}->{"delay2"}?$temphash{$bor}->{"delay2"}:undef),
                             ($temphash{$bor}->{"letter2"}?$temphash{$bor}->{"letter2"}:""),
                             ($temphash{$bor}->{"debarred2"}?$temphash{$bor}->{"debarred2"}:0),
-                            ($temphash{$bor}->{"delay3"}?$temphash{$bor}->{"delay3"}:0),
+                            ($temphash{$bor}->{"delay3"}?$temphash{$bor}->{"delay3"}:undef),
                             ($temphash{$bor}->{"letter3"}?$temphash{$bor}->{"letter3"}:""),
                             ($temphash{$bor}->{"debarred3"}?$temphash{$bor}->{"debarred3"}:0),
                             $branch ,$bor
@@ -155,6 +179,15 @@ if ($op eq 'save') {
                             ($temphash{$bor}->{"debarred3"}?$temphash{$bor}->{"debarred3"}:0)
                             );
                     }
+
+                    $sth_delete_mtt->execute( $branch, $bor );
+                    for my $letternumber ( 1..3 ) {
+                        my @mtt = $input->param( "mtt${letternumber}-$bor" );
+                        next unless @mtt;
+                        for my $mtt ( @mtt ) {
+                            $sth_insert_mtt->execute( $branch, $bor, $letternumber, $mtt);
+                        }
+                    }
                 }
         }
     }
@@ -166,57 +199,43 @@ if ($op eq 'save') {
         $input_saved = 1;
     }
 }
-my $branches = GetBranches();
-my @branchloop;
-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,
-                                branchname => $branches->{$thisbranch}->{'branchname'},
-                        );
-        push @branchloop, \%row;
-}
-
-my $letters = GetLetters("circulation");
+my $branchloop = GetBranchesLoop($branch);
 
-my $countletters = scalar $letters;
+my $letters = GetLetters({ module => "circulation" });
 
 my @line_loop;
-my $toggle = 1;
 
+my $message_transport_types = C4::Letters::GetMessageTransportTypes();
+my ( @first, @second, @third );
 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 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 (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,
-                                    lettername => $letters->{$thisletter},
-                                    );
-                    push @letterloop, \%letterrow;
+        for my $i ( 1..3 ){
+            my %row = (
+                overduename => $data->{'categorycode'},
+                line        => $data->{'description'}
+            );
+            $row{delay}=$temphash{$data->{'categorycode'}}->{"delay$i"};
+            $row{debarred}=$temphash{$data->{'categorycode'}}->{"debarred$i"};
+            $row{selected_lettercode} = $temphash{ $data->{categorycode} }->{"letter$i"};
+            my @selected_mtts = @{ GetOverdueMessageTransportTypes( $branch, $data->{'categorycode'}, $i) };
+            my @mtts;
+            for my $mtt ( @$message_transport_types ) {
+                push @mtts, {
+                    value => $mtt,
+                    selected => ( grep {/$mtt/} @selected_mtts ) ? 1 : 0 ,
                 }
-                $row{"letterloop$i"}=\@letterloop;
+            }
+            $row{message_transport_types} = \@mtts;
+            if ( $i == 1 ) {
+                push @first, \%row;
+            } elsif ( $i == 2 ) {
+                push @second, \%row;
             } else {
-                $row{"noletter"}=1;
-                $row{"letter$i"}=$temphash{$data->{'categorycode'}}->{"letter$i"};
+                push @third, \%row;
             }
         }
     } else {
@@ -224,31 +243,61 @@ for my $data (@categories) {
         my $sth2=$dbh->prepare("SELECT * from overduerules WHERE branchcode=? AND categorycode=?");
         $sth2->execute($branch,$data->{'categorycode'});
         my $dat=$sth2->fetchrow_hashref;
-        for (my $i=1;$i<=3;$i++){
-            if ($countletters){
-                my @letterloop;
-                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,
-                                    lettername => $letters->{$thisletter},
-                                    );
-                    push @letterloop, \%letterrow;
+        for my $i ( 1..3 ){
+            my %row = (
+                overduename => $data->{'categorycode'},
+                line        => $data->{'description'}
+            );
+
+            $row{selected_lettercode} = $dat->{"letter$i"};
+
+            if ($dat->{"delay$i"}){$row{delay}=$dat->{"delay$i"};}
+            if ($dat->{"debarred$i"}){$row{debarred}=$dat->{"debarred$i"};}
+            my @selected_mtts = @{ GetOverdueMessageTransportTypes( $branch, $data->{'categorycode'}, $i) };
+            my @mtts;
+            for my $mtt ( @$message_transport_types ) {
+                push @mtts, {
+                    value => $mtt,
+                    selected => ( grep {/$mtt/} @selected_mtts ) ? 1 : 0 ,
                 }
-                $row{"letterloop$i"}=\@letterloop;
+            }
+            $row{message_transport_types} = \@mtts;
+            if ( $i == 1 ) {
+                push @first, \%row;
+            } elsif ( $i == 2 ) {
+                push @second, \%row;
             } else {
-                $row{"noletter"}=1;
-                if ($dat->{"letter$i"}){$row{"letter$i"}=$dat->{"letter$i"};}
+                push @third, \%row;
             }
-            if ($dat->{"delay$i"}){$row{"delay$i"}=$dat->{"delay$i"};}
-            if ($dat->{"debarred$i"}){$row{"debarred$i"}=$dat->{"debarred$i"};}
+
         }
-        $sth2->finish;
     }
-    push @line_loop,\%row;
 }
 
-$template->param(table=> \@line_loop,
-                branchloop => \@branchloop,
-                branch => $branch);
+my @tabs = (
+    {
+        id => 'first',
+        number => 1,
+        values => \@first,
+    },
+    {
+        id => 'second',
+        number => 2,
+        values => \@second,
+    },
+    {
+        id => 'third',
+        number => 3,
+        values => \@third,
+    },
+);
+
+$template->param(
+    table => ( @first or @second or @third ? 1 : 0 ),
+    branchloop => $branchloop,
+    branch => $branch,
+    tabs => \@tabs,
+    message_transport_types => $message_transport_types,
+    letters => $letters,
+);
 output_html_with_http_headers $input, $cookie, $template->output;