Passed
Pull Request — master (#14)
by Adrien
16:51
created
tests/Acl/MultipleRolesTest.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -19,7 +19,7 @@
 block discarded – undo
19 19
         self::assertSame(['writer'], $roles->getRoles(), 'must be added');
20 20
         self::assertTrue($roles->has('writer'), 'must be existing');
21 21
         self::assertFalse($roles->has('foo'), 'must not be existing');
22
-        self::assertTrue($roles->has('foo', 'writer', ), 'at least one of them');
22
+        self::assertTrue($roles->has('foo', 'writer',), 'at least one of them');
23 23
         self::assertFalse($roles->has('foo', 'bar'), 'none of of them');
24 24
         self::assertFalse($roles->has());
25 25
         self::assertSame('[writer]', (string) $roles);
Please login to merge, or discard this patch.
src/Acl/Acl.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -108,7 +108,7 @@  discard block
 block discarded – undo
108 108
      *
109 109
      * This should be the main method to do all ACL checks.
110 110
      */
111
-    public function isCurrentUserAllowed(Model|string $modelOrResource, string $privilege): bool
111
+    public function isCurrentUserAllowed(Model | string $modelOrResource, string $privilege): bool
112 112
     {
113 113
         $resource = is_string($modelOrResource) ? $modelOrResource : new ModelResource($this->getClass($modelOrResource), $modelOrResource);
114 114
         $role = $this->getCurrentRole();
@@ -141,7 +141,7 @@  discard block
 block discarded – undo
141 141
         return ClassUtils::getRealClass($resource::class);
142 142
     }
143 143
 
144
-    private function getCurrentRole(): MultipleRoles|string
144
+    private function getCurrentRole(): MultipleRoles | string
145 145
     {
146 146
         $user = CurrentUser::get();
147 147
         if (!$user) {
@@ -151,7 +151,7 @@  discard block
 block discarded – undo
151 151
         return $user->getRole();
152 152
     }
153 153
 
154
-    private function buildMessage(ModelResource|string $resource, ?string $privilege, MultipleRoles|string $role, bool $isAllowed): ?string
154
+    private function buildMessage(ModelResource | string $resource, ?string $privilege, MultipleRoles | string $role, bool $isAllowed): ?string
155 155
     {
156 156
         if ($isAllowed) {
157 157
             return null;
@@ -189,7 +189,7 @@  discard block
 block discarded – undo
189 189
     /**
190 190
      * @return array<array{resource:string, privileges: array<int, array{privilege:null|string, allowed: bool, allowIf: string[], denyIf: string[]}>}>
191 191
      */
192
-    public function show(MultipleRoles|string $role, bool $useTranslations = true): array
192
+    public function show(MultipleRoles | string $role, bool $useTranslations = true): array
193 193
     {
194 194
         $result = [];
195 195
         /** @var string[] $resources */
Please login to merge, or discard this patch.