[Erp5-report] r17877 - /erp5/trunk/products/ERP5Type/TranslationProviderBase.py

[nobody at svn.erp5.org]

Author: jerome
Date: Wed Nov 28 16:15:32 2007
New Revision: 17877

URL: http://svn.erp5.org?rev=17877&view=rev
Log:
only define _property_domain_dict on the instance, and initialize if needed.


Modified:
    erp5/trunk/products/ERP5Type/TranslationProviderBase.py

Modified: erp5/trunk/products/ERP5Type/TranslationProviderBase.py
URL: http://svn.erp5.org/erp5/trunk/products/ERP5Type/TranslationProviderBase.py?rev=17877&r1=17876&r2=17877&view=diff
==============================================================================
--- erp5/trunk/products/ERP5Type/TranslationProviderBase.py (original)
+++ erp5/trunk/products/ERP5Type/TranslationProviderBase.py Wed Nov 28 16:15:32 2007
@@ -35,7 +35,6 @@
 
   security = ClassSecurityInfo()
   
-  _property_domain_dict = {}
 
   _translation_form = DTMLFile( 'editToolsTranslation', _dtmldir )
 
@@ -91,7 +90,7 @@
     property_domain_keys = property_domain_dict.keys()
     property_domain_keys.sort()
     original_property_domain_keys.sort()
-    # Only update if required in order to prevent ZODB to grow
+    # Only update if required in order to prevent ZODB from growing
     if property_domain_keys != original_property_domain_keys:
       # Update existing dict
       property_domain_dict.update(original_property_domain_dict)
@@ -103,11 +102,13 @@
     """
     Return all the translation defined by a provider.
     """
-    property_domain_dict = getattr(aq_base(self), '_property_domain_dict', _MARKER)
-    if len(property_domain_dict) is _MARKER:
-      # Force update is not defined
+    property_domain_dict = getattr(aq_base(self),
+                                   '_property_domain_dict', _MARKER)
+    # initialize if needed
+    if property_domain_dict is _MARKER:
       self.updateInitialPropertyTranslationDomainDict()
-    return self._property_domain_dict
+      return self._property_domain_dict
+    return property_domain_dict
 
   #
   #   ZMI methods