[Zope3-checkins] SVN: Zope3/trunk/src/zope/app/component/ Fixed the site deletion bug. Jim please check this fix for

[Stephan Richter]

Log message for revision 67639:
  Fixed the site deletion bug. Jim please check this fix for 
  correctness!!!
  

Changed:
  U   Zope3/trunk/src/zope/app/component/browser/site.txt
  U   Zope3/trunk/src/zope/app/component/site.py

-=-
Modified: Zope3/trunk/src/zope/app/component/browser/site.txt
===================================================================
--- Zope3/trunk/src/zope/app/component/browser/site.txt	2006-04-27 12:17:28 UTC (rev 67638)
+++ Zope3/trunk/src/zope/app/component/browser/site.txt	2006-04-27 12:30:38 UTC (rev 67639)
@@ -35,10 +35,10 @@
     >>> browser.getControl(name='ids:list').getControl(
     ...     value='samplesite').selected = True
 
-#    >>> browser.handleErrors = False
-#    >>> browser.getControl('Delete').click()
-#
-#The site should be gone now.
-#
-#    >>> 'samplesite' in self.contents
-#    False
+    >>> browser.handleErrors = False
+    >>> browser.getControl('Delete').click()
+
+The site should be gone now.
+
+    >>> 'samplesite' in browser.contents
+    False

Modified: Zope3/trunk/src/zope/app/component/site.py
===================================================================
--- Zope3/trunk/src/zope/app/component/site.py	2006-04-27 12:17:28 UTC (rev 67638)
+++ Zope3/trunk/src/zope/app/component/site.py	2006-04-27 12:30:38 UTC (rev 67639)
@@ -406,7 +406,6 @@
 
 def changeSiteConfigurationAfterMove(site, event):
     """After a site is moved, its site manager links have to be updated."""
-    next = None
     if event.newParent is not None:
         next = _findNextSiteManager(site)
-    site.getSiteManager().__bases__ = (next, )
+        site.getSiteManager().__bases__ = (next, )