Merge 0.6->0.7
[prosody.git] / core / rostermanager.lua
index 516983a9dfcb934af9f0b8837bb42f7aa41180ea..506cf20593aa7da447d09ca5836517bbda1bab3b 100644 (file)
@@ -1,6 +1,6 @@
 -- Prosody IM
--- Copyright (C) 2008-2009 Matthew Wild
--- Copyright (C) 2008-2009 Waqas Hussain
+-- Copyright (C) 2008-2010 Matthew Wild
+-- Copyright (C) 2008-2010 Waqas Hussain
 -- 
 -- This project is MIT/X11 licensed. Please see the
 -- COPYING file in the source package for more information.
@@ -93,15 +93,18 @@ function load_roster(username, host)
        else -- Attempt to load roster for non-loaded user
                log("debug", "load_roster: loading for offline user: "..username.."@"..host);
        end
-       roster = datamanager.load(username, host, "roster") or {};
+       local data, err = datamanager.load(username, host, "roster");
+       roster = data or {};
        if user then user.roster = roster; end
-       if not roster[false] then roster[false] = { }; end
+       if not roster[false] then roster[false] = { broken = err or nil }; end
        if roster[jid] then
                roster[jid] = nil;
                log("warn", "roster for "..jid.." has a self-contact");
        end
-       hosts[host].events.fire_event("roster-load", username, host, roster);
-       return roster;
+       if not err then
+               hosts[host].events.fire_event("roster-load", username, host, roster);
+       end
+       return roster, err;
 end
 
 function save_roster(username, host, roster)
@@ -114,8 +117,15 @@ function save_roster(username, host, roster)
                --end
        end
        if roster then
-               if not roster[false] then roster[false] = {}; end
-               roster[false].version = (roster[false].version or 0) + 1;
+               local metadata = roster[false];
+               if not metadata then
+                       metadata = {};
+                       roster[false] = metadata;
+               end
+               if metadata.version ~= true then
+                       metadata.version = (metadata.version or 0) + 1;
+               end
+               if roster[false].broken then return nil, "Not saving broken roster" end
                return datamanager.store(username, host, "roster", roster);
        end
        log("warn", "save_roster: user had no roster to save");
@@ -181,9 +191,9 @@ function process_inbound_unsubscribe(username, host, jid)
 end
 
 function is_contact_subscribed(username, host, jid)
-       local roster = load_roster(username, host);
+       local roster, err = load_roster(username, host);
        local item = roster[jid];
-       return item and (item.subscription == "from" or item.subscription == "both");
+       return item and (item.subscription == "from" or item.subscription == "both"), err;
 end
 
 function is_contact_pending_in(username, host, jid)