Completed
Pull Request — master (#6301)
by Grégoire
05:14 queued 02:31
created
src/Security/Handler/AclSecurityHandlerInterface.php 1 patch
Doc Comments   +4 added lines, -1 removed lines patch added patch discarded remove patch
@@ -25,6 +25,7 @@  discard block
 block discarded – undo
25 25
 {
26 26
     /**
27 27
      * Set the permissions not related to an object instance and also to be available when objects do not exist.
28
+     * @return void
28 29
      */
29 30
     public function setAdminPermissions(array $permissions);
30 31
 
@@ -37,6 +38,7 @@  discard block
 block discarded – undo
37 38
 
38 39
     /**
39 40
      * Set the permissions related to an object instance.
41
+     * @return void
40 42
      */
41 43
     public function setObjectPermissions(array $permissions);
42 44
 
@@ -110,6 +112,7 @@  discard block
 block discarded – undo
110 112
 
111 113
     /**
112 114
      * Delete the ACL.
115
+     * @return void
113 116
      */
114 117
     public function deleteAcl(ObjectIdentityInterface $objectIdentity);
115 118
 
@@ -120,7 +123,7 @@  discard block
 block discarded – undo
120 123
      *
121 124
      * @param string $role
122 125
      *
123
-     * @return array-key|false index if found, FALSE if not found
126
+     * @return integer index if found, FALSE if not found
124 127
      */
125 128
     public function findClassAceIndexByRole(AclInterface $acl, $role);
126 129
 
Please login to merge, or discard this patch.
src/Security/Handler/AclSecurityHandler.php 1 patch
Doc Comments   +1 added lines patch added patch discarded remove patch
@@ -186,6 +186,7 @@
 block discarded – undo
186 186
      * NEXT_MAJOR: change signature to `addObjectOwner(MutableAclInterface $acl, ?UserSecurityIdentity $securityIdentity = null): void`.
187 187
      *
188 188
      * @param MutableAclInterface $acl
189
+     * @param UserSecurityIdentity $securityIdentity
189 190
      */
190 191
     public function addObjectOwner(AclInterface $acl, ?UserSecurityIdentity $securityIdentity = null): void
191 192
     {
Please login to merge, or discard this patch.