diff --git a/servatrice/migrations/servatrice_0000_to_0001.sql b/servatrice/migrations/servatrice_0000_to_0001.sql new file mode 100644 index 00000000..a59f9afc --- /dev/null +++ b/servatrice/migrations/servatrice_0000_to_0001.sql @@ -0,0 +1,13 @@ +-- Servatrice db migration from version 0 to version 1 + +-- FIX #153 +CREATE TABLE IF NOT EXISTS `cockatrice_schema_version` ( + `version` int(7) unsigned NOT NULL, + PRIMARY KEY (`version`) +) ENGINE=MyISAM DEFAULT CHARSET=utf8; + +INSERT INTO cockatrice_schema_version VALUES(1); + +-- FIX #1119 +ALTER TABLE `cockatrice_rooms_gametypes` DROP PRIMARY KEY; +ALTER TABLE `cockatrice_rooms_gametypes` ADD KEY (`id_room`); diff --git a/servatrice/servatrice.sql b/servatrice/servatrice.sql index 41efdf25..3dc10831 100644 --- a/servatrice/servatrice.sql +++ b/servatrice/servatrice.sql @@ -11,6 +11,17 @@ SET SQL_MODE="NO_AUTO_VALUE_ON_ZERO"; /*!40101 SET @OLD_COLLATION_CONNECTION=@@COLLATION_CONNECTION */; /*!40101 SET NAMES utf8 */; +-- Every time the database schema changes, the schema version number +-- must be incremented. Also remember to update the corresponding +-- number in servatrice/src/servatrice_database_interface.h + +CREATE TABLE IF NOT EXISTS `cockatrice_schema_version` ( + `version` int(7) unsigned NOT NULL, + PRIMARY KEY (`version`) +) ENGINE=MyISAM DEFAULT CHARSET=utf8; + +INSERT INTO cockatrice_schema_version VALUES(1); + CREATE TABLE IF NOT EXISTS `cockatrice_decklist_files` ( `id` int(7) unsigned zerofill NOT NULL auto_increment, `id_folder` int(7) unsigned zerofill NOT NULL, diff --git a/servatrice/src/servatrice_database_interface.cpp b/servatrice/src/servatrice_database_interface.cpp index b2027709..722ddf99 100644 --- a/servatrice/src/servatrice_database_interface.cpp +++ b/servatrice/src/servatrice_database_interface.cpp @@ -60,6 +60,25 @@ bool Servatrice_DatabaseInterface::openDatabase() return false; } + QSqlQuery *versionQuery = prepareQuery("select version from {prefix}_schema_version limit 1"); + if (!execSqlQuery(versionQuery)) { + qCritical() << QString("[%1] Error opening database: unable to load database schema version (hint: ensure the cockatrice_schema_version exists)").arg(poolStr); + return false; + } + + if (versionQuery->next()) { + const int dbversion = versionQuery->value(0).toInt(); + const int expectedversion = DATABASE_SCHEMA_VERSION; + if(dbversion != expectedversion) + { + qCritical() << QString("[%1] Error opening database: the database schema version is too old, yum need to run the migrations to update it from version %2 to version %3").arg(poolStr).arg(dbversion).arg(expectedversion); + return false; + } + } else { + qCritical() << QString("[%1] Error opening database: unable to load database schema version (hint: ensure the cockatrice_schema_version contains a single record)").arg(poolStr); + return false; + } + // reset all prepared statements qDeleteAll(preparedStatements); preparedStatements.clear(); diff --git a/servatrice/src/servatrice_database_interface.h b/servatrice/src/servatrice_database_interface.h index d7715544..10770691 100644 --- a/servatrice/src/servatrice_database_interface.h +++ b/servatrice/src/servatrice_database_interface.h @@ -9,6 +9,8 @@ #include "server.h" #include "server_database_interface.h" +#define DATABASE_SCHEMA_VERSION 1 + class Servatrice; class Servatrice_DatabaseInterface : public Server_DatabaseInterface {