PEP notification bugfix

SVN Revision: 1085

diff --git a/ChangeLog b/ChangeLog
index bc29c9e097174c5f4661b99ba55ff40a5c94d306..4ee67440e8ad25ef723f6ea9003e0c82240f85e7 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -4,6 +4,8 @@
        ejabberdctl usage example (EJAB-420)
        * doc/guide.html: Likewise
 
+       * src/mod_pubsub/mod_pubsub.erl: PEP notification bugfix
+
 2007-12-20  Badlop  <badlop@process-one.net>
 
        * src/msgs/zh.msg: Small update (thanks to Shelley Shyan)

diff --git a/src/mod_pubsub/mod_pubsub.erl b/src/mod_pubsub/mod_pubsub.erl
index 2ce785f7c14abd601d7d5280c02b138423a46533..194bc920aaec447e9fcfaed6cf6706dced724237 100644 (file)
--- a/src/mod_pubsub/mod_pubsub.erl
+++ b/src/mod_pubsub/mod_pubsub.erl
@@ -2341,13 +2341,15 @@ broadcast_by_caps({LUser, LServer, LResource}, Node, _Type, Stanza) ->
                _ ->
                    ok
            end,
-           %% also send a notification to any
-           %% of the account owner's available resources.
-           %% See: XEP-0163 1.1 section 3
-           lists:foreach(fun(Resource) ->
-                                 To = jlib:make_jid(LUser, LServer, Resource),
-                                 ejabberd_router:route(Sender, To, Stanza)
-                         end, SenderResources),
+       %% also send a notification to any
+       %% of the account owner's available resources.
+       %% See: XEP-0163 1.1 section 3
+       %% Note: ejabberd_c2s:get_subscribed_and_online already returns owner online resources
+       %%       this loop should be deleted
+       %    lists:foreach(fun(Resource) ->
+       %                         To = jlib:make_jid(LUser, LServer, Resource),
+       %                         ejabberd_router:route(Sender, To, Stanza)
+       %                 end, SenderResources),
            ok;
        _ ->
            ok