Merge branch 'm33_MDL-59596_DDL_Issues_With_MariaDB_10p2_SQLMode' of h…

…ttps://github.com/scara/moodle into MOODLE_33_STABLE
moodle · May 28, 2018 · 149552615cdb9f21bb4a46adfdef357711566bb0 · 1495526
2 parents 33ef4d9 + 23c1612
commit 149552615cdb9f21bb4a46adfdef357711566bb0
Unified Split

Showing with 35 additions and 0 deletions.

+21 −0 lib/ddl/mysql_sql_generator.php

+6 −0 lib/dml/mariadb_native_moodle_database.php

+8 −0 lib/dml/mysqli_native_moodle_database.php
diff --git a/lib/ddl/mysql_sql_generator.php b/lib/ddl/mysql_sql_generator.php
@@ -321,6 +321,27 @@ public function getAddFieldSQL($xmldb_table, $xmldb_field, $skip_type_clause = N
        return $sqls;
    }

+    public function getAlterFieldSQL($xmldb_table, $xmldb_field, $skip_type_clause = NULL, $skip_default_clause = NULL, $skip_notnull_clause = NULL)
+    {
+        $tablename = $xmldb_table->getName();
+        $dbcolumnsinfo = $this->mdb->get_columns($tablename);
+
+        if (($this->mdb->has_breaking_change_sqlmode()) &&
+            ($dbcolumnsinfo[$xmldb_field->getName()]->meta_type == 'X') &&
+            ($xmldb_field->getType() == XMLDB_TYPE_INTEGER)) {
+            // Ignore 1292 ER_TRUNCATED_WRONG_VALUE Truncated incorrect INTEGER value: '%s'.
+            $altercolumnsqlorig = $this->alter_column_sql;
+            $this->alter_column_sql = str_replace('ALTER TABLE', 'ALTER IGNORE TABLE', $this->alter_column_sql);
+            $result = parent::getAlterFieldSQL($xmldb_table, $xmldb_field, $skip_type_clause, $skip_default_clause, $skip_notnull_clause);
+            // Restore the original ALTER SQL statement pattern.
+            $this->alter_column_sql = $altercolumnsqlorig;
+
+            return $result;
+        }
+
+        return parent::getAlterFieldSQL($xmldb_table, $xmldb_field, $skip_type_clause, $skip_default_clause, $skip_notnull_clause);
+    }
+
    /**
     * Given one correct xmldb_table, returns the SQL statements
     * to create temporary table (inside one array).
diff --git a/lib/dml/mariadb_native_moodle_database.php b/lib/dml/mariadb_native_moodle_database.php
@@ -94,6 +94,12 @@ protected function has_breaking_change_quoted_defaults() {
        return version_compare($version, '10.2.7', '>=');
    }

+    public function has_breaking_change_sqlmode() {
+        $version = $this->get_server_info()['version'];
+        // Breaking change since 10.2.4: https://mariadb.com/kb/en/the-mariadb-library/sql-mode/#setting-sql_mode.
+        return version_compare($version, '10.2.4', '>=');
+    }
+
    /**
     * It is time to require transactions everywhere.
     *
diff --git a/lib/dml/mysqli_native_moodle_database.php b/lib/dml/mysqli_native_moodle_database.php
@@ -817,6 +817,14 @@ protected function has_breaking_change_quoted_defaults() {
        return false;
    }

+    /**
+     * Indicates whether SQL_MODE default value has changed in a not backward compatible way.
+     * @return boolean True when SQL_MODE breaks BC; otherwise, false.
+     */
+    public function has_breaking_change_sqlmode() {
+        return false;
+    }
+
    /**
     * Returns moodle column info for raw column from information schema.
     * @param stdClass $rawcolumn