sminnee / silverstripe-framework

tests/core/ConvertTest.php

Unused Use Statements -1 removed lines

@@ -1,7 +1,6 @@
 <?php
 
 use SilverStripe\Core\Convert;
-use SilverStripe\Core\Config\Config;
 use SilverStripe\Dev\SapphireTest;
 use SilverStripe\View\Parsers\URLSegmentFilter;
 

tests/forms/gridfield/GridFieldAddExistingAutocompleterTest.php

Unused Use Statements -1 removed lines

@@ -1,6 +1,5 @@
 <?php
 
-use SilverStripe\ORM\DataObject;
 use SilverStripe\Security\Member;
 use SilverStripe\Security\Permission;
 use SilverStripe\Security\Security;

tests/forms/GridFieldTest.php

Doc Comments +1 added lines, -1 removed lines

@@ -129,7 +129,7 @@
 	/**
 	 *
 	 * @param GridField $gridField
-	 * @return array
+	 * @return string[]
 	 */
 	public function getActions($gridField) {
 		return array('addto');

tests/forms/OptionsetFieldTest.php

Unused Use Statements -1 removed lines

@@ -5,7 +5,6 @@
 use SilverStripe\Filesystem\ImageManipulation;
 use SilverStripe\Filesystem\Storage\AssetContainer;
 use SilverStripe\Filesystem\Storage\AssetStore;
-
 use Injector;
 use AssetField;
 use File;

tests/forms/RequirementsTest.php

Doc Comments +7 added lines

@@ -127,6 +127,10 @@ 
 		);
 	}
 
+	/**
+	 * @param boolean $async
+	 * @param boolean $defer
+	 */
 	protected function setupCombinedRequirementsJavascriptAsyncDefer($backend, $async, $defer) {
         $basePath = $this->getCurrentRelativePath();
         $this->setupRequirements($backend);
@@ -836,6 +840,9 @@ 
 		}
 	}
 
+	/**
+	 * @param string $type
+	 */
 	public function assertFileNotIncluded($backend, $type, $files) {
 		$includedFiles = $this->getBackendFiles($backend, $type);
 		if(is_array($files)) {

tests/forms/uploadfield/AssetFieldTest.php

Doc Comments +1 added lines, -1 removed lines

@@ -356,7 +356,7 @@
 	 * Simulates a file upload
 	 *
 	 * @param string $fileField Name of the field to mock upload for
-	 * @param array $tmpFileName Name of temporary file to upload
+	 * @param string $tmpFileName Name of temporary file to upload
 	 * @return HTTPResponse form response
 	 */
 	protected function mockFileUpload($fileField, $tmpFileName) {

tests/forms/uploadfield/UploadFieldTest.php

Doc Comments +4 added lines, -1 removed lines

@@ -946,7 +946,7 @@ 
 	 * Simulates a file upload
 	 *
 	 * @param string $fileField Name of the field to mock upload for
-	 * @param array $tmpFileName Name of temporary file to upload
+	 * @param string $tmpFileName Name of temporary file to upload
 	 * @return HTTPResponse form response
 	 */
 	protected function mockFileUpload($fileField, $tmpFileName) {
@@ -958,6 +958,9 @@ 
 		);
 	}
 
+	/**
+	 * @param string $fileField
+	 */
 	protected function mockFileExists($fileField, $fileName) {
 		return $this->get(
 			"UploadFieldTest_Controller/Form/field/{$fileField}/fileexists?filename=".urlencode($fileName)

tests/model/ImageTest.php

Unused Use Statements -4 removed lines

@@ -1,9 +1,5 @@
 <?php
 
-use League\Flysystem\Filesystem;
-use SilverStripe\Assets\Flysystem\AssetAdapter;
-use SilverStripe\Assets\Flysystem\FlysystemAssetStore;
-use SilverStripe\Assets\Flysystem\FlysystemUrlPlugin;
 use SilverStripe\Assets\File;
 use SilverStripe\Assets\Filesystem as SSFilesystem;
 use SilverStripe\Assets\Image;

View/Parsers/Diff.php

Doc Comments +1 added lines, -1 removed lines

@@ -160,7 +160,7 @@
 	}
 
 	/**
-	 * @param string|array $content If passed as an array, values will be concatenated with a comma.
+	 * @param string $content If passed as an array, values will be concatenated with a comma.
 	 * @return array
 	 */
 	public static function getHTMLChunks($content) {