[Zope3-checkins] SVN: Zope3/branches/ZopeX3-3.0/ Merged from trunk:

[Jim Fulton]

Log message for revision 28210:
  Merged from trunk:
  
    27663 (Thu Sep 23 17:47:53 2004 UTC (3 weeks, 4 days ago)):
  
      A small bug fix reported by Gustavo.
  
  This is actually a critical fix.
  

Changed:
  U   Zope3/branches/ZopeX3-3.0/doc/CHANGES.txt
  U   Zope3/branches/ZopeX3-3.0/src/zope/app/securitypolicy/securitymap.py

-=-
Modified: Zope3/branches/ZopeX3-3.0/doc/CHANGES.txt
===================================================================
--- Zope3/branches/ZopeX3-3.0/doc/CHANGES.txt	2004-10-18 10:01:20 UTC (rev 28209)
+++ Zope3/branches/ZopeX3-3.0/doc/CHANGES.txt	2004-10-18 18:03:16 UTC (rev 28210)
@@ -25,7 +25,8 @@
       - Factory IDs have to be real IDs now, i.e. either a dotted
         name or a URI.
 
-    Restructuring
+      - Small but important fix to security map. It was not marking objects
+        with modified permissions as changed, due to a missing assignment.
 
     Much thanks to everyone who contributed to this release:
 

Modified: Zope3/branches/ZopeX3-3.0/src/zope/app/securitypolicy/securitymap.py
===================================================================
--- Zope3/branches/ZopeX3-3.0/src/zope/app/securitypolicy/securitymap.py	2004-10-18 10:01:20 UTC (rev 28209)
+++ Zope3/branches/ZopeX3-3.0/src/zope/app/securitypolicy/securitymap.py	2004-10-18 18:03:16 UTC (rev 28210)
@@ -141,7 +141,7 @@
     def _changed(self):
         map = self.map
         if isinstance(map, PersistentSecurityMap):
-            map._p_changed
+            map._p_changed = 1
         else:
             map = PersistentSecurityMap()
             map._byrow = self._byrow