review corrections

symfony · eltharin · Aug 27, 2024 · Aug 27, 2024 · Aug 27, 2024 · Aug 29, 2024
commit 1caecb5806974c00874f2003b3327bf921ce86a9
@@ -24,11 +24,14 @@
 final class AccessDecision
 {
    /**
-     * @param int    $access One of the VoterInterface::ACCESS_* constants
+     * @param int $access One of the VoterInterface constants (ACCESS_GRANTED, ACCESS_ABSTAIN, ACCESS_DENIED)
     * @param Vote[] $votes
     */
-    public function __construct(private readonly int $access, private readonly array $votes = [], private readonly string $message = '')
-    {
+    public function __construct(
+        private readonly int $access,
+        private readonly array $votes = [],
+        private readonly string $message = '',
+    ) {
    }

    public function getAccess(): int

@@ -29,10 +29,15 @@ class Vote implements VoteInterface
    private array $messages;

    /**
-     * @param int $access One of the VoterInterface::ACCESS_* constants if scoring is false
+     * @param int $access One of the VoterInterface constants (ACCESS_GRANTED, ACCESS_ABSTAIN, ACCESS_DENIED)
+     *                    or an integer when scoring is false
     */
-    public function __construct(private int $access, string|array $messages = [], private array $context = [], private $scoring = false)
-    {
+    public function __construct(
+        private int $access,
+        string|array $messages = [],
+        private array $context = [],
+        private $scoring = false,
+    ) {
        if (!$scoring && !\in_array($access, [VoterInterface::ACCESS_GRANTED, VoterInterface::ACCESS_ABSTAIN, VoterInterface::ACCESS_DENIED], true)) {
            throw new \LogicException(\sprintf('"$access" must return one of "%s" constants ("ACCESS_GRANTED", "ACCESS_DENIED" or "ACCESS_ABSTAIN") when "$scoring" is false, "%s" returned.', VoterInterface::class, $access));
        }

@@ -12,14 +12,12 @@
 namespace Symfony\Component\Security\Core\Authorization\Voter;

 /**
- * A VoteInterface Object contain information about vote, access/score, messages.
+ * A VoteInterface object contain information about vote, access/score, messages.
 *
 * @author Roman JOLY <eltharin18@outlook.fr>
 */
 interface VoteInterface
 {
-    public function __construct(int $access, string|array $messages = [], array $context = []);
-
    public function __debugInfo(): array;

    public function getAccess(): int;

@@ -17,7 +17,7 @@ CHANGELOG
 * Add `$token` argument to `UserCheckerInterface::checkPostAuth()`
 * Deprecate argument `$secret` of `RememberMeToken`
 * Deprecate returning an empty string in `UserInterface::getUserIdentifier()`
- * Add the ability for voter to return decision reason and a score by passing a Vote object
+ * Add the ability for voter to return decision reason and a score by passing a `Vote` object

 7.0
 ---

@@ -69,9 +69,6 @@ public function setAccessDecision(AccessDecision $accessDecision): void
        }
    }

-    /**
-     * Gets the access decision.
-     */
    public function getAccessDecision(): ?AccessDecision
    {
        return $this->accessDecision;
-Original file line number
+Diff line change
@@ -17,7 +17,7 @@ CHANGELOG
      * Add `$token` argument to `UserCheckerInterface::checkPostAuth()`
      * Deprecate argument `$secret` of `RememberMeToken`
      * Deprecate returning an empty string in `UserInterface::getUserIdentifier()`
-     * Add the ability for voter to return decision reason and a score by passing a Vote object
+     * Add the ability for voter to return decision reason and a score by passing a `Vote` object
                   eltharin marked this conversation as resolved.
              

          
            Show resolved
            Hide resolved
        

                
  
            
  

  
    
  
    
      

              Uh oh!

              
There was an error while loading. Please reload this page.
 .0
     ---
-          Expand Down