Merge pull request #14923 from owncloud/fix-preference-key-definition

Fix user preference key definition for insertIfNotExists()

lib/private/allconfig.php

@@ -196,7 +196,7 @@ class AllConfig implements \OCP\IConfig {
 				'userid'		=> $userId,
 				'appid'			=> $appName,
 				'configkey'		=> $key,
-			], ['configvalue', 'userid', 'appid']);
+			], ['configkey', 'userid', 'appid']);
 			$affectedRows = 1;
 		} elseif ($exists) {
 			$data = array($value, $userId, $appName, $key);

lib/private/appframework/db/db.php

@@ -127,6 +127,7 @@ class Db implements IDb {
 	 * @param array $input data that should be inserted into the table  (column name => value)
 	 * @param array|null $compare List of values that should be checked for "if not exists"
 	 *				If this is null or an empty array, all keys of $input will be compared
+	 *				Please note: text fields (clob) must not be used in the compare array
 	 * @return int number of inserted rows
 	 * @throws \Doctrine\DBAL\DBALException
 	 */

lib/private/db/adapter.php

@@ -46,6 +46,7 @@ class Adapter {
 	 * @param array $input data that should be inserted into the table  (column name => value)
 	 * @param array|null $compare List of values that should be checked for "if not exists"
 	 *				If this is null or an empty array, all keys of $input will be compared
+	 *				Please note: text fields (clob) must not be used in the compare array
 	 * @return int number of inserted rows
 	 * @throws \Doctrine\DBAL\DBALException
 	 */

lib/private/db/adaptersqlite.php

@@ -25,6 +25,7 @@ class AdapterSqlite extends Adapter {
 	 * @param array $input data that should be inserted into the table  (column name => value)
 	 * @param array|null $compare List of values that should be checked for "if not exists"
 	 *				If this is null or an empty array, all keys of $input will be compared
+	 *				Please note: text fields (clob) must not be used in the compare array
 	 * @return int number of inserted rows
 	 * @throws \Doctrine\DBAL\DBALException
 	 */

lib/private/db/connection.php

@@ -163,6 +163,7 @@ class Connection extends \Doctrine\DBAL\Connection implements IDBConnection {
 	 * @param array $input data that should be inserted into the table  (column name => value)
 	 * @param array|null $compare List of values that should be checked for "if not exists"
 	 *				If this is null or an empty array, all keys of $input will be compared
+	 *				Please note: text fields (clob) must not be used in the compare array
 	 * @return int number of inserted rows
 	 * @throws \Doctrine\DBAL\DBALException
 	 */

lib/public/idbconnection.php

@@ -83,6 +83,7 @@ interface IDBConnection {
 	 * @param array $input data that should be inserted into the table  (column name => value)
 	 * @param array|null $compare List of values that should be checked for "if not exists"
 	 *				If this is null or an empty array, all keys of $input will be compared
+	 *				Please note: text fields (clob) must not be used in the compare array
 	 * @return int number of inserted rows
 	 * @throws \Doctrine\DBAL\DBALException
 	 */