Changeset

6349:0cee68dd35f8

mod_storage_sql2: DELETE then INSERT in map stores
author Kim Alvefur <zash@zash.se>
date Tue, 12 Aug 2014 11:38:12 +0200 (2014-08-12)
parents 6348:bffc885dc378
children 6354:bbb4a82db32e
files plugins/mod_storage_sql2.lua
diffstat 1 files changed, 8 insertions(+), 13 deletions(-) [+]
line wrap: on
line diff
--- a/plugins/mod_storage_sql2.lua	Mon Aug 11 11:36:30 2014 +0200
+++ b/plugins/mod_storage_sql2.lua	Tue Aug 12 11:38:12 2014 +0200
@@ -221,13 +221,8 @@
 function map_store:get(username, key)
 	local ok, result = engine:transaction(function()
 		if type(key) == "string" and key ~= "" then
-			local iter, state, first = engine:select("SELECT * FROM `prosody` WHERE `host`=? AND `user`=? AND `store`=? AND `key`=?",
-				host, username, self.store, key or "");
-			local row = iter(state, first);
-			if row then
-				return deserialize(row.type, row.value);
-			else
-				return nil;
+			for row in engine:select("SELECT `type`, `value` FROM `prosody` WHERE `host`=? AND `user`=? AND `store`=? AND `key`=?", host, username or "", self.store, key) do
+				return deserialize(row[1], row[2]);
 			end
 		else
 			error("TODO: non-string keys");
@@ -238,13 +233,13 @@
 end
 function map_store:set(username, key, data)
 	local ok, result = engine:transaction(function()
-		if data == nil then
+		if type(key) == "string" and key ~= "" then
 			engine:delete("DELETE FROM `prosody` WHERE `host`=? AND `user`=? AND `store`=? AND `key`=?",
-				host, username, self.store, key or "");
-		elseif type(key) == "string" and key ~= "" then
-			local t, value = assert(serialize(data));
-			engine:update("UPDATE `prosody` SET `type`=?, `value`=? WHERE `host`=? AND `user`=? AND `store`=? AND `key`=?",
-				t, value, host, username, self.store, key);
+				host, username or "", self.store, key);
+			if data ~= nil then
+				local t, value = assert(serialize(data));
+				engine:insert("INSERT INTO `prosody` (`host`,`user`,`store`,`key`,`type`,`value`) VALUES (?,?,?,?,?,?)", host, username or "", self.store, key, t, value);
+			end
 		else
 			error("TODO: non-string keys");
 		end