rofl

eltharin · eltharin · commit b513d95ba9ea · 2025-09-23T16:38:58.000+02:00
diff --git a/src/Symfony/Component/Security/Core/Authorization/Strategy/AccessDecisionStrategyInterface.php b/src/Symfony/Component/Security/Core/Authorization/Strategy/AccessDecisionStrategyInterface.php
@@ -24,5 +24,5 @@ interface AccessDecisionStrategyInterface
      * @param \Traversable<int> $results
      * @param ?AccessDecision   $accessDecision
      */
-    public function decide(\Traversable $results/* , AccessDecision $accessDecision = null */): bool;
+    public function decide(\Traversable $results/* , ?AccessDecision $accessDecision = null */): bool;
 }
diff --git a/src/Symfony/Component/Security/Core/Authorization/Strategy/AffirmativeStrategy.php b/src/Symfony/Component/Security/Core/Authorization/Strategy/AffirmativeStrategy.php
@@ -30,7 +30,7 @@ public function __construct(
     ) {
     }
 
-    public function decide(\Traversable $results, ?AccessDecision $accessDecision): bool
+    public function decide(\Traversable $results, ?AccessDecision $accessDecision = null): bool
     {
         $deny = 0;
         foreach ($results as $result) {
diff --git a/src/Symfony/Component/Security/Core/Authorization/Strategy/ConsensusStrategy.php b/src/Symfony/Component/Security/Core/Authorization/Strategy/ConsensusStrategy.php
@@ -39,7 +39,7 @@ public function __construct(
     ) {
     }
 
-    public function decide(\Traversable $results, ?AccessDecision $accessDecision): bool
+    public function decide(\Traversable $results, ?AccessDecision $accessDecision = null): bool
     {
         $grant = 0;
         $deny = 0;
diff --git a/src/Symfony/Component/Security/Core/Authorization/Strategy/PriorityStrategy.php b/src/Symfony/Component/Security/Core/Authorization/Strategy/PriorityStrategy.php
@@ -31,7 +31,7 @@ public function __construct(
     ) {
     }
 
-    public function decide(\Traversable $results, ?AccessDecision $accessDecision): bool
+    public function decide(\Traversable $results, ?AccessDecision $accessDecision = null): bool
     {
         foreach ($results as $result) {
             if (VoterInterface::ACCESS_GRANTED === $result) {
diff --git a/src/Symfony/Component/Security/Core/Authorization/Strategy/UnanimousStrategy.php b/src/Symfony/Component/Security/Core/Authorization/Strategy/UnanimousStrategy.php
@@ -30,7 +30,7 @@ public function __construct(
     ) {
     }
 
-    public function decide(\Traversable $results, ?AccessDecision $accessDecision): bool
+    public function decide(\Traversable $results, ?AccessDecision $accessDecision = null): bool
     {
         $grant = 0;
         foreach ($results as $result) {
diff --git a/src/Symfony/Component/Security/Core/Tests/Authorization/AccessDecisionManagerTest.php b/src/Symfony/Component/Security/Core/Tests/Authorization/AccessDecisionManagerTest.php
@@ -41,7 +41,7 @@ public function testVoterCalls()
         ];
 
         $strategy = new class implements AccessDecisionStrategyInterface {
-            public function decide(\Traversable $results, ?AccessDecision $accessDecision): bool
+            public function decide(\Traversable $results, ?AccessDecision $accessDecision = null): bool
             {
                 $i = 0;
                 foreach ($results as $result) {

Original file line number	Diff line number	Diff line change
`@@ -24,5 +24,5 @@ interface AccessDecisionStrategyInterface`
`24`	`24`	`* @param \Traversable<int> $results`
`25`	`25`	`* @param ?AccessDecision $accessDecision`
`26`	`26`	`*/`
`27`		`- public function decide(\Traversable $results/* , AccessDecision $accessDecision = null */): bool;`
	`27`	`+ public function decide(\Traversable $results/* , ?AccessDecision $accessDecision = null */): bool;`
`28`	`28`	`}`
Original file line number	Diff line number	Diff line change
`@@ -30,7 +30,7 @@ public function __construct(`
`30`	`30`	`) {`
`31`	`31`	`}`
`32`	`32`
`33`		`- public function decide(\Traversable $results, ?AccessDecision $accessDecision): bool`
	`33`	`+ public function decide(\Traversable $results, ?AccessDecision $accessDecision = null): bool`
`34`	`34`	`{`
`35`	`35`	`$deny = 0;`
`36`	`36`	`foreach ($results as $result) {`
Original file line number	Diff line number	Diff line change
`@@ -39,7 +39,7 @@ public function __construct(`
`39`	`39`	`) {`
`40`	`40`	`}`
`41`	`41`
`42`		`- public function decide(\Traversable $results, ?AccessDecision $accessDecision): bool`
	`42`	`+ public function decide(\Traversable $results, ?AccessDecision $accessDecision = null): bool`
`43`	`43`	`{`
`44`	`44`	`$grant = 0;`
`45`	`45`	`$deny = 0;`
Original file line number	Diff line number	Diff line change
`@@ -31,7 +31,7 @@ public function __construct(`
`31`	`31`	`) {`
`32`	`32`	`}`
`33`	`33`
`34`		`- public function decide(\Traversable $results, ?AccessDecision $accessDecision): bool`
	`34`	`+ public function decide(\Traversable $results, ?AccessDecision $accessDecision = null): bool`
`35`	`35`	`{`
`36`	`36`	`foreach ($results as $result) {`
`37`	`37`	`if (VoterInterface::ACCESS_GRANTED === $result) {`
Original file line number	Diff line number	Diff line change
`@@ -41,7 +41,7 @@ public function testVoterCalls()`
`41`	`41`	`];`
`42`	`42`
`43`	`43`	`$strategy = new class implements AccessDecisionStrategyInterface {`
`44`		`- public function decide(\Traversable $results, ?AccessDecision $accessDecision): bool`
	`44`	`+ public function decide(\Traversable $results, ?AccessDecision $accessDecision = null): bool`
`45`	`45`	`{`
`46`	`46`	`$i = 0;`
`47`	`47`	`foreach ($results as $result) {`