Attachment 50481 Details for Bug 10459 – Bug 10459: Follow up to update to atomic update methodology

[patch] Bug 10459: Follow up to update to atomic update methodology

Bug-10459-Follow-up-to-update-to-atomic-update-met.patch (text/plain), 2.30 KB, created by Mark Tompsett on 2016-04-20 19:42:32 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Mark Tompsett

Created: 2016-04-20 19:42:32 UTC

Size: 2.30 KB

patch

obsolete

>From 1324045d0f868ccc64cf02ea3e49d542afd42320 Mon Sep 17 00:00:00 2001
>From: Mark Tompsett <mtompset@hotmail.com>
>Date: Wed, 20 Apr 2016 15:22:36 -0400
>Subject: [PATCH] Bug 10459: Follow up to update to atomic update methodology
>
>Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel@gmail.com>
>New column created, no errors.
>
>Signed-off-by: Mark Tompsett <mtompset@hotmail.com>
>---
> .../mysql/atomicupdate/Bug10459_AddTimestampToBorrowers.sql |  8 ++++++++
> installer/data/mysql/updatedatabase.pl                      | 13 -------------
> 2 files changed, 8 insertions(+), 13 deletions(-)
> create mode 100644 installer/data/mysql/atomicupdate/Bug10459_AddTimestampToBorrowers.sql
>
>diff --git a/installer/data/mysql/atomicupdate/Bug10459_AddTimestampToBorrowers.sql b/installer/data/mysql/atomicupdate/Bug10459_AddTimestampToBorrowers.sql
>new file mode 100644
>index 0000000..dc3bbb2
>--- /dev/null
>+++ b/installer/data/mysql/atomicupdate/Bug10459_AddTimestampToBorrowers.sql
>@@ -0,0 +1,8 @@
>+ALTER TABLE borrowers
>+    ADD COLUMN timestamp timestamp NOT NULL DEFAULT CURRENT_TIMESTAMP
>+    ON UPDATE CURRENT_TIMESTAMP
>+    AFTER privacy_guarantor_checkouts;
>+ALTER TABLE deletedborrowers
>+    ADD COLUMN timestamp timestamp NOT NULL DEFAULT CURRENT_TIMESTAMP
>+    ON UPDATE CURRENT_TIMESTAMP
>+    AFTER privacy_guarantor_checkouts;
>diff --git a/installer/data/mysql/updatedatabase.pl b/installer/data/mysql/updatedatabase.pl
>index 4ed2d11..fb09c5e 100755
>--- a/installer/data/mysql/updatedatabase.pl
>+++ b/installer/data/mysql/updatedatabase.pl
>@@ -12220,19 +12220,6 @@ if ( C4::Context->preference("Version") < TransformToNum($DBversion) ) {
>     SetVersion($DBversion);
> }
> 
>-$DBversion = "3.23.00.XXX";
>-if ( CheckVersion($DBversion) ) {
>-    $dbh->do(q|
>-ALTER TABLE borrowers ADD COLUMN timestamp timestamp NOT NULL DEFAULT CURRENT_TIMESTAMP ON UPDATE CURRENT_TIMESTAMP AFTER privacy;
>-    |);
>-    $dbh->do(q|
>-ALTER TABLE deletedborrowers ADD COLUMN timestamp timestamp NOT NULL DEFAULT CURRENT_TIMESTAMP ON UPDATE CURRENT_TIMESTAMP AFTER privacy;
>-    |);
>-    print "Upgrade to $DBversion done (Bug 10459: Borrowers should have a timestamp)\n";
>-    SetVersion($DBversion);
>-}
>-
>-
> # DEVELOPER PROCESS, search for anything to execute in the db_update directory
> # SEE bug 13068
> # if there is anything in the atomicupdate, read and execute it.
>-- 
>2.1.4

Actions: View | Diff | Splinter Review

Attachments on bug 10459: 33421 | 33424 | 33435 | 33490 | 33491 | 33509 | 50475 | 50476 | 50478 | 50479 | 50480 | 50481 | 51807 | 51808 | 51809 | 51810 | 51842 | 51843 | 51844 | 51867 | 51868 | 52113 | 52114 | 52115 | 52294