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 +10 added lines, -11 removed lines

@@ -48,8 +48,8 @@ 
     public static function callPolicyAlgorithm(
         ContextInterface $context,
         array $policiesAndSetsData,
-        ?LoggerInterface $logger
-    ): array {
+        ? LoggerInterface $logger
+    ) : array {
         return static::callAlgorithm(
             static::getCallable($policiesAndSetsData),
             $context,
@@ -109,8 +109,8 @@ 
         ContextInterface $context,
         int $match,
         array $encodedPolicy,
-        ?LoggerInterface $logger
-    ): array {
+        ? LoggerInterface $logger
+    ) : array {
         assert(Encoder::isPolicy($encodedPolicy));
         assert($match !== TargetMatchEnum::NO_TARGET);
 
@@ -177,8 +177,8 @@ 
         ContextInterface $context,
         int $match,
         array $encodedPolicySet,
-        ?LoggerInterface $logger
-    ): array {
+        ? LoggerInterface $logger
+    ) : 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) */
@@ -241,13 +241,12 @@ 
         ContextInterface $context,
         int $match,
         array $encodedItem,
-        ?LoggerInterface $logger
-    ): array {
+        ? LoggerInterface $logger
+    ) : 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 +6 added lines, -6 removed lines

@@ -237,7 +237,7 @@ 
      *
      * @return string|null
      */
-    public static function targetName(array $encodedTarget): ?string
+    public static function targetName(array $encodedTarget): ? string
     {
         assert(static::isTarget($encodedTarget));
 
@@ -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));
 
@@ -333,7 +333,7 @@ 
      *
      * @return string|null
      */
-    public static function policyName(array $encodedPolicy): ?string
+    public static function policyName(array $encodedPolicy): ? string
     {
         assert(static::isPolicy($encodedPolicy));
 
@@ -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 +10 added lines, -10 removed lines

@@ -132,8 +132,8 @@ 
         ContextInterface $context,
         array $optimizedTargets,
         array $encodedItems,
-        ?LoggerInterface $logger
-    ): array {
+        ? LoggerInterface $logger
+    ) : array {
         foreach (static::evaluateTargets($context, $optimizedTargets, $logger) as $match => $itemId) {
             $encodedItem      = $encodedItems[$itemId];
             $packedEvaluation = static::evaluateItem($context, $match, $encodedItem, $logger);
@@ -161,8 +161,8 @@ 
         ContextInterface $context,
         array $optimizedTargets,
         array $encodedItems,
-        ?LoggerInterface $logger
-    ): array {
+        ? LoggerInterface $logger
+    ) : array {
         foreach (static::evaluateTargets($context, $optimizedTargets, $logger) as $match => $itemId) {
             $encodedItem      = $encodedItems[$itemId];
             $packedEvaluation = static::evaluateItem($context, $match, $encodedItem, $logger);
@@ -190,8 +190,8 @@ 
         ContextInterface $context,
         array $optimizedTargets,
         array $encodedItems,
-        ?LoggerInterface $logger
-    ): array {
+        ? LoggerInterface $logger
+    ) : array {
         foreach (static::evaluateTargets($context, $optimizedTargets, $logger) as $match => $itemId) {
             $encodedItem      = $encodedItems[$itemId];
             $packedEvaluation = static::evaluateItem($context, $match, $encodedItem, $logger);
@@ -223,8 +223,8 @@ 
         ContextInterface $context,
         array $optimizedTargets,
         array $encodedItems,
-        ?LoggerInterface $logger
-    ): array {
+        ? LoggerInterface $logger
+    ) : array {
         $foundDeny          = false;
         $foundPermit        = false;
         $foundIntDeny       = false;
@@ -301,8 +301,8 @@ 
         ContextInterface $context,
         array $optimizedTargets,
         array $encodedItems,
-        ?LoggerInterface $logger
-    ): array {
+        ? LoggerInterface $logger
+    ) : array {
         $foundDeny          = false;
         $foundPermit        = false;
         $foundIntDeny       = false;

src/Authorization/PolicyAdministration/Policy.php

Spacing +4 added lines, -4 removed lines

@@ -81,7 +81,7 @@ 
     /**
      * @inheritdoc
      */
-    public function getName(): ?string
+    public function getName(): ? string
     {
         return $this->name;
     }
@@ -91,7 +91,7 @@ 
      *
      * @return self
      */
-    public function setName(?string $name): self
+    public function setName(? string $name) : self
     {
         $this->name = $name;
 
@@ -101,7 +101,7 @@ 
     /**
      * @inheritdoc
      */
-    public function getTarget(): ?TargetInterface
+    public function getTarget(): ? TargetInterface
     {
         return $this->target;
     }
@@ -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 +10 added lines, -10 removed lines

@@ -80,7 +80,7 @@ 
     /**
      * @inheritdoc
      */
-    public function getName(): ?string
+    public function getName(): ? string
     {
         return $this->name;
     }
@@ -90,7 +90,7 @@ 
      *
      * @return self
      */
-    public function setName(?string $name): self
+    public function setName(? string $name) : self
     {
         $this->name = $name;
 
@@ -100,7 +100,7 @@ 
     /**
      * @inheritdoc
      */
-    public function getTarget(): ?TargetInterface
+    public function getTarget(): ? TargetInterface
     {
         return $this->target;
     }
@@ -110,7 +110,7 @@ 
      *
      * @return self
      */
-    public function setTarget(?TargetInterface $target): self
+    public function setTarget(? TargetInterface $target) : self
     {
         $this->target = $target;
 
@@ -120,7 +120,7 @@ 
     /**
      * @inheritdoc
      */
-    public function getCondition(): ?MethodInterface
+    public function getCondition(): ? MethodInterface
     {
         return $this->condition;
     }
@@ -130,7 +130,7 @@ 
      *
      * @return self
      */
-    public function setCondition(?MethodInterface $condition): self
+    public function setCondition(? MethodInterface $condition) : self
     {
         $this->condition = $condition;
 
@@ -140,7 +140,7 @@ 
     /**
      * @inheritdoc
      */
-    public function effect(): ?MethodInterface
+    public function effect(): ? MethodInterface
     {
         return $this->effect;
     }
@@ -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/PolicyAdministration/Target.php

Spacing +2 added lines, -2 removed lines

@@ -54,7 +54,7 @@ 
     /**
      * @inheritdoc
      */
-    public function getName(): ?string
+    public function getName(): ? string
     {
         return $this->name;
     }
@@ -64,7 +64,7 @@ 
      *
      * @return $this
      */
-    public function setName(?string $name)
+    public function setName(? string $name)
     {
         $this->name = $name;