limoncello-php-dist / auth

src/Authorization/PolicyDecision/Algorithms/BasePolicyOrSetAlgorithm.php

Doc Comments +2 added lines, -2 removed lines

@@ -329,7 +329,7 @@ 
     /**
      * @param array $obligations
      * @param int   $evaluation
-     * @param array $obligationsMap
+     * @param callable[] $obligationsMap
      *
      * @return array
      *
@@ -343,7 +343,7 @@ 
     /**
      * @param array $advice
      * @param int   $evaluation
-     * @param array $adviceMap
+     * @param callable[] $adviceMap
      *
      * @return array
      *

Spacing +6 added lines, -7 removed lines

@@ -49,7 +49,7 @@ 
         ContextInterface $context,
         array $policiesAndSetsData,
         ?LoggerInterface $logger
-    ): array {
+    ) : array {
         return static::callAlgorithm(
             static::getCallable($policiesAndSetsData),
             $context,
@@ -110,7 +110,7 @@ 
         int $match,
         array $encodedPolicy,
         ?LoggerInterface $logger
-    ): array {
+    ) : array {
         assert(Encoder::isPolicy($encodedPolicy));
         assert($match !== TargetMatchEnum::NO_TARGET);
 
@@ -178,7 +178,7 @@ 
         int $match,
         array $encodedPolicySet,
         ?LoggerInterface $logger
-    ): array {
+    ) : array {
         assert(Encoder::isPolicySet($encodedPolicySet));
 
         /** @see http://docs.oasis-open.org/xacml/3.0/xacml-3.0-core-spec-os-en.html #7.13 (table 6) */
@@ -242,12 +242,11 @@ 
         int $match,
         array $encodedItem,
         ?LoggerInterface $logger
-    ): array {
+    ) : array {
         $isSet = Encoder::isPolicySet($encodedItem);
 
         return $isSet === true ?
-            static::evaluatePolicySet($context, $match, $encodedItem, $logger) :
-            static::evaluatePolicy($context, $match, $encodedItem, $logger);
+            static::evaluatePolicySet($context, $match, $encodedItem, $logger) : static::evaluatePolicy($context, $match, $encodedItem, $logger);
     }
 
     /**
@@ -258,7 +257,7 @@ 
      *
      * @SuppressWarnings(PHPMD.StaticAccess)
      */
-    private static function correctEvaluationOnIntermediateTarget(int $evaluation, ?LoggerInterface $logger): int
+    private static function correctEvaluationOnIntermediateTarget(int $evaluation, ?LoggerInterface $logger) : int
     {
         /** @see http://docs.oasis-open.org/xacml/3.0/xacml-3.0-core-spec-os-en.html #7.14 (table 7) */
 

src/Authorization/PolicyDecision/Algorithms/Encoder.php

Doc Comments +1 added lines, -1 removed lines

@@ -542,7 +542,7 @@
 
     /**
      * @param RuleCombiningAlgorithmInterface $algorithm
-     * @param array                           $rules
+     * @param RuleInterface[]                           $rules
      *
      * @return array
      */

Spacing +4 added lines, -4 removed lines

@@ -249,7 +249,7 @@ 
      *
      * @return array|null
      */
-    public static function targetAnyOfs(array $encodedTarget): ?array
+    public static function targetAnyOfs(array $encodedTarget): ? array
     {
         assert(static::isTarget($encodedTarget));
 
@@ -285,7 +285,7 @@ 
      *
      * @return array|null
      */
-    public static function ruleEffect(array $encodedRule): ?array
+    public static function ruleEffect(array $encodedRule): ? array
     {
         assert(static::isRule($encodedRule));
 
@@ -297,7 +297,7 @@ 
      *
      * @return callable|null
      */
-    public static function ruleCondition(array $encodedRule): ?callable
+    public static function ruleCondition(array $encodedRule): ? callable
     {
         assert(static::isRule($encodedRule));
 
@@ -503,7 +503,7 @@ 
      *
      * @return callable|null
      */
-    private static function serializeMethod(MethodInterface $method = null): ?callable
+    private static function serializeMethod(MethodInterface $method = null): ? callable
     {
         return $method === null ? null : $method->getCallable();
     }

src/Authorization/PolicyAdministration/Method.php

Spacing +1 added lines, -1 removed lines

@@ -34,7 +34,7 @@
     public function __construct(callable $staticMethod)
     {
         assert(
-            is_array($staticMethod) === true && count((array)$staticMethod) === 2 &&
+            is_array($staticMethod) === true && count((array) $staticMethod) === 2 &&
             is_string($staticMethod[0]) === true && is_string($staticMethod[1]) === true,
             'Only array form of static callable method is supported.'
         );

src/Contracts/Authorization/PolicyAdministration/EvaluationEnum.php

Spacing +1 added lines, -1 removed lines

@@ -27,7 +27,7 @@
     const PERMIT = (1 << 0);
 
     /** Combine result */
-    const DENY =  (1 << 1);
+    const DENY = (1 << 1);
 
     /** Combine result */
     const INDETERMINATE = (1 << 2);

src/Contracts/Authorization/PolicyAdministration/TargetMatchEnum.php

Spacing +2 added lines, -2 removed lines

@@ -27,10 +27,10 @@
     const MATCH = 0;
 
     /** Combine result */
-    const NOT_MATCH =  self::MATCH + 1;
+    const NOT_MATCH = self::MATCH + 1;
 
     /** Combine result */
-    const NO_TARGET =  self::NOT_MATCH + 1;
+    const NO_TARGET = self::NOT_MATCH + 1;
 
     /** Combine result */
     const INDETERMINATE = self::NO_TARGET + 1;

src/Authorization/PolicyDecision/Algorithms/BaseAlgorithm.php

Doc Comments +2 added lines, -2 removed lines

@@ -58,7 +58,7 @@ 
 
     /**
      * @param ContextInterface $context
-     * @param array|null       $serializedLogical
+     * @param null|callable       $serializedLogical
      *
      * @return bool
      */
@@ -365,7 +365,7 @@ 
 
     /**
      * @param array      $storage
-     * @param string|int $key
+     * @param integer $key
      * @param array      $list
      *
      * @return array

Spacing +5 added lines, -5 removed lines

@@ -133,7 +133,7 @@ 
         array $optimizedTargets,
         array $encodedItems,
         ?LoggerInterface $logger
-    ): array {
+    ) : array {
         foreach (static::evaluateTargets($context, $optimizedTargets, $logger) as $match => $itemId) {
             $encodedItem      = $encodedItems[$itemId];
             $packedEvaluation = static::evaluateItem($context, $match, $encodedItem, $logger);
@@ -162,7 +162,7 @@ 
         array $optimizedTargets,
         array $encodedItems,
         ?LoggerInterface $logger
-    ): array {
+    ) : array {
         foreach (static::evaluateTargets($context, $optimizedTargets, $logger) as $match => $itemId) {
             $encodedItem      = $encodedItems[$itemId];
             $packedEvaluation = static::evaluateItem($context, $match, $encodedItem, $logger);
@@ -191,7 +191,7 @@ 
         array $optimizedTargets,
         array $encodedItems,
         ?LoggerInterface $logger
-    ): array {
+    ) : array {
         foreach (static::evaluateTargets($context, $optimizedTargets, $logger) as $match => $itemId) {
             $encodedItem      = $encodedItems[$itemId];
             $packedEvaluation = static::evaluateItem($context, $match, $encodedItem, $logger);
@@ -224,7 +224,7 @@ 
         array $optimizedTargets,
         array $encodedItems,
         ?LoggerInterface $logger
-    ): array {
+    ) : array {
         $foundDeny          = false;
         $foundPermit        = false;
         $foundIntDeny       = false;
@@ -302,7 +302,7 @@ 
         array $optimizedTargets,
         array $encodedItems,
         ?LoggerInterface $logger
-    ): array {
+    ) : array {
         $foundDeny          = false;
         $foundPermit        = false;
         $foundIntDeny       = false;

src/Authorization/PolicyAdministration/Policy.php

Spacing +2 added lines, -2 removed lines

@@ -91,7 +91,7 @@ 
      *
      * @return self
      */
-    public function setName(?string $name): self
+    public function setName(?string $name) : self
     {
         $this->name = $name;
 
@@ -111,7 +111,7 @@ 
      *
      * @return self
      */
-    public function setTarget(?TargetInterface $target): self
+    public function setTarget(?TargetInterface $target) : self
     {
         $this->target = $target;
 

src/Authorization/PolicyAdministration/Rule.php

Spacing +6 added lines, -6 removed lines

@@ -90,7 +90,7 @@ 
      *
      * @return self
      */
-    public function setName(?string $name): self
+    public function setName(?string $name) : self
     {
         $this->name = $name;
 
@@ -110,7 +110,7 @@ 
      *
      * @return self
      */
-    public function setTarget(?TargetInterface $target): self
+    public function setTarget(?TargetInterface $target) : self
     {
         $this->target = $target;
 
@@ -130,7 +130,7 @@ 
      *
      * @return self
      */
-    public function setCondition(?MethodInterface $condition): self
+    public function setCondition(?MethodInterface $condition) : self
     {
         $this->condition = $condition;
 
@@ -150,7 +150,7 @@ 
      *
      * @return self
      */
-    public function setEffect(?MethodInterface $effect): self
+    public function setEffect(?MethodInterface $effect) : self
     {
         $this->effect = $effect;
 
@@ -174,7 +174,7 @@ 
     {
         // check every item is Obligation (debug mode only)
         assert(call_user_func(
-            function () use ($obligations) {
+            function() use ($obligations) {
                 foreach ($obligations as $item) {
                     assert($item instanceof ObligationInterface);
                 }
@@ -204,7 +204,7 @@ 
     {
         // check every item is Obligation (debug mode only)
         assert(call_user_func(
-            function () use ($advice) {
+            function() use ($advice) {
                 foreach ($advice as $item) {
                     assert($item instanceof AdviceInterface);
                 }

src/Authorization/PolicyDecision/Algorithms/RulesDenyOverrides.php

Spacing +1 added lines, -1 removed lines

@@ -42,7 +42,7 @@
         array $optimizedTargets,
         array $serializedRules,
         ?LoggerInterface $logger
-    ): array {
+    ) : array {
         return static::evaluateDenyOverrides($context, $optimizedTargets, $serializedRules, $logger);
     }
 }