@@ -31,7 +31,7 @@ discard block |
||
| 31 | 31 | /** |
| 32 | 32 | * Modify method visibility to public for testing |
| 33 | 33 | * |
| 34 | - * @param mixed $errors |
|
| 34 | + * @param string $errors |
|
| 35 | 35 | */ |
| 36 | 36 | public function setDisplayErrors($errors) |
| 37 | 37 | { |
@@ -40,6 +40,10 @@ discard block |
||
| 40 | 40 | } |
| 41 | 41 | |
| 42 | 42 | // Change function visibility to be testable directly |
| 43 | + |
|
| 44 | + /** |
|
| 45 | + * @param string $memstring |
|
| 46 | + */ |
|
| 43 | 47 | public function translateMemstring($memstring) |
| 44 | 48 | { |
| 45 | 49 | return parent::translateMemstring($memstring); |
@@ -271,6 +271,9 @@ |
||
| 271 | 271 | } |
| 272 | 272 | |
| 273 | 273 | |
| 274 | + /** |
|
| 275 | + * @param resource $file |
|
| 276 | + */ |
|
| 274 | 277 | protected function getLineCount(&$file) |
| 275 | 278 | { |
| 276 | 279 | $i = 0; |
@@ -385,7 +385,7 @@ |
||
| 385 | 385 | * Simulates a file upload |
| 386 | 386 | * |
| 387 | 387 | * @param string $fileField Name of the field to mock upload for |
| 388 | - * @param array $tmpFileName Name of temporary file to upload |
|
| 388 | + * @param string $tmpFileName Name of temporary file to upload |
|
| 389 | 389 | * @return HTTPResponse form response |
| 390 | 390 | */ |
| 391 | 391 | protected function mockFileUpload($fileField, $tmpFileName) |
@@ -24,7 +24,7 @@ |
||
| 24 | 24 | |
| 25 | 25 | /** |
| 26 | 26 | * @skipUpgrade |
| 27 | - * @param null $controller |
|
| 27 | + * @param TestController $controller |
|
| 28 | 28 | * @param string $name |
| 29 | 29 | */ |
| 30 | 30 | public function __construct($controller = null, $name = 'Form') |
@@ -399,7 +399,7 @@ |
||
| 399 | 399 | * string of HTML. |
| 400 | 400 | * |
| 401 | 401 | * @param string $html HTML to scan for elements |
| 402 | - * @return SimpleXMLElement |
|
| 402 | + * @return \SimpleXMLElement |
|
| 403 | 403 | */ |
| 404 | 404 | public function findOptionElements($html) |
| 405 | 405 | { |
@@ -20,6 +20,9 @@ |
||
| 20 | 20 | |
| 21 | 21 | protected $id; |
| 22 | 22 | |
| 23 | + /** |
|
| 24 | + * @param string $link |
|
| 25 | + */ |
|
| 23 | 26 | public function __construct($gridField, $id, $link) |
| 24 | 27 | { |
| 25 | 28 | $this->gridField = $gridField; |
@@ -29,6 +29,9 @@ |
||
| 29 | 29 | return str_replace(' ', $nbsp, trim($input)); |
| 30 | 30 | } |
| 31 | 31 | |
| 32 | + /** |
|
| 33 | + * @param string $locale |
|
| 34 | + */ |
|
| 32 | 35 | protected function checkInputValidation($locale, $tests) |
| 33 | 36 | { |
| 34 | 37 | i18n::set_locale($locale); |
@@ -1023,7 +1023,7 @@ discard block |
||
| 1023 | 1023 | * Simulates a file upload |
| 1024 | 1024 | * |
| 1025 | 1025 | * @param string $fileField Name of the field to mock upload for |
| 1026 | - * @param array $tmpFileName Name of temporary file to upload |
|
| 1026 | + * @param string $tmpFileName Name of temporary file to upload |
|
| 1027 | 1027 | * @return HTTPResponse form response |
| 1028 | 1028 | */ |
| 1029 | 1029 | protected function mockFileUpload($fileField, $tmpFileName) |
@@ -1036,6 +1036,9 @@ discard block |
||
| 1036 | 1036 | ); |
| 1037 | 1037 | } |
| 1038 | 1038 | |
| 1039 | + /** |
|
| 1040 | + * @param string $fileField |
|
| 1041 | + */ |
|
| 1039 | 1042 | protected function mockFileExists($fileField, $fileName) |
| 1040 | 1043 | { |
| 1041 | 1044 | return $this->get( |
@@ -39,6 +39,9 @@ |
||
| 39 | 39 | return $this->can(__FUNCTION__, $member, $context); |
| 40 | 40 | } |
| 41 | 41 | |
| 42 | + /** |
|
| 43 | + * @param string $perm |
|
| 44 | + */ |
|
| 42 | 45 | public function can($perm, $member = null, $context = array()) |
| 43 | 46 | { |
| 44 | 47 | $perms = [ |