moduleapi: in module:provides(), add the name of the module in item._provided_by

[prosody.git] / plugins / mod_vcard.lua

diff --git a/plugins/mod_vcard.lua b/plugins/mod_vcard.lua
index e2016c97a4b871891fc8cede95e015f645aeec91..26b30e3ab3c10621e9f8fa47a9eb05a4cca8413b 100644 (file)
--- a/plugins/mod_vcard.lua
+++ b/plugins/mod_vcard.lua
@@ -1,77 +1,54 @@
 -- 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.
 --
 
-
-
-local hosts = _G.hosts;
-local datamanager = require "util.datamanager"
-
 local st = require "util.stanza"
-local t_concat, t_insert = table.concat, table.insert;
-
-local jid = require "util.jid"
-local jid_split = jid.split;
-
-local xmlns_vcard = "vcard-temp";
-
-module:add_feature(xmlns_vcard);
-
-function handle_vcard(session, stanza)
-       if stanza.tags[1].name == "vCard" then
-               local to = stanza.attr.to;
-               if stanza.attr.type == "get" then
-                       local vCard;
-                       if to then
-                               local node, host = jid_split(to);
-                               if hosts[host] and hosts[host].type == "local" then
-                                       vCard = st.deserialize(datamanager.load(node, host, "vcard")); -- load vCard for user or server
-                               end
-                       else
-                               vCard = st.deserialize(datamanager.load(session.username, session.host, "vcard"));-- load user's own vCard
-                       end
-                       if vCard then
-                               session.send(st.reply(stanza):add_child(vCard)); -- send vCard!
-                       else
-                               session.send(st.error_reply(stanza, "cancel", "item-not-found"));
-                       end
-               elseif stanza.attr.type == "set" then
-                       if not to or to == session.username.."@"..session.host then
-                               if datamanager.store(session.username, session.host, "vcard", st.preserialize(stanza.tags[1])) then
-                                       session.send(st.reply(stanza));
-                               else
-                                       -- TODO unable to write file, file may be locked, etc, what's the correct error?
-                                       session.send(st.error_reply(stanza, "wait", "internal-server-error"));
-                               end
+local jid_split = require "util.jid".split;
+
+local vcards = module:open_store();
+
+module:add_feature("vcard-temp");
+
+local function handle_vcard(event)
+       local session, stanza = event.origin, event.stanza;
+       local to = stanza.attr.to;
+       if stanza.attr.type == "get" then
+               local vCard;
+               if to then
+                       local node, host = jid_split(to);
+                       vCard = st.deserialize(vcards:get(node)); -- load vCard for user or server
+               else
+                       vCard = st.deserialize(vcards:get(session.username));-- load user's own vCard
+               end
+               if vCard then
+                       session.send(st.reply(stanza):add_child(vCard)); -- send vCard!
+               else
+                       session.send(st.error_reply(stanza, "cancel", "item-not-found"));
+               end
+       else
+               if not to then
+                       if vcards:set(session.username, st.preserialize(stanza.tags[1])) then
+                               session.send(st.reply(stanza));
                        else
-                               session.send(st.error_reply(stanza, "auth", "forbidden"));
+                               -- TODO unable to write file, file may be locked, etc, what's the correct error?
+                               session.send(st.error_reply(stanza, "wait", "internal-server-error"));
                        end
+               else
+                       session.send(st.error_reply(stanza, "auth", "forbidden"));
                end
-               return true;
        end
+       return true;
 end
 
-module:add_iq_handler({"c2s", "s2sin", "component"}, xmlns_vcard, handle_vcard);
+module:hook("iq/bare/vcard-temp:vCard", handle_vcard);
+module:hook("iq/host/vcard-temp:vCard", handle_vcard);
 
--- COMPAT: https://support.process-one.net/browse/EJAB-1045
-if module:get_option("vcard_compatibility") then
-       module:hook("iq/full", function (data)
-               local stanza = data.stanza;
-               if stanza.attr.type == "get" and stanza.tags[1]
-                       and stanza.tags[1].attr.xmlns == xmlns_vcard then
-                               return handle_vcard(data.origin, stanza);
-               end
-       end, 1);
+-- COMPAT w/0.8
+if module:get_option("vcard_compatibility") ~= nil then
+       module:log("error", "The vcard_compatibility option has been removed, see"..
+               "mod_compat_vcard in prosody-modules if you still need this.");
 end
-
-local feature_vcard_attr = { var=xmlns_vcard };
-module:add_event_hook("stream-features",
-                                       function (session, features)
-                                               if session.type == "c2s" then
-                                                       features:tag("feature", feature_vcard_attr):up();
-                                               end
-                                       end);