[Erp5-report] r35822 nicolas - /erp5/trunk/products/ERP5SyncML/Conduit/ERP5Conduit.py

[nobody at svn.erp5.org]

Author: nicolas
Date: Mon May 31 18:45:57 2010
New Revision: 35822

URL: http://svn.erp5.org?rev=35822&view=rev
Log:
Add some comments

Modified:
    erp5/trunk/products/ERP5SyncML/Conduit/ERP5Conduit.py

Modified: erp5/trunk/products/ERP5SyncML/Conduit/ERP5Conduit.py
URL: http://svn.erp5.org/erp5/trunk/products/ERP5SyncML/Conduit/ERP5Conduit.py?rev=35822&r1=35821&r2=35822&view=diff
==============================================================================
--- erp5/trunk/products/ERP5SyncML/Conduit/ERP5Conduit.py [utf8] (original)
+++ erp5/trunk/products/ERP5SyncML/Conduit/ERP5Conduit.py [utf8] Mon May 31 18:45:57 2010
@@ -707,6 +707,8 @@
 
   security.declareProtected(Permissions.AccessContentsInformation,'afterNewObject')
   def afterNewObject(self, object):
+    """Overloadable method
+    """
     pass
 
   security.declareProtected(Permissions.AccessContentsInformation,'getStatusFromXml')
@@ -734,6 +736,8 @@
   def getElementFromXupdate(self, xml):
     """
     return a fragment node with applied xupdate
+    This method simulate an xupdate transformation on given XML.
+    It transform the xupdate into node handleable by Conduit
     """
     if xml.xpath('name()') in self.XUPDATE_ELEMENT:
       new_node = Element(xml.get('name'), nsmap=xml.nsmap)
@@ -746,6 +750,8 @@
       new_node.tail = xml.tail
       return new_node
     if xml.xpath('name()') in (self.XUPDATE_UPDATE + self.XUPDATE_DEL):
+      # This condition seems not used anymore and not efficient
+      # Usage of xupdate_processor is recommanded
       result = u'<'
       attribute = xml.attrib.get('select')
       s = '[@id='