@@ -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 |