Merge pull request #101 from zopefoundation/dataflake/issue_95

Fix method signature of ``PluggableAuthService._setObject``

CHANGES.rst

@@ -4,6 +4,10 @@ Change Log
 2.6.4 (unreleased)
 ------------------
 
+- Fix method signature of ``PluggableAuthService._setObject``
+  (`#95
+  <https://github.com/zopefoundation/Products.PluggableAuthService/issues/95>`_)
+
 
 2.6.3 (2021-06-08)
 ------------------

src/Products/PluggableAuthService/PluggableAuthService.py

@@ -422,7 +422,8 @@ def __creatable_by_emergency_user__(self):
         return 1
 
     @security.private
-    def _setObject(self, id, object, roles=None, user=None, set_owner=0):
+    def _setObject(self, id, object, roles=None, user=None, set_owner=0,
+                   suppress_events=False):
         #
         #   Override ObjectManager's version to change the default for
         #   'set_owner' (we don't want to enforce ownership on contained

src/Products/PluggableAuthService/tests/test_PluggableAuthService.py

@@ -2149,6 +2149,14 @@ def test__setObject_no_ownership_fixup(self):
 
         zcuf._setObject('sub', sub)
 
+    def test__setObject_arguments(self):
+        # Make sure the overridden definition matches the
+        # superclass definition
+        from OFS.Folder import Folder
+        zcuf = self._makeOne()
+        sub = Folder('sub')
+        zcuf._setObject('sub', sub, set_owner=False, suppress_events=True)
+
     def test__delOb_unregisters_plugin(self):
 
         from ..interfaces.plugins import IAuthenticationPlugin