@@ -120,7 +120,7 @@ discard block |
||
120 | 120 | * Get the commenting option for the current state |
121 | 121 | * |
122 | 122 | * @param string $key |
123 | - * @return mixed Result if the setting is available, or null otherwise |
|
123 | + * @return integer Result if the setting is available, or null otherwise |
|
124 | 124 | */ |
125 | 125 | public function getOption($key) { |
126 | 126 | // If possible use the current record |
@@ -279,6 +279,7 @@ discard block |
||
279 | 279 | * Redirect back to referer if available, ensuring that only site URLs |
280 | 280 | * are allowed to avoid phishing. If it's an AJAX request render the |
281 | 281 | * comment in it's new state |
282 | + * @param DataObject $comment |
|
282 | 283 | */ |
283 | 284 | private function renderChangedCommentState($comment) { |
284 | 285 | $referer = $this->request->getHeader('Referer'); |
@@ -334,7 +334,7 @@ discard block |
||
334 | 334 | /** |
335 | 335 | * Checks if the comment can be edited. |
336 | 336 | * |
337 | - * @param null|int|Member $member |
|
337 | + * @param DataObject|null $member |
|
338 | 338 | * |
339 | 339 | * @return Boolean |
340 | 340 | */ |
@@ -365,7 +365,7 @@ discard block |
||
365 | 365 | /** |
366 | 366 | * Checks if the comment can be deleted. |
367 | 367 | * |
368 | - * @param null|int|Member $member |
|
368 | + * @param Member|null $member |
|
369 | 369 | * |
370 | 370 | * @return Boolean |
371 | 371 | */ |
@@ -389,7 +389,7 @@ discard block |
||
389 | 389 | * Resolves Member object. |
390 | 390 | * |
391 | 391 | * @param Member|int|null $member |
392 | - * @return Member|null |
|
392 | + * @return DataObject|null |
|
393 | 393 | */ |
394 | 394 | protected function getMember($member = null) |
395 | 395 | { |
@@ -24,6 +24,7 @@ discard block |
||
24 | 24 | * @param string classname to add commenting to |
25 | 25 | * @param array $settings Settings. See {@link self::$default_config} for |
26 | 26 | * available settings |
27 | + * @param string $class |
|
27 | 28 | * |
28 | 29 | * @throws InvalidArgumentException |
29 | 30 | */ |
@@ -78,7 +79,7 @@ discard block |
||
78 | 79 | * @param string $class Class to set the value on. Passing 'all' will set it to all |
79 | 80 | * active mappings |
80 | 81 | * @param string $key setting to change |
81 | - * @param mixed $value value of the setting |
|
82 | + * @param string $value value of the setting |
|
82 | 83 | */ |
83 | 84 | public static function set_config_value($class, $key, $value = false) |
84 | 85 | { |
@@ -126,7 +127,7 @@ discard block |
||
126 | 127 | * @param string $class |
127 | 128 | * @param string $key |
128 | 129 | * @param string $value Expected value |
129 | - * @return boolean |
|
130 | + * @return boolean|null |
|
130 | 131 | */ |
131 | 132 | public static function config_value_equals($class, $key, $value) |
132 | 133 | { |