- found a bug, it caused MSSQL not being able to use templates stored in the DB since the first beta. removing pointless auto_increment

git-svn-id: file:///svn/phpbb/trunk@7990 89ea8834-ac86-4346-8a33-228a782c2dd0

phpBB/develop/create_schema_files.php

@@ -1590,7 +1590,7 @@ function get_schema_struct()
 
 	$schema_data['phpbb_styles_template_data'] = array(
 		'COLUMNS'		=> array(
-			'template_id'			=> array('USINT', NULL, 'auto_increment'),
+			'template_id'			=> array('USINT', 0),
 			'template_filename'		=> array('VCHAR:100', ''),
 			'template_included'		=> array('TEXT', ''),
 			'template_mtime'		=> array('TIMESTAMP', 0),

phpBB/install/database_update.php

@@ -401,7 +401,7 @@ $database_update_info = array(
 				'template_id'		=> array('USINT', NULL, 'auto_increment'),
 			),
 			STYLES_TEMPLATE_DATA_TABLE	=> array(
-				'template_id'		=> array('USINT', NULL, 'auto_increment'),
+				'template_id'		=> array('USINT', 0),
 			),
 			STYLES_THEME_TABLE			=> array(
 				'theme_id'			=> array('USINT', NULL, 'auto_increment'),
@@ -1297,7 +1297,6 @@ if (version_compare($current_version, '3.0.RC4', '<='))
 	$update_auto_increment = array(
 		STYLES_TABLE				=> 'style_id',
 		STYLES_TEMPLATE_TABLE		=> 'template_id',
-		STYLES_TEMPLATE_DATA_TABLE	=> 'template_id',
 		STYLES_THEME_TABLE			=> 'theme_id',
 		STYLES_IMAGESET_TABLE		=> 'imageset_id'
 	);
@@ -1413,15 +1412,6 @@ if (version_compare($current_version, '3.0.RC4', '<='))
 
 		$no_updates = false;
 	}
-	else if ($map_dbms == 'sqlite')
-	{
-		foreach ($update_auto_increment as $auto_table_name => $auto_column_name)
-		{
-			sql_column_change($dbms, $auto_table_name, $auto_column_name, array('USINT', NULL, 'auto_increment'));
-		}
-
-		$no_updates = false;
-	}
 	else if ($map_dbms == 'postgres')
 	{
 		foreach ($update_auto_increment as $auto_table_name => $auto_column_name)
@@ -1429,6 +1419,69 @@ if (version_compare($current_version, '3.0.RC4', '<='))
 			$sql = "SELECT SETVAL('" . $auto_table_name . "_seq',(select case when max({$auto_column_name})>0 then max({$auto_column_name})+1 else 1 end from " . $auto_table_name . '));';
 			_sql($sql, $errored, $error_ary);
 		}
+
+		$sql = 'DROP SEQUENCE ' . STYLES_TEMPLATE_DATA_TABLE . '_seq';
+		_sql($sql, $errored, $error_ary);
+	}
+	else if ($map_dbms == 'firebird')
+	{
+		$sql = 'DROP TRIGGER t_' . STYLES_TEMPLATE_DATA_TABLE;
+		_sql($sql, $errored, $error_ary);
+
+		$sql = 'DROP GENERATOR ' . STYLES_TEMPLATE_DATA_TABLE . '_gen';
+		_sql($sql, $errored, $error_ary);
+	}
+	else if ($map_dbms == 'oracle')
+	{
+		$sql = 'DROP TRIGGER t_' . STYLES_TEMPLATE_DATA_TABLE;
+		_sql($sql, $errored, $error_ary);
+
+		$sql = 'DROP SEQUENCE ' . STYLES_TEMPLATE_DATA_TABLE . '_seq';
+		_sql($sql, $errored, $error_ary);
+	}
+	else if ($map_dbms == 'mssql')
+	{
+		// we use transactions because we need to have a working DB at the end of all of this
+		$db->sql_transaction('begin');
+
+		$sql = 'SELECT *
+			FROM ' . STYLES_TEMPLATE_TABLE;
+		$result = _sql($sql, $errored, $error_ary);
+		$old_style_rows = array();
+		while ($row = $db->sql_fetchrow($result))
+		{
+			$old_style_rows[] = $row;
+		}
+		$db->sql_freeresult($result);
+
+		// death to the table, it is evil!
+		$sql = 'DROP TABLE ' . STYLES_TEMPLATE_DATA_TABLE;
+		_sql($sql, $errored, $error_ary);
+
+		// the table of awesomeness, praise be to it (or something)
+		$sql = 'CREATE TABLE [' . STYLES_TEMPLATE_DATA_TABLE . "] (
+			[template_id] [int] DEFAULT (0) NOT NULL ,
+			[template_filename] [varchar] (100) DEFAULT ('') NOT NULL ,
+			[template_included] [varchar] (8000) DEFAULT ('') NOT NULL ,
+			[template_mtime] [int] DEFAULT (0) NOT NULL ,
+			[template_data] [text] DEFAULT ('') NOT NULL 
+		) ON [PRIMARY] TEXTIMAGE_ON [PRIMARY]";
+		_sql($sql, $errored, $error_ary);
+
+		// index? index
+		$sql = 'CREATE  INDEX [tid] ON [' . STYLES_TEMPLATE_DATA_TABLE . ']([template_id]) ON [PRIMARY]';
+		_sql($sql, $errored, $error_ary);
+
+		// yet another index
+		$sql = 'CREATE  INDEX [tfn] ON [' . STYLES_TEMPLATE_DATA_TABLE . ']([template_filename]) ON [PRIMARY]';
+		_sql($sql, $errored, $error_ary);
+
+		foreach ($old_style_rows as $return_row)
+		{
+			_sql('INSERT INTO ' . STYLES_TEMPLATE_DATA_TABLE . '(' . implode(', ', array_keys($return_row)) . ') VALUES (' . implode(', ', $return_row) . ')');
+		}
+
+		$db->sql_transaction('commit');
 	}
 }
 
@@ -1882,7 +1935,11 @@ function prepare_column_data($dbms, $column_data, $table_name, $column_name)
 
 			// In Oracle empty strings ('') are treated as NULL.
 			// Therefore in oracle we allow NULL's for all DEFAULT '' entries
-			$sql .= ($column_data[1] === '') ? '' : 'NOT NULL';
+			// Oracle does not like setting NOT NULL on a column that is already NOT NULL (this happens only on number fields)
+			if (preg_match('/number/i', $column_type))
+			{
+				$sql .= ($column_data[1] === '') ? '' : 'NOT NULL';
+			}
 		break;
 
 		case 'postgres':

phpBB/install/schemas/firebird_schema.sql

@@ -1093,7 +1093,7 @@ END;;
 
 # Table: 'phpbb_styles_template_data'
 CREATE TABLE phpbb_styles_template_data (
-	template_id INTEGER NOT NULL,
+	template_id INTEGER DEFAULT 0 NOT NULL,
 	template_filename VARCHAR(100) CHARACTER SET NONE DEFAULT '' NOT NULL,
 	template_included BLOB SUB_TYPE TEXT CHARACTER SET NONE DEFAULT '' NOT NULL,
 	template_mtime INTEGER DEFAULT 0 NOT NULL,
@@ -1103,17 +1103,6 @@ CREATE TABLE phpbb_styles_template_data (
 CREATE INDEX phpbb_styles_template_data_tid ON phpbb_styles_template_data(template_id);;
 CREATE INDEX phpbb_styles_template_data_tfn ON phpbb_styles_template_data(template_filename);;
 
-CREATE GENERATOR phpbb_styles_template_data_gen;;
-SET GENERATOR phpbb_styles_template_data_gen TO 0;;
-
-CREATE TRIGGER t_phpbb_styles_template_data FOR phpbb_styles_template_data
-BEFORE INSERT
-AS
-BEGIN
-	NEW.template_id = GEN_ID(phpbb_styles_template_data_gen, 1);
-END;;
-
-
 # Table: 'phpbb_styles_theme'
 CREATE TABLE phpbb_styles_theme (
 	theme_id INTEGER NOT NULL,

phpBB/install/schemas/mssql_schema.sql

@@ -1309,7 +1309,7 @@ GO
 	Table: 'phpbb_styles_template_data'
 */
 CREATE TABLE [phpbb_styles_template_data] (
-	[template_id] [int] IDENTITY (1, 1) NOT NULL ,
+	[template_id] [int] DEFAULT (0) NOT NULL ,
 	[template_filename] [varchar] (100) DEFAULT ('') NOT NULL ,
 	[template_included] [varchar] (8000) DEFAULT ('') NOT NULL ,
 	[template_mtime] [int] DEFAULT (0) NOT NULL ,

phpBB/install/schemas/mysql_40_schema.sql

@@ -743,7 +743,7 @@ CREATE TABLE phpbb_styles_template (
 
 # Table: 'phpbb_styles_template_data'
 CREATE TABLE phpbb_styles_template_data (
-	template_id smallint(4) UNSIGNED NOT NULL auto_increment,
+	template_id smallint(4) UNSIGNED DEFAULT '0' NOT NULL,
 	template_filename varbinary(100) DEFAULT '' NOT NULL,
 	template_included blob NOT NULL,
 	template_mtime int(11) UNSIGNED DEFAULT '0' NOT NULL,

phpBB/install/schemas/mysql_41_schema.sql

@@ -743,7 +743,7 @@ CREATE TABLE phpbb_styles_template (
 
 # Table: 'phpbb_styles_template_data'
 CREATE TABLE phpbb_styles_template_data (
-	template_id smallint(4) UNSIGNED NOT NULL auto_increment,
+	template_id smallint(4) UNSIGNED DEFAULT '0' NOT NULL,
 	template_filename varchar(100) DEFAULT '' NOT NULL,
 	template_included text NOT NULL,
 	template_mtime int(11) UNSIGNED DEFAULT '0' NOT NULL,

phpBB/install/schemas/oracle_schema.sql

@@ -1453,7 +1453,7 @@ END;
 	Table: 'phpbb_styles_template_data'
 */
 CREATE TABLE phpbb_styles_template_data (
-	template_id number(4) NOT NULL,
+	template_id number(4) DEFAULT '0' NOT NULL,
 	template_filename varchar2(100) DEFAULT '' ,
 	template_included clob DEFAULT '' ,
 	template_mtime number(11) DEFAULT '0' NOT NULL,
@@ -1466,22 +1466,6 @@ CREATE INDEX phpbb_styles_template_data_tid ON phpbb_styles_template_data (templ
 CREATE INDEX phpbb_styles_template_data_tfn ON phpbb_styles_template_data (template_filename)
 /
 
-CREATE SEQUENCE phpbb_styles_template_data_seq
-/
-
-CREATE OR REPLACE TRIGGER t_phpbb_styles_template_data
-BEFORE INSERT ON phpbb_styles_template_data
-FOR EACH ROW WHEN (
-	new.template_id IS NULL OR new.template_id = 0
-)
-BEGIN
-	SELECT phpbb_styles_template_data_seq.nextval
-	INTO :new.template_id
-	FROM dual;
-END;
-/
-
-
 /*
 	Table: 'phpbb_styles_theme'
 */

phpBB/install/schemas/postgres_schema.sql

@@ -977,10 +977,8 @@ CREATE UNIQUE INDEX phpbb_styles_template_tmplte_nm ON phpbb_styles_template (te
 /*
 	Table: 'phpbb_styles_template_data'
 */
-CREATE SEQUENCE phpbb_styles_template_data_seq;
-
 CREATE TABLE phpbb_styles_template_data (
-	template_id INT2 DEFAULT nextval('phpbb_styles_template_data_seq'),
+	template_id INT2 DEFAULT '0' NOT NULL CHECK (template_id >= 0),
 	template_filename varchar(100) DEFAULT '' NOT NULL,
 	template_included varchar(8000) DEFAULT '' NOT NULL,
 	template_mtime INT4 DEFAULT '0' NOT NULL CHECK (template_mtime >= 0),

phpBB/install/schemas/sqlite_schema.sql

@@ -717,7 +717,7 @@ CREATE UNIQUE INDEX phpbb_styles_template_tmplte_nm ON phpbb_styles_template (te
 
 # Table: 'phpbb_styles_template_data'
 CREATE TABLE phpbb_styles_template_data (
-	template_id INTEGER PRIMARY KEY NOT NULL ,
+	template_id INTEGER UNSIGNED NOT NULL DEFAULT '0',
 	template_filename varchar(100) NOT NULL DEFAULT '',
 	template_included text(65535) NOT NULL DEFAULT '',
 	template_mtime INTEGER UNSIGNED NOT NULL DEFAULT '0',