Turn off strict mode on import (#3784)

.codeclimate.yml

@@ -38,4 +38,4 @@ exclude_patterns:
  - "**/*.module"
  - ".circleci/"
  - ".gitignore"
-
+ 

modules/common/src/Storage/AbstractDatabaseTable.php

@@ -266,9 +266,9 @@ private function sanitizedErrorMessage(string $unsanitizedMessage) {
   }
 
   /**
-   * Private.
+   * Create the table in the db if it does not yet exist.
    */
-  private function setTable() {
+  protected function setTable() {
     if (!$this->tableExist($this->getTableName())) {
       if ($this->schema) {
         $this->tableCreate($this->getTableName(), $this->schema);

modules/datastore/modules/datastore_mysql_import/src/Service/MysqlImport.php

@@ -115,10 +115,10 @@ protected function runIt() {
     $spec = $this->generateTableSpec($columns);
     $this->dataStorage->setSchema(['fields' => $spec]);
 
-    // Call `count` on database table in order to ensure a database table has
-    // been created for the datastore.
     // @todo Find a better way to ensure creation of datastore tables.
+    $this->dataStorage->innodbStrictMode(FALSE);
     $this->dataStorage->count();
+    $this->dataStorage->innodbStrictMode(TRUE);
     // Construct and execute a SQL import statement using the information
     // gathered from the CSV file being imported.
     $this->getDatabaseConnectionCapableOfDataLoad()->query(

modules/datastore/src/Storage/DatabaseTable.php

@@ -6,6 +6,7 @@
 use Dkan\Datastore\Resource;
 use Drupal\common\LoggerTrait;
 use Drupal\common\Storage\AbstractDatabaseTable;
+use Drupal\Core\Database\Driver\mysql\Connection as MysqlConnection;
 
 /**
  * Database storage object.
@@ -158,6 +159,20 @@ public function setSchema($schema) {
     parent::setSchema($schema);
   }
 
+  /**
+   * Disable/enable InnoDB strict mode for the given database connection.
+   *
+   * @param bool $on
+   *   Whether strict mode should be "ON" or "OFF".
+   */
+  public function innodbStrictMode(bool $on) {
+    $value = $on ? "ON" : "OFF";
+    // Only if we're using MySQL.
+    if ($this->connection instanceof MysqlConnection) {
+      $this->connection->query("SET SESSION innodb_strict_mode=:value", [':value' => $value]);
+    }
+  }
+
   /**
    * Get table schema.
    *

modules/datastore/tests/src/Functional/Storage/DatabaseTableTest.php

@@ -0,0 +1,48 @@
+<?php
+
+namespace Drupal\Tests\datastore\Functional\Storage;
+
+use Dkan\Datastore\Resource;
+use Drupal\datastore\Storage\DatabaseTable;
+use Drupal\Tests\common\Traits\CleanUp;
+use Drupal\Tests\common\Traits\GetDataTrait;
+use weitzman\DrupalTestTraits\ExistingSiteBase;
+
+/**
+ * Test ResourcePurger service.
+ *
+ * @package Drupal\Tests\datastore\Functional
+ * @group datastore
+ */
+class DatabaseTableTest extends ExistingSiteBase {
+  use GetDataTrait;
+  use CleanUp;
+
+  /**
+   * Test that setInnodbMode() turns strict mode off for datastore.
+   */
+  public function testSetInnodbMode() {
+    $connection = \Drupal::service('dkan.datastore.database');
+    $result = $connection->query("SHOW SESSION VARIABLES LIKE 'innodb_strict_mode'")->fetchObject();
+    $this->assertEquals($result->Value, "ON");
+
+    $resource = new Resource('123', '/tmp', 'text/csv');
+
+    $databaseTable = new DatabaseTable($connection, $resource);
+    $databaseTable->innodbStrictMode(FALSE);
+
+    $result = $connection->query("SHOW SESSION VARIABLES LIKE 'innodb_strict_mode'")->fetchObject();
+    $this->assertEquals($result->Value, "OFF");
+
+    // Other database connection not affected.
+    $connection2 = \Drupal::service('database');
+    $result = $connection2->query("SHOW SESSION VARIABLES LIKE 'innodb_strict_mode'")->fetchObject();
+    $this->assertEquals($result->Value, "ON");
+
+    // Safe mode can be turned back on.
+    $databaseTable->innodbStrictMode(TRUE);
+    $result = $connection->query("SHOW SESSION VARIABLES LIKE 'innodb_strict_mode'")->fetchObject();
+    $this->assertEquals($result->Value, "ON");
+  }
+
+}