Skip to content

Commit

Permalink
Merge pull request #45629 from nextcloud/enh/noid/remove-workaround-p…
Browse files Browse the repository at this point in the history
…qsql-94

feat(db): remove workaround for pqsql 9.4 and older
  • Loading branch information
kesselb authored Jun 3, 2024
2 parents 88fe734 + cac6443 commit b4467bf
Showing 1 changed file with 0 additions and 18 deletions.
18 changes: 0 additions & 18 deletions lib/private/DB/AdapterPgSql.php
Original file line number Diff line number Diff line change
Expand Up @@ -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()');
Expand All @@ -25,10 +24,6 @@ public function fixupStatement($statement) {
}

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();
Expand All @@ -39,17 +34,4 @@ public function insertIgnoreConflict(string $table, array $values) : int {
$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;
}
}

0 comments on commit b4467bf

Please sign in to comment.