mbk-lab
/
nextcloud
2
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Fix updates of NULL appconfig values 

The comparisson of NULL is a bit special.
So we need to handle this a tad beter else it might not replace NULL
values. or allow you to set NULL values on updates.

Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
This commit is contained in:
Roeland Jago Douma 2020-10-21 10:04:06 +02:00 committed by backportbot[bot]
parent e6b0df1986
commit af8ed35a1b
1 changed files with 22 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

31
lib/private/AppConfig.php
View File

@ -201,12 +201,9 @@ class AppConfig implements IAppConfig {
$sql = $this->conn->getQueryBuilder(); $sql = $this->conn->getQueryBuilder();
$sql->update('appconfig') $sql->update('appconfig')
->set('configvalue', $sql->createParameter('configvalue')) ->set('configvalue', $sql->createNamedParameter($value))
->where($sql->expr()->eq('appid', $sql->createParameter('app'))) ->where($sql->expr()->eq('appid', $sql->createNamedParameter($app)))
->andWhere($sql->expr()->eq('configkey', $sql->createParameter('configkey'))) ->andWhere($sql->expr()->eq('configkey', $sql->createNamedParameter($key)));
->setParameter('configvalue', $value)
->setParameter('app', $app)
->setParameter('configkey', $key);
/* /*
* Only limit to the existing value for non-Oracle DBs: * Only limit to the existing value for non-Oracle DBs:
@ -214,9 +211,25 @@ class AppConfig implements IAppConfig {
* > Large objects (LOBs) are not supported in comparison conditions. * > Large objects (LOBs) are not supported in comparison conditions.
*/ */
if (!($this->conn instanceof OracleConnection)) { if (!($this->conn instanceof OracleConnection)) {
// Only update the value when it is not the same
$sql->andWhere($sql->expr()->neq('configvalue', $sql->createParameter('configvalue'))) /*
->setParameter('configvalue', $value); * Only update the value when it is not the same
* Note that NULL requires some special handling. Since comparing
* against null can have special results.
*/
if ($value === null) {
$sql->andWhere(
$sql->expr()->isNotNull('configvalue')
);
} else {
$sql->andWhere(
$sql->expr()->orX(
$sql->expr()->isNull('configvalue'),
$sql->expr()->neq('configvalue', $sql->createNamedParameter($value))
)
);
}
} }
$changedRow = (bool) $sql->execute(); $changedRow = (bool) $sql->execute();