silverstripe / silverstripe-framework

tests/forms/uploadfield/AssetFieldTest.php

Doc Comments +1 added lines, -1 removed lines

@@ -359,7 +359,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 SS_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 SS_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)

thirdparty/difflib/difflib.php

Doc Comments +12 added lines

@@ -245,6 +245,10 @@ 
      * match.  The caller must trim matching lines from the beginning and end
      * of the portions it is going to specify.
      */
+
+    /**
+     * @param double $nchunks
+     */
     private function diag($xoff, $xlim, $yoff, $ylim, $nchunks)
     {
         $flip = false;
@@ -356,6 +360,11 @@ 
      * Note that XLIM, YLIM are exclusive bounds.
      * All line numbers are origin-0 and discarded lines are not counted.
      */
+
+    /**
+     * @param integer $xoff
+     * @param integer $yoff
+     */
     private function compareseq($xoff, $xlim, $yoff, $ylim)
     {
         // Slide down the bottom initial diagonal.
@@ -779,6 +788,9 @@ 
         return $xbeg . ($xlen ? ($ylen ? 'c' : 'd') : 'a') . $ybeg;
     }
 
+    /**
+     * @param string $header
+     */
     protected function start_block($header)
     {
         echo $header;

thirdparty/simpletest/page.php

Doc Comments +7 added lines, -7 removed lines

@@ -145,7 +145,7 @@ 
     /**
      *    Reads the raw content and send events
      *    into the page to be built.
-     *    @param $response SimpleHttpResponse  Fetched response.
+     *    @param SilverStripe\Dev\TestSession_STResponseWrapper $response SimpleHttpResponse  Fetched response.
      *    @return SimplePage                   Newly parsed page.
      *    @access public
      */
@@ -170,8 +170,8 @@ 
 
     /**
      *    Creates the parser used with the builder.
-     *    @param $listener SimpleSaxListener   Target of parser.
-     *    @return SimpleSaxParser              Parser to generate
+     *    @param SimplePageBuilder $listener SimpleSaxListener   Target of parser.
+     *    @return SimpleHtmlSaxParser              Parser to generate
      *                                         events for the builder.
      *    @access protected
      */
@@ -406,7 +406,7 @@ 
 
     /**
      *    Original request as bytes sent down the wire.
-     *    @return mixed              Sent content.
+     *    @return string|false              Sent content.
      *    @access public
      */
     function getRequest() {
@@ -415,7 +415,7 @@ 
 
     /**
      *    Accessor for raw text of page.
-     *    @return string        Raw unparsed content.
+     *    @return boolean        Raw unparsed content.
      *    @access public
      */
     function getRaw() {
@@ -467,7 +467,7 @@ 
 
     /**
      *    Base URL if set via BASE tag page url otherwise
-     *    @return SimpleUrl        Base url.
+     *    @return boolean        Base url.
      *    @access public
      */
     function getBaseUrl() {
@@ -658,7 +658,7 @@ 
     /**
      *    Opens a frameset. A frameset may contain nested
      *    frameset tags.
-     *    @param SimpleFramesetTag $tag      Tag to accept.
+     *    @param SimpleTag $tag      Tag to accept.
      *    @access public
      */
     function acceptFramesetStart(&$tag) {

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

View/SSTemplateParser.php.inc

Doc Comments +7 added lines, -3 removed lines

@@ -114,6 +114,8 @@ 
 
 	/**
 	 * Override the function that constructs the result arrays to also prepare a 'php' item in the array
+	 * @param string $matchrule
+	 * @param string $name
 	 */
 	function construct($matchrule, $name, $arguments = null) {
 		$res = parent::construct($matchrule, $name, $arguments);
@@ -175,9 +177,9 @@ 
 
 	/**
 	 * Ensures that the arguments to addOpenBlock and addClosedBlock are valid
-	 * @param $name
-	 * @param $callable
-	 * @param $type
+	 * @param string $name
+	 * @param callable $callable
+	 * @param string $type
 	 * @throws InvalidArgumentException
 	 */
 	protected function validateExtensionBlock($name, $callable, $type) {
@@ -262,6 +264,7 @@ 
 	 * The basic generated PHP of LookupStep and LastLookupStep is the same, except that LookupStep calls 'obj' to
 	 * get the next ViewableData in the sequence, and LastLookupStep calls different methods (XML_val, hasValue, obj)
 	 * depending on the context the lookup is used in.
+	 * @param string $method
 	 */
 	function Lookup_AddLookupStep(&$res, $sub, $method) {
 		$res['LookupSteps'][] = $sub;
@@ -1174,6 +1177,7 @@ 
 
 	/**
 	 * @param string $code
+	 * @param string $templateName
 	 * @return string $code
 	 */
 	protected function includeDebuggingComments($code, $templateName) {