limoncello-php-dist / auth

src/Authorization/PolicyAdministration/PolicySet.php

Doc Comments +1 added lines, -1 removed lines

@@ -60,7 +60,7 @@
     private $advice;
 
     /**
-     * @param PolicyInterface[]|PolicySetInterface[] $policiesAndSets
+     * @param PolicyInterface[] $policiesAndSets
      * @param PolicyCombiningAlgorithmInterface      $combiningAlgorithm
      * @param null|string                            $name
      * @param TargetInterface                        $target

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 callable|null       $serializedLogical
      *
      * @return bool
      */
@@ -365,7 +365,7 @@ 
 
     /**
      * @param array      $storage
-     * @param string|int $key
+     * @param integer $key
      * @param array      $list
      *
      * @return array

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 +1 added lines, -2 removed lines

@@ -246,8 +246,7 @@
         $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);
     }
 
     /**

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
      */

tests/Authorization/PolicyDecision/PolicyDecisionTest.php

Doc Comments +1 added lines, -1 removed lines

@@ -355,7 +355,7 @@
      * @param bool $addTargetAll
      * @param bool $exInCondition
      *
-     * @return RuleInterface[]
+     * @return Rule[]
      */
     private function createRules($addTargetAll = true, $exInCondition = false)
     {

Spacing +3 added lines, -3 removed lines

@@ -197,7 +197,7 @@ 
 
         $exceptionThrown = false;
         $context         = new Context(new Request([]), [
-            'key11_1' => function () use (&$exceptionThrown) {
+            'key11_1' => function() use (&$exceptionThrown) {
                 $exceptionThrown = true;
                 throw new RuntimeException();
             },
@@ -377,7 +377,7 @@ 
             'key22_1' => 'value22_1',
             'key22_2' => 'value22_2',
         ]);
-        $allOf3  = new AllOf([
+        $allOf3 = new AllOf([
             'key31' => 'value31',
         ]);
 
@@ -395,7 +395,7 @@ 
             ->setCondition(new Logical([self::class, $methodName]))
             ->setObligations([new Obligation(EvaluationEnum::PERMIT, [self::class, 'ruleObligation1'])])
             ->setAdvice([new Advice(EvaluationEnum::PERMIT, [self::class, 'ruleAdvice1'])]);
-        $rule2      = (new Rule())
+        $rule2 = (new Rule())
             ->setEffect(new Logical([self::class, 'effectDeny']))
             ->setTarget($target2)
             ->setCondition(new Logical([self::class, $methodName]))

tests/Authorization/PolicyEnforcement/PolicyEnforcementTest.php

Spacing +1 added lines, -1 removed lines

@@ -515,7 +515,7 @@
             ContextProperties::PARAM_CURRENT_USER_ROLE => $this->currentUserRole,
             ContextProperties::PARAM_USER_IS_SIGNED_IN => $this->currentUserId !== null &&
                 $this->currentUserRole !== null,
-            ContextProperties::PARAM_IS_WORK_TIME      => function () {
+            ContextProperties::PARAM_IS_WORK_TIME      => function() {
                 return $this->isWorkTime;
             },
         ];

src/Authorization/PolicyAdministration/Rule.php

Spacing +2 added lines, -2 removed lines

@@ -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/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);