Bug 13624 - renaming atomicupdate from .pl to .perl
authorCharles Farmer <charles.farmer@inlibro.com>
Fri, 16 Oct 2015 14:59:24 +0000 (10:59 -0400)
committerKyle M Hall <kyle@bywatersolutions.com>
Thu, 31 Dec 2015 12:47:16 +0000 (12:47 +0000)
I have not tested this very thoroughly e.g. understood the SQL changes etc. since
the patch has been signed off twice before.

But it seems to me like this patch is OK.
Applied, ran automatic tests and saved some values in the overduerules matrix/tabs OK.

Signed-off-by: Eivin Giske Skaaren <eivin@sysmystic.com>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
installer/data/mysql/atomicupdate/update_13624_overduerules_transport_type.perl [new file with mode: 0755]
installer/data/mysql/atomicupdate/update_13624_overduerules_transport_type.pl [deleted file]

diff --git a/installer/data/mysql/atomicupdate/update_13624_overduerules_transport_type.perl b/installer/data/mysql/atomicupdate/update_13624_overduerules_transport_type.perl
new file mode 100755 (executable)
index 0000000..be3af3c
--- /dev/null
@@ -0,0 +1,50 @@
+#! /usr/bin/perl
+
+use strict;
+use warnings;
+use C4::Context;
+my $dbh=C4::Context->dbh;
+
+print "Will do : Bug 13624 - Remove columns branchcode, categorytype from table overduerules_transport_types\n";
+
+#if ( CheckVersion($DBversion) ) {
+    $dbh->do("SET FOREIGN_KEY_CHECKS=0");
+    $dbh->do("ALTER TABLE overduerules RENAME old_overduerules");
+    $dbh->do("CREATE TABLE overduerules (
+        `overduerules_id` int(11) NOT NULL AUTO_INCREMENT,
+        `branchcode` varchar(10) NOT NULL DEFAULT '',
+        `categorycode` varchar(10) NOT NULL DEFAULT '',
+        `delay1` int(4) DEFAULT NULL,
+        `letter1` varchar(20) DEFAULT NULL,
+        `debarred1` varchar(1) DEFAULT '0',
+        `delay2` int(4) DEFAULT NULL,
+        `debarred2` varchar(1) DEFAULT '0',
+        `letter2` varchar(20) DEFAULT NULL,
+        `delay3` int(4) DEFAULT NULL,
+        `letter3` varchar(20) DEFAULT NULL,
+        `debarred3` int(1) DEFAULT '0',
+        PRIMARY KEY (`overduerules_id`),
+        UNIQUE KEY `overduerules_branch_cat` (`branchcode`,`categorycode`)
+        ) ENGINE=InnoDB DEFAULT CHARSET=utf8;");
+    $dbh->do("INSERT INTO overduerules(branchcode, categorycode, delay1, letter1, debarred1, delay2, debarred2, letter2, delay3, letter3, debarred3) SELECT * FROM old_overduerules");
+    $dbh->do("DROP TABLE old_overduerules");
+    $dbh->do("ALTER TABLE overduerules_transport_types
+              ADD COLUMN overduerules_id int(11) NOT NULL");
+    my $mtts = $dbh->selectall_arrayref("SELECT * FROM overduerules_transport_types", { Slice => {} });
+    $dbh->do("DELETE FROM overduerules_transport_types");
+    $dbh->do("ALTER TABLE overduerules_transport_types
+              DROP FOREIGN KEY overduerules_fk,
+              ADD FOREIGN KEY overduerules_transport_types_fk (overduerules_id) REFERENCES overduerules (overduerules_id) ON DELETE CASCADE ON UPDATE CASCADE,
+              DROP COLUMN branchcode,
+              DROP COLUMN categorycode");
+    my $s = $dbh->prepare("INSERT INTO overduerules_transport_types (overduerules_id, id, letternumber, message_transport_type) "
+                         ." VALUES((SELECT overduerules_id FROM overduerules WHERE branchcode = ? AND categorycode = ?),?,?,?)");
+    foreach my $mtt(@$mtts){
+        $s->execute($mtt->{branchcode}, $mtt->{categorycode}, $mtt->{id}, $mtt->{letternumber}, $mtt->{message_transport_type} );
+    }
+    $dbh->do("SET FOREIGN_KEY_CHECKS=1");
+#    print "Upgrade to $DBversion done (Bug 13624 - Remove columns branchcode, categorytype from table overduerules_transport_types)\n";
+#   SetVersion ($DBversion);
+#}
+
+print "\nDone\n";
diff --git a/installer/data/mysql/atomicupdate/update_13624_overduerules_transport_type.pl b/installer/data/mysql/atomicupdate/update_13624_overduerules_transport_type.pl
deleted file mode 100755 (executable)
index be3af3c..0000000
+++ /dev/null
@@ -1,50 +0,0 @@
-#! /usr/bin/perl
-
-use strict;
-use warnings;
-use C4::Context;
-my $dbh=C4::Context->dbh;
-
-print "Will do : Bug 13624 - Remove columns branchcode, categorytype from table overduerules_transport_types\n";
-
-#if ( CheckVersion($DBversion) ) {
-    $dbh->do("SET FOREIGN_KEY_CHECKS=0");
-    $dbh->do("ALTER TABLE overduerules RENAME old_overduerules");
-    $dbh->do("CREATE TABLE overduerules (
-        `overduerules_id` int(11) NOT NULL AUTO_INCREMENT,
-        `branchcode` varchar(10) NOT NULL DEFAULT '',
-        `categorycode` varchar(10) NOT NULL DEFAULT '',
-        `delay1` int(4) DEFAULT NULL,
-        `letter1` varchar(20) DEFAULT NULL,
-        `debarred1` varchar(1) DEFAULT '0',
-        `delay2` int(4) DEFAULT NULL,
-        `debarred2` varchar(1) DEFAULT '0',
-        `letter2` varchar(20) DEFAULT NULL,
-        `delay3` int(4) DEFAULT NULL,
-        `letter3` varchar(20) DEFAULT NULL,
-        `debarred3` int(1) DEFAULT '0',
-        PRIMARY KEY (`overduerules_id`),
-        UNIQUE KEY `overduerules_branch_cat` (`branchcode`,`categorycode`)
-        ) ENGINE=InnoDB DEFAULT CHARSET=utf8;");
-    $dbh->do("INSERT INTO overduerules(branchcode, categorycode, delay1, letter1, debarred1, delay2, debarred2, letter2, delay3, letter3, debarred3) SELECT * FROM old_overduerules");
-    $dbh->do("DROP TABLE old_overduerules");
-    $dbh->do("ALTER TABLE overduerules_transport_types
-              ADD COLUMN overduerules_id int(11) NOT NULL");
-    my $mtts = $dbh->selectall_arrayref("SELECT * FROM overduerules_transport_types", { Slice => {} });
-    $dbh->do("DELETE FROM overduerules_transport_types");
-    $dbh->do("ALTER TABLE overduerules_transport_types
-              DROP FOREIGN KEY overduerules_fk,
-              ADD FOREIGN KEY overduerules_transport_types_fk (overduerules_id) REFERENCES overduerules (overduerules_id) ON DELETE CASCADE ON UPDATE CASCADE,
-              DROP COLUMN branchcode,
-              DROP COLUMN categorycode");
-    my $s = $dbh->prepare("INSERT INTO overduerules_transport_types (overduerules_id, id, letternumber, message_transport_type) "
-                         ." VALUES((SELECT overduerules_id FROM overduerules WHERE branchcode = ? AND categorycode = ?),?,?,?)");
-    foreach my $mtt(@$mtts){
-        $s->execute($mtt->{branchcode}, $mtt->{categorycode}, $mtt->{id}, $mtt->{letternumber}, $mtt->{message_transport_type} );
-    }
-    $dbh->do("SET FOREIGN_KEY_CHECKS=1");
-#    print "Upgrade to $DBversion done (Bug 13624 - Remove columns branchcode, categorytype from table overduerules_transport_types)\n";
-#   SetVersion ($DBversion);
-#}
-
-print "\nDone\n";