[Zope-Checkins] SVN: Zope/branches/2.10/ Fixed bug in ZPublisher.BaseRequest with persistent site managers. An EndRequestEvent was thrown after the ZODB connection was already closed and thus the site manager not being available anymore.

[Hanno Schlichting]

Log message for revision 77042:
  Fixed bug in ZPublisher.BaseRequest with persistent site managers. An EndRequestEvent was thrown after the ZODB connection was already closed and thus the site manager not being available anymore.
  

Changed:
  U   Zope/branches/2.10/doc/CHANGES.txt
  U   Zope/branches/2.10/lib/python/ZPublisher/BaseRequest.py

-=-
Modified: Zope/branches/2.10/doc/CHANGES.txt
===================================================================
--- Zope/branches/2.10/doc/CHANGES.txt	2007-06-25 14:07:04 UTC (rev 77041)
+++ Zope/branches/2.10/doc/CHANGES.txt	2007-06-25 14:10:23 UTC (rev 77042)
@@ -8,6 +8,10 @@
 
     Bugs fixed
 
+      - Fixed bug in ZPublisher.BaseRequest with persistent site managers. An
+        EndRequestEvent was thrown after the ZODB connection was already
+        closed and thus the site manager not being available anymore.
+
       - Collector #2295: Comments in PythonScripts could lead to syntax
         errors
 

Modified: Zope/branches/2.10/lib/python/ZPublisher/BaseRequest.py
===================================================================
--- Zope/branches/2.10/lib/python/ZPublisher/BaseRequest.py	2007-06-25 14:07:04 UTC (rev 77041)
+++ Zope/branches/2.10/lib/python/ZPublisher/BaseRequest.py	2007-06-25 14:10:23 UTC (rev 77042)
@@ -202,8 +202,8 @@
 
     def close(self):
         self.other.clear()
+        notify(EndRequestEvent(None, self))
         self._held=None
-        notify(EndRequestEvent(None, self))
 
     def processInputs(self):
         """Do any input processing that could raise errors