SpareParts / Overseer

src/StrategyEnum.php

Doc Comments +1 added lines, -1 removed lines

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

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 ];
     }
 
 

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)

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('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];
-    }
-
-
-    /**
-     * @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('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];
+	}
+
+
+	/**
+	 * @return string
+	 */
+	public function __toString()
+	{
+		return $this->value;
+	}
 }
\ No newline at end of file

Spacing +4 added lines, -4 removed lines

@@ -12,7 +12,7 @@ 
     /**
      * @var self[]
      */
-    static private $registry = [];
+    static private $registry = [ ];
 
     private function __construct($value)
     {
@@ -44,10 +44,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 ];
     }
 
 

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/GenericVotingManager.php

Indentation +4 added lines, -4 removed lines

@@ -25,10 +25,10 @@
 	 */
 	public function vote($action, $votingSubject, IVotingContext $votingContext)
 	{
-        if (!($votingSubject instanceof IVotingSubject)) {
-            $votingSubject = new VotingSubject($votingSubject);
-        }
+		if (!($votingSubject instanceof IVotingSubject)) {
+			$votingSubject = new VotingSubject($votingSubject);
+		}
 
-        return $this->innerVote($action, $votingSubject, $votingContext);
+		return $this->innerVote($action, $votingSubject, $votingContext);
 	}
 }
\ No newline at end of file

src/Assembly/VotingAssembly.php

Indentation +16 added lines, -16 removed lines

@@ -89,7 +89,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!');
@@ -103,13 +103,13 @@ 
 	 */
 	private function strategyAllowUnlessDenied($votingSubject, $votingContext)
 	{
-	    $results = [];
+		$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);
-                }
+				$results[] = $lastResult;
+				if ($lastResult->getDecision() == VotingDecisionEnum::DENIED()) {
+					return new VotingResult(VotingDecisionEnum::DENIED(), $results);
+				}
 			}
 		}
 		return new VotingResult(VotingDecisionEnum::ALLOWED(), $results);
@@ -123,16 +123,16 @@ 
 	 */
 	private function strategyDenyUnlessAllowed($votingSubject, $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::ALLOWED(), $results);
-                }
-            }
-        }
-        return new VotingResult(VotingDecisionEnum::DENIED(), $results);
+		$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::ALLOWED(), $results);
+				}
+			}
+		}
+		return new VotingResult(VotingDecisionEnum::DENIED(), $results);
 	}
 
 

Spacing +5 added lines, -5 removed lines

@@ -89,7 +89,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!');
@@ -103,10 +103,10 @@ 
 	 */
 	private function strategyAllowUnlessDenied($votingSubject, $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);
                 }
@@ -123,10 +123,10 @@ 
 	 */
 	private function strategyDenyUnlessAllowed($votingSubject, $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::ALLOWED(), $results);
                 }

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();
 }
\ No newline at end of file

src/Voter/RoleVoter.php

Indentation +19 added lines, -19 removed lines

@@ -12,28 +12,28 @@
 	private $allowedRoles;
 
 	/**
-     * @var VotingDecisionEnum
-     */
-    private $resultDecision;
-
-    /**
-     * @var mixed|null
-     */
-    private $reason;
-
-
-    /**
-     * RoleVoter constructor.
-     * @param VotingDecisionEnum $resultDecision
-     * @param string|string[] $allowedRoles
-     * @param mixed $reason
-     */
+	 * @var VotingDecisionEnum
+	 */
+	private $resultDecision;
+
+	/**
+	 * @var mixed|null
+	 */
+	private $reason;
+
+
+	/**
+	 * RoleVoter constructor.
+	 * @param VotingDecisionEnum $resultDecision
+	 * @param string|string[] $allowedRoles
+	 * @param mixed $reason
+	 */
 	public function __construct(VotingDecisionEnum $resultDecision, $allowedRoles, $reason = null)
 	{
 		$this->allowedRoles = (array) $allowedRoles;
-        $this->resultDecision = $resultDecision;
-        $this->reason = $reason;
-    }
+		$this->resultDecision = $resultDecision;
+		$this->reason = $reason;
+	}
 
 
 	/**

Spacing +1 added lines, -1 removed lines

@@ -30,7 +30,7 @@
      */
 	public function __construct(VotingDecisionEnum $resultDecision, $allowedRoles, $reason = null)
 	{
-		$this->allowedRoles = (array) $allowedRoles;
+		$this->allowedRoles = (array)$allowedRoles;
         $this->resultDecision = $resultDecision;
         $this->reason = $reason;
     }

src/Voter/VotingSubject.php

Indentation +6 added lines, -6 removed lines

@@ -27,12 +27,12 @@
 	 */
 	public function getVotingSubjectName()
 	{
-	    if (is_string($this->subject)) {
-            return $this->subject;
-        }
-        if (is_object($this->subject)) {
-            return get_class($this->subject);
-        }
+		if (is_string($this->subject)) {
+			return $this->subject;
+		}
+		if (is_object($this->subject)) {
+			return get_class($this->subject);
+		}
 		return (string) $this->subject;
 	}
 }

Spacing +1 added lines, -1 removed lines

@@ -33,6 +33,6 @@
         if (is_object($this->subject)) {
             return get_class($this->subject);
         }
-		return (string) $this->subject;
+		return (string)$this->subject;
 	}
 }

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