| @@ -104,7 +104,7 @@ discard block | ||
| 104 | 104 | /** | 
| 105 | 105 | * @param string $voterName | 
| 106 | 106 | * @param string|IVotingResult $partialResult | 
| 107 | - * @return \SpareParts\Overseer\VotingResult|null | |
| 107 | + * @return null|IVotingResult | |
| 108 | 108 | * @throws \SpareParts\Overseer\InvalidVotingResultException | 
| 109 | 109 | */ | 
| 110 | 110 | protected function prepareResult($voterName, $partialResult) | 
| @@ -122,6 +122,10 @@ discard block | ||
| 122 | 122 | } | 
| 123 | 123 | |
| 124 | 124 | |
| 125 | + /** | |
| 126 | + * @param IVotingSubject $votingSubject | |
| 127 | + * @param IVotingContext $votingContext | |
| 128 | + */ | |
| 125 | 129 | private function strategyAllowUnlessDenied($votingSubject, $votingContext) | 
| 126 | 130 |  	{ | 
| 127 | 131 |  		foreach ($this->voters as $name => $voter) { | 
| @@ -26,7 +26,7 @@ | ||
| 26 | 26 | */ | 
| 27 | 27 | public function __construct($purpose, $allowedRoles) | 
| 28 | 28 |  	{ | 
| 29 | -		if (!in_array($purpose, [IVotingResult::ALLOW, IVotingResult::DENY ])) { | |
| 29 | +		if (!in_array($purpose, [ IVotingResult::ALLOW, IVotingResult::DENY ])) { | |
| 30 | 30 |  			throw new InvalidArgumentException(sprintf("Wrong voting purpose for this voter: got %s, expected one of allow, deny.", $purpose)); | 
| 31 | 31 | } | 
| 32 | 32 | |