# HG changeset patch
# User souliane <souliane@mailoo.org>
# Date 1449007790 -3600
# Node ID e5b569d0c2e7d2a102b0cf8532c63d71a1b675a4
# Parent  7226280e70da1c907fc1548b171a371d936b117c
plugin XEP-0277: don't try to create a new comments node if it already exists

diff -r 7226280e70da -r e5b569d0c2e7 src/plugins/plugin_xep_0277.py
--- a/src/plugins/plugin_xep_0277.py	Tue Dec 01 17:17:09 2015 +0100
+++ b/src/plugins/plugin_xep_0277.py	Tue Dec 01 23:09:50 2015 +0100
@@ -478,24 +478,16 @@
 
         comments_node_base = u"{}{}".format(NS_COMMENT_PREFIX, item_id)
         comments_node = comments_node_base
-
-        suffix = None
         comments_service = client.pubsub_service if client.pubsub_service is not None else service
 
-        # we try to create the comment nodes #
-        max_tries = 3
-
-        for i in xrange(max_tries+1):
-            try:
-                yield self._p.createNode(comments_service, comments_node, options, profile_key=profile)
-                break
-            except error.StanzaError as e:
-                if e.condition == 'conflict' and i<max_tries:
-                    log.warning(u"node {} already exists on service {}".format(comments_node, comments_service))
-                    suffix = 0 if suffix is None else suffix + 1
-                    comments_node = u"{}_{}".format(comments_node_base, suffix)
-                else:
-                    raise e
+        try:
+            yield self._p.createNode(comments_service, comments_node, options, profile_key=profile)
+            break
+        except error.StanzaError as e:
+            if e.condition == 'conflict':
+                log.info(u"node {} already exists on service {}".format(comments_node, comments_service))
+            else:
+                raise e
 
         if comments_service is None:
             comments_service = client.jid.userhostJID()