]> source.dussan.org Git - nextcloud-server.git/commitdiff
feat(db): remove workaround for pqsql 9.4 and older 45629/head
authorDaniel Kesselberg <mail@danielkesselberg.de>
Sun, 2 Jun 2024 18:16:47 +0000 (20:16 +0200)
committerDaniel Kesselberg <mail@danielkesselberg.de>
Mon, 3 Jun 2024 10:47:31 +0000 (12:47 +0200)
The workaround was required because "ON CONFLICT DO NOTHING" was added with pqsql 9.5.

PostgreSQL 9.4 is eol since 2020-02-13 and thus everyone should use a version with upsert support nowadays.

Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>
lib/private/DB/AdapterPgSql.php

index 847ea3052b7333764aef4c7f7ad677a08ea52ff9..db48c81c2c54c653cbec9a1d09585ffba9a97e66 100644 (file)
@@ -8,7 +8,6 @@
 namespace OC\DB;
 
 class AdapterPgSql extends Adapter {
-       protected $compatModePre9_5 = null;
 
        public function lastInsertId($table) {
                $result = $this->conn->executeQuery('SELECT lastval()');
@@ -25,10 +24,6 @@ class AdapterPgSql extends Adapter {
        }
 
        public function insertIgnoreConflict(string $table, array $values) : int {
-               if ($this->isPre9_5CompatMode() === true) {
-                       return parent::insertIgnoreConflict($table, $values);
-               }
-
                // "upsert" is only available since PgSQL 9.5, but the generic way
                // would leave error logs in the DB.
                $builder = $this->conn->getQueryBuilder();
@@ -39,17 +34,4 @@ class AdapterPgSql extends Adapter {
                $queryString = $builder->getSQL() . ' ON CONFLICT DO NOTHING';
                return $this->conn->executeUpdate($queryString, $builder->getParameters(), $builder->getParameterTypes());
        }
-
-       protected function isPre9_5CompatMode(): bool {
-               if ($this->compatModePre9_5 !== null) {
-                       return $this->compatModePre9_5;
-               }
-
-               $result = $this->conn->executeQuery('SHOW SERVER_VERSION');
-               $version = $result->fetchOne();
-               $result->free();
-               $this->compatModePre9_5 = version_compare($version, '9.5', '<');
-
-               return $this->compatModePre9_5;
-       }
 }