libervia-backend: src/plugins/plugin_xep

plugin XEP-0277: accept several <title/> elements, there is a standard issue so for now it's the most flexible behaviour

comparison

equal deleted inserted replaced

-:94c450972346
+:6d6eae490681
 microblog_data['atom_id'] = ""
 else:
 microblog_data['atom_id'] = unicode(id_elt)
 # title/content(s)
-try:
-title_elt = entry_elt.elements(NS_ATOM, 'title').next()
+# FIXME: ATOM and XEP-0277 only allow 1 <title/> element
-except StopIteration:
+#        but in the wild we have some blogs with several ones
+#        so we don't respect the standard for now (it doesn't break
+#        anything anyway), and we'll find a better option later
+# try:
+#     title_elt = entry_elt.elements(NS_ATOM, 'title').next()
+# except StopIteration:
+#     msg = u'No atom title found in the pubsub item {}'.format(id_)
+#     raise failure.Failure(exceptions.DataError(msg))
+title_elts = list(entry_elt.elements(NS_ATOM, 'title'))
+if not title_elts:
 msg = u'No atom title found in the pubsub item {}'.format(id_)
 raise failure.Failure(exceptions.DataError(msg))
+for title_elt in title_elts:
 yield parseElement(title_elt)
+# FIXME: as for <title/>, Atom only authorise at most 1 content
+#        but XEP-0277 allows several ones. So for no we handle as
+#        if more than one can be present
 for content_elt in entry_elt.elements(NS_ATOM, 'content'):
 yield parseElement(content_elt)
 # we check that text content is present
 for key in ('title', 'content'):

Mercurial > libervia-backend