From 0ae41c3afffc44acec09b2fe5a49d95a41bc9dda Mon Sep 17 00:00:00 2001 From: Andreas Fischer Date: Thu, 31 Oct 2013 15:44:19 +0100 Subject: [PATCH] Get rid of the SELECT query. Try UPDATE, on failure INSERT. --- lib/ocs/privatedata.php | 12 ++++-------- 1 file changed, 4 insertions(+), 8 deletions(-) diff --git a/lib/ocs/privatedata.php b/lib/ocs/privatedata.php index 2c53d576092..932413711b8 100644 --- a/lib/ocs/privatedata.php +++ b/lib/ocs/privatedata.php @@ -69,18 +69,14 @@ class OC_OCS_Privatedata { $key = addslashes(strip_tags($parameters['key'])); $value = OC_OCS::readData('post', 'value', 'text'); - // check if key is already set - $query = \OCP\DB::prepare('SELECT `value` FROM `*PREFIX*privatedata` WHERE `user` = ? AND `app` = ? AND `key` = ? '); - $result = $query->execute(array($user, $app, $key)); + // update in DB + $query = \OCP\DB::prepare('UPDATE `*PREFIX*privatedata` SET `value` = ? WHERE `user` = ? AND `app` = ? AND `key` = ?'); + $numRows = $query->execute(array($value, $user, $app, $key)); - if ($result->numRows()==0) { + if ($numRows === false || $numRows === 0) { // store in DB $query = \OCP\DB::prepare('INSERT INTO `*PREFIX*privatedata` (`user`, `app`, `key`, `value`)' . ' VALUES(?, ?, ?, ?)'); $query->execute(array($user, $app, $key, $value)); - } else { - // update in DB - $query = \OCP\DB::prepare('UPDATE `*PREFIX*privatedata` SET `value` = ? WHERE `user` = ? AND `app` = ? AND `key` = ? '); - $query->execute(array($value, $user, $app, $key )); } return new OC_OCS_Result(null, 100); -- 2.39.5