| @@ -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 |      { | 
| @@ -43,6 +43,9 @@ | ||
| 43 | 43 | $form = $commentAdmin->getEditForm(); | 
| 44 | 44 | } | 
| 45 | 45 | |
| 46 | + /** | |
| 47 | + * @param Form $form | |
| 48 | + */ | |
| 46 | 49 |      private function getFormFieldNames($form) { | 
| 47 | 50 | $result = array(); | 
| 48 | 51 | $fields = $form->Fields(); | 
| @@ -1196,6 +1196,9 @@ | ||
| 1196 | 1196 | } | 
| 1197 | 1197 | |
| 1198 | 1198 | |
| 1199 | + /** | |
| 1200 | + * @param string $name | |
| 1201 | + */ | |
| 1199 | 1202 |      protected static function getMethod($name) { | 
| 1200 | 1203 |          $class = new ReflectionClass('Comment'); | 
| 1201 | 1204 | $method = $class->getMethod($name); | 
| @@ -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 |      { |