Apply rector ForeachToArrayAllRector and ForeachToArrayAnyRector #12152

Author: PowerKiKi

src/Acl/Acl.php

@@ -92,14 +92,7 @@ public function isAllowed($role = null, $resource = null, $privilege = null): bo
             $roles = [$role];
         }
 
-        // If at least one role is allowed, then return early
-        foreach ($roles as $role) {
-            if (parent::isAllowed($role, $resource, $privilege)) {
-                return true;
-            }
-        }
-
-        return false;
+        return array_any($roles, fn (null|RoleInterface|string $role) => parent::isAllowed($role, $resource, $privilege));
     }
 
     /**

src/Acl/Assertion/All.php

@@ -31,13 +31,7 @@ public function __construct(NamedAssertion ...$asserts)
      */
     public function assert(Acl $acl, ?RoleInterface $role = null, ?ResourceInterface $resource = null, $privilege = null): bool
     {
-        foreach ($this->asserts as $assert) {
-            if (!$assert->assert($acl, $role, $resource, $privilege)) {
-                return false;
-            }
-        }
-
-        return true;
+        return array_all($this->asserts, fn (NamedAssertion $assert) => $assert->assert($acl, $role, $resource, $privilege));
     }
 
     public function getName(): string

src/Acl/Assertion/One.php

@@ -31,13 +31,7 @@ public function __construct(NamedAssertion ...$asserts)
      */
     public function assert(Acl $acl, ?RoleInterface $role = null, ?ResourceInterface $resource = null, $privilege = null): bool
     {
-        foreach ($this->asserts as $assert) {
-            if ($assert->assert($acl, $role, $resource, $privilege)) {
-                return true;
-            }
-        }
-
-        return false;
+        return array_any($this->asserts, fn (NamedAssertion $assert) => $assert->assert($acl, $role, $resource, $privilege));
     }
 
     public function getName(): string

src/Acl/DebugAcl.php

@@ -172,14 +172,7 @@ public function isAllowed($role = null, $resource = null, $privilege = null): bo
             $roles = [$role];
         }
 
-        // If at least one role is allowed, then return early
-        foreach ($roles as $role) {
-            if (parent::isAllowed($role, $resource, $privilege)) {
-                return true;
-            }
-        }
-
-        return false;
+        return array_any($roles, fn (null|RoleInterface|string $role) => parent::isAllowed($role, $resource, $privilege));
     }
 
     /**

src/DBAL/Types/SetType.php

@@ -64,13 +64,8 @@ private function isValid(mixed $values): bool
         }
 
         $possibleValues = $this->getPossibleValues();
-        foreach ($values as $value) {
-            if (!in_array($value, $possibleValues, true)) {
-                return false;
-            }
-        }
 
-        return true;
+        return array_all($values, fn (mixed $value) => in_array($value, $possibleValues, true));
     }
 
     /**

src/Validator/IPRange.php

@@ -13,13 +13,7 @@ abstract class IPRange
      */
     final public static function matches(string $ip, array $cidrs): bool
     {
-        foreach ($cidrs as $cidr) {
-            if (self::matchesOne($ip, $cidr)) {
-                return true;
-            }
-        }
-
-        return false;
+        return array_any($cidrs, fn (string $cidr) => self::matchesOne($ip, $cidr));
     }
 
     private static function matchesOne(string $ip, string $cidr): bool