Attachment 52113 Details for Bug 10459 – Bug 10459: Borrowers should have a timestamp

[patch] Bug 10459: Borrowers should have a timestamp

Bug-10459-Borrowers-should-have-a-timestamp.patch (text/plain), 4.06 KB, created by Jonathan Druart on 2016-06-07 09:21:35 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Jonathan Druart

Created: 2016-06-07 09:21:35 UTC

Size: 4.06 KB

patch

obsolete

>From c4e491ec5db75c7bdb36e9950494841ba7562308 Mon Sep 17 00:00:00 2001
>From: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
>Date: Mon, 10 Nov 2014 14:17:40 +0100
>Subject: [PATCH] Bug 10459: Borrowers should have a timestamp
>
>This patch adds a timestamp column to the borrowers table in kohastructure
>and updatedatabase. (And also to the deletedborrowers table.)
>A timestamp may be useful in synchronizing with external systems (among other
>reasons).
>
>Test plan:
>Run updatestructure on an existing database, or install a new one.
>Verify that the borrowers table has a timestamp now.
>
>Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
>Tested updatestructure and running kohastructure.sql.
>Passed t/db_dependent/Members.t.
>
>updatedatabase.pl did not apply. I edited and then run it. Columns were added as expected.
>Signed-off-by: Marc Veron <veron@veron.ch>
>
>Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel@gmail.com>
>
>Signed-off-by: Mark Tompsett <mtompset@hotmail.com>
>
>Signed-off-by: Joy Nelson <joy@bywatersolutions.com>
>
>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>
>
>Signed-off-by: Joy Nelson <joy@bywatersolutions.com>
>
>Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
>
>Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
>---
> .../data/mysql/atomicupdate/Bug10459_AddTimestampToBorrowers.sql  | 8 ++++++++
> installer/data/mysql/kohastructure.sql                            | 2 ++
> 2 files changed, 10 insertions(+)
> 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/kohastructure.sql b/installer/data/mysql/kohastructure.sql
>index c5a3f4a..494a4b7 100644
>--- a/installer/data/mysql/kohastructure.sql
>+++ b/installer/data/mysql/kohastructure.sql
>@@ -618,6 +618,7 @@ CREATE TABLE `deletedborrowers` ( -- stores data related to the patrons/borrower
>   `sms_provider_id` int(11) DEFAULT NULL, -- the provider of the mobile phone number defined in smsalertnumber
>   `privacy` integer(11) DEFAULT '1' NOT NULL, -- patron/borrower's privacy settings related to their reading history  KEY `borrowernumber` (`borrowernumber`),
>   `privacy_guarantor_checkouts` tinyint(1) NOT NULL DEFAULT '0', -- controls if relatives can see this patron's checkouts
>+  `timestamp` timestamp NOT NULL DEFAULT CURRENT_TIMESTAMP ON UPDATE CURRENT_TIMESTAMP, -- time of last change could be useful for synchronization with external systems (among others)
>   KEY borrowernumber (borrowernumber),
>   KEY `cardnumber` (`cardnumber`),
>   KEY `sms_provider_id` (`sms_provider_id`)
>@@ -1623,6 +1624,7 @@ CREATE TABLE `borrowers` ( -- this table includes information about your patrons
>   `sms_provider_id` int(11) DEFAULT NULL, -- the provider of the mobile phone number defined in smsalertnumber
>   `privacy` integer(11) DEFAULT '1' NOT NULL, -- patron/borrower's privacy settings related to their reading history
>   `privacy_guarantor_checkouts` tinyint(1) NOT NULL DEFAULT '0', -- controls if relatives can see this patron's checkouts
>+  `timestamp` timestamp NOT NULL DEFAULT CURRENT_TIMESTAMP ON UPDATE CURRENT_TIMESTAMP, -- time of last change could be useful for synchronization with external systems (among others)
>   UNIQUE KEY `cardnumber` (`cardnumber`),
>   PRIMARY KEY `borrowernumber` (`borrowernumber`),
>   KEY `categorycode` (`categorycode`),
>-- 
>2.8.1

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