improveVoteObject

eltharin · eltharin · commit 290b65e0ce3a · 2025-03-30T11:04:48.000+02:00
diff --git a/src/Symfony/Component/Security/Core/Authorization/AccessDecisionManager.php b/src/Symfony/Component/Security/Core/Authorization/AccessDecisionManager.php
@@ -72,6 +72,7 @@ public function decide(TokenInterface $token, array $attributes, mixed $object =
         try {
             return $accessDecision->isGranted = $this->strategy->decide(
                 $this->collectResults($token, $attributes, $object, $accessDecision)
+                , $accessDecision
             );
         } finally {
             array_pop($this->accessDecisionStack);
diff --git a/src/Symfony/Component/Security/Core/Authorization/Strategy/AccessDecisionStrategyInterface.php b/src/Symfony/Component/Security/Core/Authorization/Strategy/AccessDecisionStrategyInterface.php
@@ -11,6 +11,8 @@
 
 namespace Symfony\Component\Security\Core\Authorization\Strategy;
 
+use Symfony\Component\Security\Core\Authorization\AccessDecision;
+
 /**
  * A strategy for turning a stream of votes into a final decision.
  *
@@ -21,5 +23,5 @@ interface AccessDecisionStrategyInterface
     /**
      * @param \Traversable<int> $results
      */
-    public function decide(\Traversable $results): bool;
+    public function decide(\Traversable $results, /* AccessDecision $accessDecision = null */): bool;
 }
diff --git a/src/Symfony/Component/Security/Core/Authorization/Voter/Vote.php b/src/Symfony/Component/Security/Core/Authorization/Voter/Vote.php
@@ -28,6 +28,8 @@ class Vote
      */
     public array $reasons = [];
 
+    public array|object|null $extraData = null;
+
     public function addReason(string $reason): void
     {
         $this->reasons[] = $reason;
diff --git a/src/Symfony/Component/Security/Core/CHANGELOG.md b/src/Symfony/Component/Security/Core/CHANGELOG.md
@@ -11,6 +11,7 @@ CHANGELOG
  * Add ability for voters to explain their vote
  * Add support for voting on closures
  * Add `OAuth2User` with OAuth2 Access Token Introspection support for `OAuth2TokenHandler`
+ * Add extraData in Vote object and pass it to AccessStrategy
 
 7.2
 ---

Original file line number	Diff line number	Diff line change
`@@ -11,6 +11,8 @@`
`11`	`11`
`12`	`12`	`namespace Symfony\Component\Security\Core\Authorization\Strategy;`
`13`	`13`
	`14`	`+use Symfony\Component\Security\Core\Authorization\AccessDecision;`
	`15`	`+`
`14`	`16`	`/**`
`15`	`17`	`* A strategy for turning a stream of votes into a final decision.`
`16`	`18`	`*`
`@@ -21,5 +23,5 @@ interface AccessDecisionStrategyInterface`
`21`	`23`	`/**`
`22`	`24`	`* @param \Traversable<int> $results`
`23`	`25`	`*/`
`24`		`- public function decide(\Traversable $results): bool;`
	`26`	`+ public function decide(\Traversable $results, /* AccessDecision $accessDecision = null */): bool;`
`25`	`27`	`}`
Original file line number	Diff line number	Diff line change
`@@ -28,6 +28,8 @@ class Vote`
`28`	`28`	`*/`
`29`	`29`	`public array $reasons = [];`
`30`	`30`
	`31`	`+ public array\|object\|null $extraData = null;`
	`32`	`+`
`31`	`33`	`public function addReason(string $reason): void`
`32`	`34`	`{`
`33`	`35`	`$this->reasons[] = $reason;`