From a6b2f6a16175229ba603ed65cfeaf4e0c0a525ce Mon Sep 17 00:00:00 2001 From: Galen Charlton Date: Fri, 13 Dec 2013 01:06:12 +0000 Subject: [PATCH] Bug 11391: drop default value on suggestions.suggesteddate column The 'default 0' clause got translated as an invalid constant default of '0000-00-00' when DBIx::Schema is used to deploy the suggestions table into a Pg database. This patch drops the default. To test: [1] Apply the patch and run the SQL specified in the database updated. [2] Verify that the suggestions table no longer has an explicit default value for the suggesteddate column. [3] Verify that prove -v t/db_dependent/Suggestions.t passes. [4] Verify that installer/data/mysql/kohastructure.sql runs cleanly in an empty database. [5] Verify that there are no visible regressions of the purchase suggestions functionality. Signed-off-by: Galen Charlton Signed-off-by: Chris Cormack Having a default of 0 on a date seems like a mad thing to do anyway, so good to get rid of it Signed-off-by: Jonathan Druart Signed-off-by: Galen Charlton --- installer/data/mysql/kohastructure.sql | 2 +- installer/data/mysql/updatedatabase.pl | 9 +++++++++ 2 files changed, 10 insertions(+), 1 deletion(-) diff --git a/installer/data/mysql/kohastructure.sql b/installer/data/mysql/kohastructure.sql index aba11b8b90..728d7ca367 100644 --- a/installer/data/mysql/kohastructure.sql +++ b/installer/data/mysql/kohastructure.sql @@ -2147,7 +2147,7 @@ DROP TABLE IF EXISTS `suggestions`; CREATE TABLE `suggestions` ( -- purchase suggestions `suggestionid` int(8) NOT NULL auto_increment, -- unique identifier assigned automatically by Koha `suggestedby` int(11) NOT NULL default 0, -- borrowernumber for the person making the suggestion, foreign key linking to the borrowers table - `suggesteddate` date NOT NULL default 0, -- date the suggestion was submitted + `suggesteddate` date NOT NULL, -- date the suggestion was submitted `managedby` int(11) default NULL, -- borrowernumber for the librarian managing the suggestion, foreign key linking to the borrowers table `manageddate` date default NULL, -- date the suggestion was updated acceptedby INT(11) default NULL, -- borrowernumber for the librarian who accepted the suggestion, foreign key linking to the borrowers table diff --git a/installer/data/mysql/updatedatabase.pl b/installer/data/mysql/updatedatabase.pl index cbb20821bc..13cde78f3d 100755 --- a/installer/data/mysql/updatedatabase.pl +++ b/installer/data/mysql/updatedatabase.pl @@ -7859,6 +7859,15 @@ if ( CheckVersion($DBversion) ) { ALTER TABLE collections_tracking CHANGE ctId collections_tracking_id integer(11) NOT NULL auto_increment; }); print "Upgrade to $DBversion done (Bug 11384) - change name of collections_tracker.ctId column)\n"; + SetVersion ($DBversion); +} + +$DBversion = "3.15.00.XXX"; +if ( CheckVersion($DBversion) ) { + $dbh->do(q{ + ALTER TABLE suggestions MODIFY suggesteddate DATE NOT NULL + }); + print "Upgrade to $DBversion done (Bug 11391) - drop default value on suggestions.suggesteddate column)\n"; SetVersion($DBversion); } -- 2.39.5