Completed
Pull Request — master (#6434)
by Will
08:33
created
php/Core/Startup/ErrorControlChainTest/ErrorControlChainTest_Chain.php 1 patch
Doc Comments   +5 added lines, -1 removed lines patch added patch discarded remove patch
@@ -31,7 +31,7 @@  discard block
 block discarded – undo
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
 block discarded – undo
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);
Please login to merge, or discard this patch.
tests/php/Dev/CsvBulkLoaderTest.php 1 patch
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -271,6 +271,9 @@
 block discarded – undo
271 271
     }
272 272
 
273 273
 
274
+    /**
275
+     * @param resource $file
276
+     */
274 277
     protected function getLineCount(&$file)
275 278
     {
276 279
         $i = 0;
Please login to merge, or discard this patch.
tests/php/Forms/AssetFieldTest.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -385,7 +385,7 @@
 block discarded – undo
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)
Please login to merge, or discard this patch.
tests/php/Forms/AssetFieldTest/TestForm.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -24,7 +24,7 @@
 block discarded – undo
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')
Please login to merge, or discard this patch.
tests/php/Forms/DropdownFieldTest.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -399,7 +399,7 @@
 block discarded – undo
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
     {
Please login to merge, or discard this patch.
php/Forms/GridField/GridField_URLHandlerTest/TestComponent_ItemRequest.php 1 patch
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -20,6 +20,9 @@
 block discarded – undo
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;
Please login to merge, or discard this patch.
tests/php/Forms/NumericFieldTest.php 1 patch
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -29,6 +29,9 @@
 block discarded – undo
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);
Please login to merge, or discard this patch.
tests/php/Forms/UploadFieldTest.php 1 patch
Doc Comments   +4 added lines, -1 removed lines patch added patch discarded remove patch
@@ -1023,7 +1023,7 @@  discard block
 block discarded – undo
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
 block discarded – undo
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(
Please login to merge, or discard this patch.
tests/php/ORM/ChangeSetTest/Permissions.php 1 patch
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -39,6 +39,9 @@
 block discarded – undo
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 = [
Please login to merge, or discard this patch.