From ace0ca3de88bbb2c8160db0aeb1b0dcf7d64a724 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Michael=20T=C3=A4nzer?= <neo@nhng.de>
Date: Fri, 4 Nov 2011 19:16:58 +0100
Subject: bug 976: allow multiple entries in schema_version, each entry marks a
 migration
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Signed-off-by: Michael Tänzer <neo@nhng.de>
---
 scripts/db_migrations/version1.sh | 28 ++++++++++++----------------
 1 file changed, 12 insertions(+), 16 deletions(-)

diff --git a/scripts/db_migrations/version1.sh b/scripts/db_migrations/version1.sh
index 7d48df8..48e24f9 100755
--- a/scripts/db_migrations/version1.sh
+++ b/scripts/db_migrations/version1.sh
@@ -21,8 +21,7 @@
 
 # This particular version migrates from the preversioned state to version 1
 # If you want to reuse it for further migrations you probably should pay special 
-# attention because you have to adjust it quite a bit (e.g. check the "version"
-# column in the "schema_version" table instead of whether it has any entries)
+# attention because you have to adjust it a bit
 
 set -e # script fails if any command fails
 
@@ -46,19 +45,18 @@ fi
 
 mysql_opt=" --batch --skip-column-names $@"
 
-schema_version_count=$( mysql $mysql_opt << 'SQL'
+schema_version=$( mysql $mysql_opt <<- 'SQL'
 	CREATE TABLE IF NOT EXISTS `schema_version` (
-		`id` int(11) NOT NULL, -- that is intended, there should only be one row
-		`version` int(11) NOT NULL,
-		`when` datetime NOT NULL,
-		PRIMARY KEY (`id`)
+		`id` int(11) PRIMARY KEY auto_increment,
+		`version` int(11) NOT NULL UNIQUE,
+		`when` datetime NOT NULL
 	) DEFAULT CHARSET=latin1;
 	
-	SELECT COUNT(*) FROM `schema_version`;
+	SELECT MAX(`version`) FROM `schema_version`;
 SQL
 )
 
-if [ $schema_version_count -ne 0 ]; then
+if [ $schema_version != "NULL" ]; then
 	cat >&$STDERR <<- ERROR
 		Error: database schema is not in the right version to do the migration!
 		Expected version: 0 (i.e. the version before there was versioning)
@@ -67,10 +65,10 @@ if [ $schema_version_count -ne 0 ]; then
 fi
 
 
-mysql $mysql_opt << 'SQL'
+mysql $mysql_opt <<- 'SQL'
 	-- CCA agreements and such
 	CREATE TABLE `user_agreements` (
-		`id` int(11) NOT NULL auto_increment,
+		`id` int(11) PRIMARY KEY auto_increment,
 		
 		-- the user that agrees
 		`memid` int(11) NOT NULL,
@@ -93,9 +91,7 @@ mysql $mysql_opt << 'SQL'
 		`method` varchar(100) NOT NULL,
 		
 		-- user comment
-		`comment` varchar(100) DEFAULT NULL,
-		
-		PRIMARY KEY (`id`)
+		`comment` varchar(100) DEFAULT NULL
 	) DEFAULT CHARSET=latin1;
 	
 	
@@ -158,8 +154,8 @@ mysql $mysql_opt << 'SQL'
 	
 	-- Update schema version number
 	INSERT INTO `schema_version`
-		(`id`, `version`, `when`) VALUES
-		('0' , '1'      , NOW() );
+		(`version`, `when`) VALUES
+		('1'      , NOW() );
 SQL
 
 
-- 
cgit v1.2.1