sminnee / silverstripe-framework

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) {

View/Parsers/HTMLValue.php

Doc Comments +1 added lines

@@ -139,6 +139,7 @@
 	/**
 	 * Get the body element, or false if there isn't one (we haven't loaded any content
 	 * or this instance is in an invalid state)
+	 * @return \DOMElement
 	 */
 	public function getBody() {
 		$doc = $this->getDocument();

View/Parsers/ShortcodeParser.php

Doc Comments +3 added lines

@@ -168,6 +168,9 @@
 
 	// --------------------------------------------------------------------------------------------------------------
 
+	/**
+	 * @param DOMElement $node
+	 */
 	protected function removeNode($node) {
 		$node->parentNode->removeChild($node);
 	}

View/Parsers/URLSegmentFilter.php

Doc Comments +1 added lines

@@ -89,6 +89,7 @@
 	 * Determines if the field should render open or closed by default.
 	 *
 	 * @param boolean
+	 * @param boolean $bool
 	 */
 	public function startClosed($bool) {
 		($bool) ? $this->addExtraClass('startClosed') : $this->removeExtraClass('startClosed');

View/Requirements_Backend.php

Doc Comments +1 added lines

@@ -321,6 +321,7 @@
 	 * Forces the JavaScript requirements to the end of the body, right before the closing tag
 	 *
 	 * @param bool
+	 * @param boolean $var
 	 * @return $this
 	 */
 	public function setForceJSToBottom($var)