Bug 12508: adding an error message if a contract cannot be removed
[koha_fer] / tools / overduerules.pl
index 478a762..11dd9bd 100755 (executable)
@@ -27,8 +27,9 @@ use C4::Koha;
 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(
@@ -36,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 {
@@ -44,11 +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;
@@ -84,6 +85,18 @@ 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 =~ /(delay|letter|debarred)([1-3])-(.*)/) {
@@ -166,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);
+                        }
+                    }
                 }
         }
     }
@@ -179,43 +201,41 @@ if ($op eq 'save') {
 }
 my $branchloop = GetBranchesLoop($branch);
 
-my $letters = GetLetters("circulation");
-
-my $countletters = keys %{$letters};
+my $letters = GetLetters({ module => "circulation" });
 
 my @line_loop;
 
+my $message_transport_types = C4::Letters::GetMessageTransportTypes();
+my ( @first, @second, @third );
 for my $data (@categories) {
-    my %row = (
-        overduename => $data->{'categorycode'},
-        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;
-                    if ( $temphash{$data->{categorycode}}->{"letter$i"} &&
-                        $thisletter eq $temphash{$data->{'categorycode'}}->{"letter$i"}) {
-                        $selected = 1;
-                    }
-                    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 {
@@ -223,33 +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;
-                    if ($dat->{"letter$i"} && $thisletter eq $dat->{"letter$i"}) {
-                        $selected = 1;
-                    }
-                    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"};}
+
         }
     }
-    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;