SpareParts / Overseer

src/VotingDecisionEnum.php

Doc Comments +1 added lines, -1 removed lines

@@ -39,7 +39,7 @@
 
 
     /**
-     * @param $string
+     * @param string $string
      * @return self
      */
     private static function instance($string)

Spacing +4 added lines, -4 removed lines

@@ -12,7 +12,7 @@ 
     /**
      * @var self[]
      */
-    static private $registry = [];
+    static private $registry = [ ];
 
     private function __construct($value)
     {
@@ -53,10 +53,10 @@ 
      */
     private static function instance($string)
     {
-        if (!isset(static::$registry[$string])) {
-            static::$registry[$string] = new static($string);
+        if (!isset(static::$registry[ $string ])) {
+            static::$registry[ $string ] = new static($string);
         }
-        return static::$registry[$string];
+        return static::$registry[ $string ];
     }
 
 

Indentation +54 added lines, -54 removed lines

@@ -4,58 +4,58 @@
 
 final class VotingDecisionEnum
 {
-    /**
-     * @var string
-     */
-    private $value;
-
-    /**
-     * @var self[]
-     */
-    static private $registry = [];
-
-    private function __construct($value)
-    {
-        $this->value = $value;
-    }
-
-
-    /**
-     * @return self
-     */
-    public static function ALLOWED()
-    {
-        return static::instance('allowed');
-    }
-
-
-    /**
-     * @return self
-     */
-    public static function DENIED()
-    {
-        return static::instance('denied');
-    }
-
-
-    /**
-     * @param $string
-     * @return self
-     */
-    private static function instance($string)
-    {
-        if (!isset(static::$registry[$string])) {
-            static::$registry[$string] = new static($string);
-        }
-        return static::$registry[$string];
-    }
-
-
-    /**
-     * @return string
-     */
-    public function __toString()
-    {
-        return $this->value;
-    }
+	/**
+	 * @var string
+	 */
+	private $value;
+
+	/**
+	 * @var self[]
+	 */
+	static private $registry = [];
+
+	private function __construct($value)
+	{
+		$this->value = $value;
+	}
+
+
+	/**
+	 * @return self
+	 */
+	public static function ALLOWED()
+	{
+		return static::instance('allowed');
+	}
+
+
+	/**
+	 * @return self
+	 */
+	public static function DENIED()
+	{
+		return static::instance('denied');
+	}
+
+
+	/**
+	 * @param $string
+	 * @return self
+	 */
+	private static function instance($string)
+	{
+		if (!isset(static::$registry[$string])) {
+			static::$registry[$string] = new static($string);
+		}
+		return static::$registry[$string];
+	}
+
+
+	/**
+	 * @return string
+	 */
+	public function __toString()
+	{
+		return $this->value;
+	}
 }

src/StrategyEnum.php

Indentation +51 added lines, -51 removed lines

@@ -4,67 +4,67 @@
 
 class StrategyEnum
 {
-    /**
-     * @var string
-     */
-    private $value;
+	/**
+	 * @var string
+	 */
+	private $value;
 
-    /**
-     * @var self[]
-     */
-    static private $registry = [];
+	/**
+	 * @var self[]
+	 */
+	static private $registry = [];
 
-    private function __construct($value)
-    {
-        $this->value = $value;
-    }
+	private function __construct($value)
+	{
+		$this->value = $value;
+	}
 
 
-    /**
-     * @return self
-     */
-    public static function FIRST_VOTE_DECIDES()
-    {
-        return static::instance('first_vote_decides');
-    }
+	/**
+	 * @return self
+	 */
+	public static function FIRST_VOTE_DECIDES()
+	{
+		return static::instance('first_vote_decides');
+	}
 
 
-    /**
-     * @return self
-     */
-    public static function ALLOW_UNLESS_DENIED()
-    {
-        return static::instance('allow_unless_denied');
-    }
+	/**
+	 * @return self
+	 */
+	public static function ALLOW_UNLESS_DENIED()
+	{
+		return static::instance('allow_unless_denied');
+	}
 
 
-    /**
-     * @return self
-     */
-    public static function DENY_UNLESS_ALLOWED()
-    {
-        return static::instance('deny_unless_allowed');
-    }
+	/**
+	 * @return self
+	 */
+	public static function DENY_UNLESS_ALLOWED()
+	{
+		return static::instance('deny_unless_allowed');
+	}
 
 
-    /**
-     * @param $string
-     * @return self
-     */
-    private static function instance($string)
-    {
-        if (!isset(static::$registry[$string])) {
-            static::$registry[$string] = new static($string);
-        }
-        return static::$registry[$string];
-    }
+	/**
+	 * @param $string
+	 * @return self
+	 */
+	private static function instance($string)
+	{
+		if (!isset(static::$registry[$string])) {
+			static::$registry[$string] = new static($string);
+		}
+		return static::$registry[$string];
+	}
 
 
-    /**
-     * @return string
-     */
-    public function __toString()
-    {
-        return $this->value;
-    }
+	/**
+	 * @return string
+	 */
+	public function __toString()
+	{
+		return $this->value;
+	}
 }

Spacing +4 added lines, -4 removed lines

@@ -12,7 +12,7 @@ 
     /**
      * @var self[]
      */
-    static private $registry = [];
+    static private $registry = [ ];
 
     private function __construct($value)
     {
@@ -53,10 +53,10 @@ 
      */
     private static function instance($string)
     {
-        if (!isset(static::$registry[$string])) {
-            static::$registry[$string] = new static($string);
+        if (!isset(static::$registry[ $string ])) {
+            static::$registry[ $string ] = new static($string);
         }
-        return static::$registry[$string];
+        return static::$registry[ $string ];
     }
 
 

Doc Comments +1 added lines, -1 removed lines

@@ -39,7 +39,7 @@
 
 
     /**
-     * @param $string
+     * @param string $string
      * @return self
      */
     private static function instance($string)

src/VotingResult.php

Indentation +38 added lines, -38 removed lines

@@ -5,43 +5,43 @@
 
 class VotingResult implements IVotingResult
 {
-    /**
-     * @var VotingDecisionEnum
-     */
-    private $decision;
-
-    /**
-     * @var ISingleVoterResult[]
-     */
-    private $partialResults;
-
-
-    /**
-     * VotingResult constructor.
-     * @param VotingDecisionEnum $decision
-     * @param ISingleVoterResult[] $partialResults
-     */
-    public function __construct(VotingDecisionEnum $decision, $partialResults)
+	/**
+	 * @var VotingDecisionEnum
+	 */
+	private $decision;
+
+	/**
+	 * @var ISingleVoterResult[]
+	 */
+	private $partialResults;
+
+
+	/**
+	 * VotingResult constructor.
+	 * @param VotingDecisionEnum $decision
+	 * @param ISingleVoterResult[] $partialResults
+	 */
+	public function __construct(VotingDecisionEnum $decision, $partialResults)
 	{
-        $this->decision = $decision;
-        $this->partialResults = $partialResults;
-    }
-
-
-    /**
-     * @return VotingDecisionEnum
-     */
-    public function getDecision()
-    {
-        return $this->decision;
-    }
-
-
-    /**
-     * @return ISingleVoterResult[]
-     */
-    public function getPartialResults()
-    {
-        return $this->partialResults;
-    }
+		$this->decision = $decision;
+		$this->partialResults = $partialResults;
+	}
+
+
+	/**
+	 * @return VotingDecisionEnum
+	 */
+	public function getDecision()
+	{
+		return $this->decision;
+	}
+
+
+	/**
+	 * @return ISingleVoterResult[]
+	 */
+	public function getPartialResults()
+	{
+		return $this->partialResults;
+	}
 }

src/Voter/ISingleVoterResult.php

Indentation +4 added lines, -4 removed lines

@@ -5,8 +5,8 @@
 
 interface ISingleVoterResult extends IResult
 {
-    /**
-     * @return mixed
-     */
-    public function getReason();
+	/**
+	 * @return mixed
+	 */
+	public function getReason();
 }
\ No newline at end of file

src/Voter/SingleVoterResult.php

Indentation +38 added lines, -38 removed lines

@@ -6,42 +6,42 @@
 class SingleVoterResult implements ISingleVoterResult
 {
 
-    /**
-     * @var VotingDecisionEnum
-     */
-    private $decision;
-
-    /**
-     * @var mixed
-     */
-    private $reason;
-
-
-    /**
-     * SingleVoterResult constructor.
-     * @param VotingDecisionEnum $decision
-     * @param mixed $reason
-     */
-    public function __construct(VotingDecisionEnum $decision, $reason = null)
-    {
-        $this->decision = $decision;
-        $this->reason = $reason;
-    }
-
-
-    /**
-     * @return VotingDecisionEnum
-     */
-    public function getDecision()
-    {
-        return $this->decision;
-    }
-
-    /**
-     * @return mixed
-     */
-    public function getReason()
-    {
-        return $this->reason;
-    }
+	/**
+	 * @var VotingDecisionEnum
+	 */
+	private $decision;
+
+	/**
+	 * @var mixed
+	 */
+	private $reason;
+
+
+	/**
+	 * SingleVoterResult constructor.
+	 * @param VotingDecisionEnum $decision
+	 * @param mixed $reason
+	 */
+	public function __construct(VotingDecisionEnum $decision, $reason = null)
+	{
+		$this->decision = $decision;
+		$this->reason = $reason;
+	}
+
+
+	/**
+	 * @return VotingDecisionEnum
+	 */
+	public function getDecision()
+	{
+		return $this->decision;
+	}
+
+	/**
+	 * @return mixed
+	 */
+	public function getReason()
+	{
+		return $this->reason;
+	}
 }
\ No newline at end of file

src/IResult.php

Indentation +4 added lines, -4 removed lines

@@ -4,8 +4,8 @@
 
 interface IResult
 {
-    /**
-     * @return VotingDecisionEnum
-     */
-    public function getDecision();
+	/**
+	 * @return VotingDecisionEnum
+	 */
+	public function getDecision();
 }

src/Assembly/VotingAbilityAwareAssembly.php

Indentation +50 added lines, -50 removed lines

@@ -10,64 +10,64 @@
 class VotingAbilityAwareAssembly extends VotingAssembly implements IVotingAbilityAwareAssembly
 {
 
-    /**
-     * @var string
-     */
-    protected $subjectClassName;
+	/**
+	 * @var string
+	 */
+	protected $subjectClassName;
 
-    /**
-     * @var string
-     */
-    protected $contextClassName;
+	/**
+	 * @var string
+	 */
+	protected $contextClassName;
 
-    /**
-     * @var string[]
-     */
-    protected $actionList;
+	/**
+	 * @var string[]
+	 */
+	protected $actionList;
 
 
-    /**
-     * @param \SpareParts\Overseer\StrategyEnum $strategy
-     * @param IVoter[] $voters
-     * @param string|string[] $actions
-     * @param string $contextClassName
-     * @param string $subjectClassName
-     */
-    public function __construct(
-        StrategyEnum $strategy,
-        array $voters,
-        $actions = null,
-        $contextClassName = IIdentityContext::class,
-        $subjectClassName = null
-    ) {
-        parent::__construct($strategy, $voters);
+	/**
+	 * @param \SpareParts\Overseer\StrategyEnum $strategy
+	 * @param IVoter[] $voters
+	 * @param string|string[] $actions
+	 * @param string $contextClassName
+	 * @param string $subjectClassName
+	 */
+	public function __construct(
+		StrategyEnum $strategy,
+		array $voters,
+		$actions = null,
+		$contextClassName = IIdentityContext::class,
+		$subjectClassName = null
+	) {
+		parent::__construct($strategy, $voters);
 
-        $this->actionList = (array) $actions;
-        $this->subjectClassName = $subjectClassName;
-        $this->contextClassName = $contextClassName;
-    }
+		$this->actionList = (array) $actions;
+		$this->subjectClassName = $subjectClassName;
+		$this->contextClassName = $contextClassName;
+	}
 
 
-    /**
-     * @param string $actionName
-     * @param object $subject
-     * @param \SpareParts\Overseer\Context\IVotingContext $context
-     * @return bool
-     */
-    public function canVoteOn($actionName, $subject, IVotingContext $context)
-    {
-        if ($this->subjectClassName && !($subject instanceof $this->subjectClassName)) {
-            return false;
-        }
+	/**
+	 * @param string $actionName
+	 * @param object $subject
+	 * @param \SpareParts\Overseer\Context\IVotingContext $context
+	 * @return bool
+	 */
+	public function canVoteOn($actionName, $subject, IVotingContext $context)
+	{
+		if ($this->subjectClassName && !($subject instanceof $this->subjectClassName)) {
+			return false;
+		}
 
-        if ($this->contextClassName && !($context instanceof $this->contextClassName)) {
-            return false;
-        }
+		if ($this->contextClassName && !($context instanceof $this->contextClassName)) {
+			return false;
+		}
 
-        if ($this->actionList && !in_array($actionName, $this->actionList)) {
-            return false;
-        }
-        return true;
-    }
+		if ($this->actionList && !in_array($actionName, $this->actionList)) {
+			return false;
+		}
+		return true;
+	}
 
 }

Spacing +1 added lines, -1 removed lines

@@ -42,7 +42,7 @@
     ) {
         parent::__construct($strategy, $voters);
 
-        $this->actionList = (array) $actions;
+        $this->actionList = (array)$actions;
         $this->subjectClassName = $subjectClassName;
         $this->contextClassName = $contextClassName;
     }

src/Assembly/VotingAssembly.php

Indentation +103 added lines, -103 removed lines

@@ -12,107 +12,107 @@
 class VotingAssembly implements IVotingAssembly
 {
 
-    /**
-     * @var StrategyEnum
-     */
-    private $strategy;
-
-    /**
-     * @var IVoter[]
-     */
-    private $voters;
-
-
-    /**
-     * VotingAssembly constructor.
-     * @param StrategyEnum $strategy
-     * @param \SpareParts\Overseer\Voter\IVoter[] $voters
-     */
-    public function __construct(StrategyEnum $strategy, array $voters)
-    {
-        $this->strategy = $strategy;
-        $this->voters = $voters;
-    }
-
-
-    /**
-     * @param mixed $votingSubject
-     * @param \SpareParts\Overseer\Context\IVotingContext $votingContext
-     * @return null|\SpareParts\Overseer\IVotingResult
-     * @throws \SpareParts\Overseer\InvalidVotingResultException
-     */
-    public function commenceVote($votingSubject, IVotingContext $votingContext)
-    {
-        $result = null;
-        switch ($this->strategy) {
-            case StrategyEnum::FIRST_VOTE_DECIDES():
-                return $this->strategyFirstVoteDecides($votingSubject, $votingContext);
-
-            case StrategyEnum::ALLOW_UNLESS_DENIED():
-                return $this->strategyAllowUnlessDenied($votingSubject, $votingContext);
-
-            case StrategyEnum::DENY_UNLESS_ALLOWED():
-                return $this->strategyDenyUnlessAllowed($votingSubject, $votingContext);
-
-            default:
-                throw new InvalidVotingResultException('Unable to decide on result, invalid strategy: '.$this->strategy);
-        }
-    }
-
-
-    /**
-     * @param mixed $votingSubject
-     * @param \SpareParts\Overseer\Context\IVotingContext $votingContext
-     * @return \SpareParts\Overseer\IVotingResult
-     * @throws \SpareParts\Overseer\InvalidVotingResultException
-     */
-    private function strategyFirstVoteDecides($votingSubject, IVotingContext $votingContext)
-    {
-        foreach ($this->voters as $voter) {
-            if (($lastResult = $voter->vote($votingSubject, $votingContext)) !== null) {
-                return new VotingResult($lastResult->getDecision(), [$lastResult]);
-            }
-        }
-        throw new InvalidVotingResultException('Voting assembly did not decide on any result!');
-    }
-
-
-    /**
-     * @param mixed $votingSubject
-     * @param \SpareParts\Overseer\Context\IVotingContext $votingContext
-     * @return \SpareParts\Overseer\IVotingResult
-     */
-    private function strategyAllowUnlessDenied($votingSubject, IVotingContext $votingContext)
-    {
-        $results = [];
-        foreach ($this->voters as $name => $voter) {
-            if (($lastResult = $voter->vote($votingSubject, $votingContext)) !== null) {
-                $results[] = $lastResult;
-                if ($lastResult->getDecision() === VotingDecisionEnum::DENIED()) {
-                    return new VotingResult(VotingDecisionEnum::DENIED(), $results);
-                }
-            }
-        }
-        return new VotingResult(VotingDecisionEnum::ALLOWED(), $results);
-    }
-
-
-    /**
-     * @param mixed $votingSubject
-     * @param \SpareParts\Overseer\Context\IVotingContext $votingContext
-     * @return \SpareParts\Overseer\IVotingResult
-     */
-    private function strategyDenyUnlessAllowed($votingSubject, IVotingContext $votingContext)
-    {
-        $results = [];
-        foreach ($this->voters as $name => $voter) {
-            if (($lastResult = $voter->vote($votingSubject, $votingContext)) !== null) {
-                $results[] = $lastResult;
-                if ($lastResult->getDecision() === VotingDecisionEnum::ALLOWED()) {
-                    return new VotingResult(VotingDecisionEnum::ALLOWED(), $results);
-                }
-            }
-        }
-        return new VotingResult(VotingDecisionEnum::DENIED(), $results);
-    }
+	/**
+	 * @var StrategyEnum
+	 */
+	private $strategy;
+
+	/**
+	 * @var IVoter[]
+	 */
+	private $voters;
+
+
+	/**
+	 * VotingAssembly constructor.
+	 * @param StrategyEnum $strategy
+	 * @param \SpareParts\Overseer\Voter\IVoter[] $voters
+	 */
+	public function __construct(StrategyEnum $strategy, array $voters)
+	{
+		$this->strategy = $strategy;
+		$this->voters = $voters;
+	}
+
+
+	/**
+	 * @param mixed $votingSubject
+	 * @param \SpareParts\Overseer\Context\IVotingContext $votingContext
+	 * @return null|\SpareParts\Overseer\IVotingResult
+	 * @throws \SpareParts\Overseer\InvalidVotingResultException
+	 */
+	public function commenceVote($votingSubject, IVotingContext $votingContext)
+	{
+		$result = null;
+		switch ($this->strategy) {
+			case StrategyEnum::FIRST_VOTE_DECIDES():
+				return $this->strategyFirstVoteDecides($votingSubject, $votingContext);
+
+			case StrategyEnum::ALLOW_UNLESS_DENIED():
+				return $this->strategyAllowUnlessDenied($votingSubject, $votingContext);
+
+			case StrategyEnum::DENY_UNLESS_ALLOWED():
+				return $this->strategyDenyUnlessAllowed($votingSubject, $votingContext);
+
+			default:
+				throw new InvalidVotingResultException('Unable to decide on result, invalid strategy: '.$this->strategy);
+		}
+	}
+
+
+	/**
+	 * @param mixed $votingSubject
+	 * @param \SpareParts\Overseer\Context\IVotingContext $votingContext
+	 * @return \SpareParts\Overseer\IVotingResult
+	 * @throws \SpareParts\Overseer\InvalidVotingResultException
+	 */
+	private function strategyFirstVoteDecides($votingSubject, IVotingContext $votingContext)
+	{
+		foreach ($this->voters as $voter) {
+			if (($lastResult = $voter->vote($votingSubject, $votingContext)) !== null) {
+				return new VotingResult($lastResult->getDecision(), [$lastResult]);
+			}
+		}
+		throw new InvalidVotingResultException('Voting assembly did not decide on any result!');
+	}
+
+
+	/**
+	 * @param mixed $votingSubject
+	 * @param \SpareParts\Overseer\Context\IVotingContext $votingContext
+	 * @return \SpareParts\Overseer\IVotingResult
+	 */
+	private function strategyAllowUnlessDenied($votingSubject, IVotingContext $votingContext)
+	{
+		$results = [];
+		foreach ($this->voters as $name => $voter) {
+			if (($lastResult = $voter->vote($votingSubject, $votingContext)) !== null) {
+				$results[] = $lastResult;
+				if ($lastResult->getDecision() === VotingDecisionEnum::DENIED()) {
+					return new VotingResult(VotingDecisionEnum::DENIED(), $results);
+				}
+			}
+		}
+		return new VotingResult(VotingDecisionEnum::ALLOWED(), $results);
+	}
+
+
+	/**
+	 * @param mixed $votingSubject
+	 * @param \SpareParts\Overseer\Context\IVotingContext $votingContext
+	 * @return \SpareParts\Overseer\IVotingResult
+	 */
+	private function strategyDenyUnlessAllowed($votingSubject, IVotingContext $votingContext)
+	{
+		$results = [];
+		foreach ($this->voters as $name => $voter) {
+			if (($lastResult = $voter->vote($votingSubject, $votingContext)) !== null) {
+				$results[] = $lastResult;
+				if ($lastResult->getDecision() === VotingDecisionEnum::ALLOWED()) {
+					return new VotingResult(VotingDecisionEnum::ALLOWED(), $results);
+				}
+			}
+		}
+		return new VotingResult(VotingDecisionEnum::DENIED(), $results);
+	}
 }

Spacing +5 added lines, -5 removed lines

@@ -70,7 +70,7 @@ 
     {
         foreach ($this->voters as $voter) {
             if (($lastResult = $voter->vote($votingSubject, $votingContext)) !== null) {
-                return new VotingResult($lastResult->getDecision(), [$lastResult]);
+                return new VotingResult($lastResult->getDecision(), [ $lastResult ]);
             }
         }
         throw new InvalidVotingResultException('Voting assembly did not decide on any result!');
@@ -84,10 +84,10 @@ 
      */
     private function strategyAllowUnlessDenied($votingSubject, IVotingContext $votingContext)
     {
-        $results = [];
+        $results = [ ];
         foreach ($this->voters as $name => $voter) {
             if (($lastResult = $voter->vote($votingSubject, $votingContext)) !== null) {
-                $results[] = $lastResult;
+                $results[ ] = $lastResult;
                 if ($lastResult->getDecision() === VotingDecisionEnum::DENIED()) {
                     return new VotingResult(VotingDecisionEnum::DENIED(), $results);
                 }
@@ -104,10 +104,10 @@ 
      */
     private function strategyDenyUnlessAllowed($votingSubject, IVotingContext $votingContext)
     {
-        $results = [];
+        $results = [ ];
         foreach ($this->voters as $name => $voter) {
             if (($lastResult = $voter->vote($votingSubject, $votingContext)) !== null) {
-                $results[] = $lastResult;
+                $results[ ] = $lastResult;
                 if ($lastResult->getDecision() === VotingDecisionEnum::ALLOWED()) {
                     return new VotingResult(VotingDecisionEnum::ALLOWED(), $results);
                 }

src/Assembly/IVotingAbilityAwareAssembly.php

Indentation +7 added lines, -7 removed lines

@@ -13,11 +13,11 @@
 {
 
 
-    /**
-     * @param string $actionName
-     * @param mixed $subject
-     * @param \SpareParts\Overseer\Context\IVotingContext $context
-     * @return bool
-     */
-    public function canVoteOn($actionName, $subject, IVotingContext $context);
+	/**
+	 * @param string $actionName
+	 * @param mixed $subject
+	 * @param \SpareParts\Overseer\Context\IVotingContext $context
+	 * @return bool
+	 */
+	public function canVoteOn($actionName, $subject, IVotingContext $context);
 }