Merge pull request #7210 from nextcloud/pg10

add postgresql 10 compatibility to dbal

4 files changed, 0 insertions, 100 deletions

diff --git a/lib/composer/composer/autoload_classmap.php b/lib/composer/composer/autoload_classmap.php
index 93c139b7bbe..81dad9890b5 100644
--- a/lib/composer/composer/autoload_classmap.php
+++ b/lib/composer/composer/autoload_classmap.php
@@ -540,7 +540,6 @@ return array(
     'OC\\DB\\Migrator' => $baseDir . '/lib/private/DB/Migrator.php',
     'OC\\DB\\MySQLMigrator' => $baseDir . '/lib/private/DB/MySQLMigrator.php',
     'OC\\DB\\MySqlTools' => $baseDir . '/lib/private/DB/MySqlTools.php',
-    'OC\\DB\\OCPostgreSqlPlatform' => $baseDir . '/lib/private/DB/OCPostgreSqlPlatform.php',
     'OC\\DB\\OCSqlitePlatform' => $baseDir . '/lib/private/DB/OCSqlitePlatform.php',
     'OC\\DB\\OracleConnection' => $baseDir . '/lib/private/DB/OracleConnection.php',
     'OC\\DB\\OracleMigrator' => $baseDir . '/lib/private/DB/OracleMigrator.php',
diff --git a/lib/composer/composer/autoload_static.php b/lib/composer/composer/autoload_static.php
index c073d43ca14..b926100365b 100644
--- a/lib/composer/composer/autoload_static.php
+++ b/lib/composer/composer/autoload_static.php
@@ -570,7 +570,6 @@ class ComposerStaticInit53792487c5a8370acc0b06b1a864ff4c
         'OC\\DB\\Migrator' => __DIR__ . '/../../..' . '/lib/private/DB/Migrator.php',
         'OC\\DB\\MySQLMigrator' => __DIR__ . '/../../..' . '/lib/private/DB/MySQLMigrator.php',
         'OC\\DB\\MySqlTools' => __DIR__ . '/../../..' . '/lib/private/DB/MySqlTools.php',
-        'OC\\DB\\OCPostgreSqlPlatform' => __DIR__ . '/../../..' . '/lib/private/DB/OCPostgreSqlPlatform.php',
         'OC\\DB\\OCSqlitePlatform' => __DIR__ . '/../../..' . '/lib/private/DB/OCSqlitePlatform.php',
         'OC\\DB\\OracleConnection' => __DIR__ . '/../../..' . '/lib/private/DB/OracleConnection.php',
         'OC\\DB\\OracleMigrator' => __DIR__ . '/../../..' . '/lib/private/DB/OracleMigrator.php',
diff --git a/lib/private/DB/ConnectionFactory.php b/lib/private/DB/ConnectionFactory.php
index 082e81dfa74..fcb0117a0db 100644
--- a/lib/private/DB/ConnectionFactory.php
+++ b/lib/private/DB/ConnectionFactory.php
@@ -139,9 +139,6 @@ class ConnectionFactory {
 				unset($additionalConnectionParams['host']);
 				break;
 
-			case 'pgsql':
-				$additionalConnectionParams['platform'] = new OCPostgreSqlPlatform();
-				break;
 			case 'sqlite3':
 				$journalMode = $additionalConnectionParams['sqlite.journal_mode'];
 				$additionalConnectionParams['platform'] = new OCSqlitePlatform();
diff --git a/lib/private/DB/OCPostgreSqlPlatform.php b/lib/private/DB/OCPostgreSqlPlatform.php
deleted file mode 100644
index e66ab84252a..00000000000
--- a/lib/private/DB/OCPostgreSqlPlatform.php
+++ /dev/null
@@ -1,95 +0,0 @@
-<?php
-/**
- * @copyright Copyright (c) 2017, ownCloud GmbH
- *
- * @author Joas Schilling <coding@schilljs.com>
- *
- * @license AGPL-3.0
- *
- * This code is free software: you can redistribute it and/or modify
- * it under the terms of the GNU Affero General Public License, version 3,
- * as published by the Free Software Foundation.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
- * GNU Affero General Public License for more details.
- *
- * You should have received a copy of the GNU Affero General Public License, version 3,
- * along with this program.  If not, see <http://www.gnu.org/licenses/>
- *
- */
-
-namespace OC\DB;
-
-use Doctrine\DBAL\Platforms\PostgreSqlPlatform;
-use Doctrine\DBAL\Schema\ColumnDiff;
-use Doctrine\DBAL\Schema\TableDiff;
-use Doctrine\DBAL\Types\Type;
-
-class OCPostgreSqlPlatform extends PostgreSqlPlatform {
-
-	/**
-	 * {@inheritDoc}
-	 */
-	public function getAlterTableSQL(TableDiff $diff){
-		$queries = parent::getAlterTableSQL($diff);
-		foreach ($queries as $index => $sql){
-			// BIGSERIAL could not be used in statements altering column type
-			// That's why we replace it with BIGINT 
-			// see https://github.com/owncloud/core/pull/28364#issuecomment-315006853
-			if (preg_match('|(ALTER TABLE\s+\S+\s+ALTER\s+\S+\s+TYPE\s+)(BIGSERIAL)|i', $sql, $matches)) {
-				$alterTable = $matches[1];
-				$queries[$index] = $alterTable . 'BIGINT';
-			}
-
-			// Changing integer to bigint kills next autoincrement value
-			// see https://github.com/owncloud/core/pull/28364#issuecomment-315006853
-			if (preg_match('|ALTER TABLE\s+(\S+)\s+ALTER\s+(\S+)\s+DROP DEFAULT|i', $sql, $matches)) {
-				$queryColumnName = $matches[2];
-				$columnDiff = $this->findColumnDiffByName($diff, $queryColumnName);
-				if ($columnDiff && $this->shouldSkipDropDefault($columnDiff)) {
-					unset($queries[$index]);
-					continue;
-				}
-			}
-		}
-		
-		return $queries;
-	}
-
-	/**
-	 * We should NOT drop next sequence value if
-	 * - type was changed from INTEGER to BIGINT
-	 * - column keeps an autoincrement
-	 * - default value is kept NULL
-	 *
-	 * @param ColumnDiff $columnDiff
-	 * @return bool
-	 */
-	private function shouldSkipDropDefault(ColumnDiff $columnDiff) {
-		$column = $columnDiff->column;
-		$fromColumn = $columnDiff->fromColumn;
-		return $fromColumn->getType()->getName() === Type::INTEGER
-				&& $column->getType()->getName() === Type::BIGINT
-				&& $fromColumn->getDefault() === null
-				&& $column->getDefault() === null
-				&& $fromColumn->getAutoincrement()
-				&& $column->getAutoincrement();
-	}
-
-	/**
-	 * @param TableDiff $diff
-	 * @param string $name
-	 * @return  ColumnDiff | false
-	 */
-	private function findColumnDiffByName(TableDiff $diff, $name) {
-		foreach ($diff->changedColumns as $columnDiff) {
-			$oldColumnName = $columnDiff->getOldColumnName()->getQuotedName($this);
-			if ($oldColumnName === $name) {
-				return $columnDiff;
-			}
-		}
-		return false;
-	}
-}