SVN: CMF/trunk/CMFCore/Skinnable.py Be very careful in __del__, the module level variable may have been Noned already.

"Stefan H. Holek" <[email protected]> Tue, 3 Apr 2007 04:36:46 -0400 (EDT)
Newsgroups gmane.comp.web.zope.cmf.cvs
Message-ID <20070403083646.1D4DB203853__39369.1924437464$1175589417$gmane$org@mail.zope.org>
Log message for revision 73987:
  Be very careful in __del__, the module level variable may have been Noned already.
  

Changed:
  U   CMF/trunk/CMFCore/Skinnable.py

-=-
Modified: CMF/trunk/CMFCore/Skinnable.py
===================================================================
--- CMF/trunk/CMFCore/Skinnable.py	2007-04-03 07:36:59 UTC (rev 73986)
+++ CMF/trunk/CMFCore/Skinnable.py	2007-04-03 08:36:44 UTC (rev 73987)
@@ -58,8 +58,10 @@
         self.tid = tid
     def __del__(self):
         tid = self.tid
-        if SKINDATA.has_key(tid):
-            del SKINDATA[tid]
+        # Be extra careful in __del__
+        if SKINDATA is not None:
+            if SKINDATA.has_key(tid):
+                del SKINDATA[tid]
 
 
 class SkinnableObjectManager(ObjectManager):