| @@ -316,7 +316,7 @@ discard block | ||
| 316 | 316 | /** | 
| 317 | 317 | * Checks if the comment can be edited. | 
| 318 | 318 | * | 
| 319 | - * @param null|int|Member $member | |
| 319 | + * @param DataObject|null $member | |
| 320 | 320 | * | 
| 321 | 321 | * @return Boolean | 
| 322 | 322 | */ | 
| @@ -346,7 +346,7 @@ discard block | ||
| 346 | 346 | /** | 
| 347 | 347 | * Checks if the comment can be deleted. | 
| 348 | 348 | * | 
| 349 | - * @param null|int|Member $member | |
| 349 | + * @param Member|null $member | |
| 350 | 350 | * | 
| 351 | 351 | * @return Boolean | 
| 352 | 352 | */ | 
| @@ -369,7 +369,7 @@ discard block | ||
| 369 | 369 | * Resolves Member object. | 
| 370 | 370 | * | 
| 371 | 371 | * @param Member|int|null $member | 
| 372 | - * @return Member|null | |
| 372 | + * @return DataObject|null | |
| 373 | 373 | */ | 
| 374 | 374 |  	protected function getMember($member = null) { | 
| 375 | 375 |  		if(!$member) { | 
| @@ -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 | 
| @@ -280,6 +280,7 @@ discard block | ||
| 280 | 280 | * Redirect back to referer if available, ensuring that only site URLs | 
| 281 | 281 | * are allowed to avoid phishing. If it's an AJAX request render the | 
| 282 | 282 | * comment in it's new state | 
| 283 | + * @param DataObject $comment | |
| 283 | 284 | */ | 
| 284 | 285 |      private function renderChangedCommentState($comment) { | 
| 285 | 286 |          $referer = $this->request->getHeader('Referer'); | 
| @@ -23,6 +23,7 @@ discard block | ||
| 23 | 23 | * @param string classname to add commenting to | 
| 24 | 24 |  	 * @param array $settings Settings. See {@link self::$default_config} for | 
| 25 | 25 | * available settings | 
| 26 | + * @param string $class | |
| 26 | 27 | * | 
| 27 | 28 | * @throws InvalidArgumentException | 
| 28 | 29 | */ | 
| @@ -72,7 +73,7 @@ discard block | ||
| 72 | 73 | * @param string $class Class to set the value on. Passing 'all' will set it to all | 
| 73 | 74 | * active mappings | 
| 74 | 75 | * @param string $key setting to change | 
| 75 | - * @param mixed $value value of the setting | |
| 76 | + * @param string $value value of the setting | |
| 76 | 77 | */ | 
| 77 | 78 |  	public static function set_config_value($class, $key, $value = false) { | 
| 78 | 79 |  		Deprecation::notice('2.0', 'Commenting::set_config_value is deprecated. Use the config api instead'); | 
| @@ -116,7 +117,7 @@ discard block | ||
| 116 | 117 | * @param string $class | 
| 117 | 118 | * @param string $key | 
| 118 | 119 | * @param string $value Expected value | 
| 119 | - * @return boolean | |
| 120 | + * @return boolean|null | |
| 120 | 121 | */ | 
| 121 | 122 |  	public static function config_value_equals($class, $key, $value) { | 
| 122 | 123 | $check = self::get_config_value($class, $key); | 
| @@ -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); |