silverstripe / silverstripe-framework

forms/PhoneNumberField.php

Doc Comments +4 added lines, -1 removed lines

@@ -29,7 +29,7 @@ 
 
 	/**
 	 * @param array $properties
-	 * @return FieldGroup|HTMLText
+	 * @return FieldGroup
 	 */
 	public function Field($properties = array()) {
 		$fields = new FieldGroup( $this->name );
@@ -73,6 +73,9 @@ 
 		return $this;
 	}
 
+	/**
+	 * @return string
+	 */
 	public static function joinPhoneNumber( $value ) {
 		if( is_array( $value ) ) {
 			$completeNumber = '';

forms/RequiredFields.php

Doc Comments +1 added lines

@@ -176,6 +176,7 @@
 	 * Add {@link RequiredField} objects together
 	 *
 	 * @param RequiredFields
+	 * @param RequiredFields $requiredFields
 	 *
 	 * @return RequiredFields
 	 */

forms/SelectionGroup.php

Doc Comments +1 added lines, -1 removed lines

@@ -134,7 +134,7 @@
 
 	/**
 	 * @param String $value Form field identifier
-	 * @param FormField $field Contents of the option
+	 * @param FormField $fields Contents of the option
 	 * @param String $title Title to show for the radio button option
 	 */
 	function __construct($value, $fields = null, $title = null) {

forms/Tab.php

Doc Comments +3 added lines, -2 removed lines

@@ -23,8 +23,6 @@ 
 	 * @uses FormField::name_to_label()
 	 *
 	 * @param string $name Identifier of the tab, without characters like dots or spaces
-	 * @param string $title Natural language title of the tab. If its left out,
-	 *  the class uses {@link FormField::name_to_label()} to produce a title from the {@link $name} parameter.
 	 * @param FormField All following parameters are inserted as children to this tab
 	 */
 	public function __construct($name) {
@@ -54,6 +52,9 @@ 
 		return $this->children;
 	}
 
+	/**
+	 * @param TabSet $val
+	 */
 	public function setTabSet($val) {
 		$this->tabSet = $val;
 		return $this;

forms/TabSet.php

Doc Comments -2 removed lines

@@ -29,8 +29,6 @@
 
 	/**
 	 * @param string $name Identifier
-	 * @param string $title (Optional) Natural language title of the tabset
-	 * @param Tab|TabSet $unknown All further parameters are inserted as children into the TabSet
 	 */
 	public function __construct($name) {
 		$args = func_get_args();

forms/TextField.php

Doc Comments +1 added lines, -1 removed lines

@@ -45,7 +45,7 @@
 	}
 
 	/**
-	 * @return null|int
+	 * @return integer
 	 */
 	public function getMaxLength() {
 		return $this->maxLength;

forms/TimeField.php

Doc Comments +7 added lines

@@ -50,6 +50,10 @@ 
 	 */
 	protected $valueObj = null;
 
+	/**
+	 * @param string $name
+	 * @param string $title
+	 */
 	public function __construct($name, $title = null, $value = ""){
 		if(!$this->locale) {
 			$this->locale = i18n::get_locale();
@@ -228,6 +232,9 @@ 
 		return $this->castedCopy('TimeField_Readonly');
 	}
 
+	/**
+	 * @param string $class
+	 */
 	public function castedCopy($class) {
 		$copy = parent::castedCopy($class);
 		if($copy->hasMethod('setConfig')) {

forms/TreeDropdownField.php

Doc Comments +2 added lines, -3 removed lines

@@ -76,7 +76,7 @@ 
 	 *
 	 * @param string $name the field name
 	 * @param string $title the field label
-	 * @param string|array $sourceObject The object-type to list in the tree. This could
+	 * @param string $sourceObject The object-type to list in the tree. This could
 	 * be one of the following:
 	 * - A DataObject class name with the {@link Hierarchy} extension.
 	 * - An array of key/value pairs, like a {@link DropdownField} source. In
@@ -385,7 +385,7 @@ 
 	 * If a filter function has been set, that will be called. And if search text is set,
 	 * filter on that too. Return true if all applicable conditions are true, false otherwise.
 	 * @param $node
-	 * @return unknown_type
+	 * @return boolean
 	 */
 	public function filterMarking($node) {
 		if ($this->filterCallback && !call_user_func($this->filterCallback, $node)) return false;
@@ -436,7 +436,6 @@ 
 	}
 
 	/**
-	 * @param String $field
 	 */
 	public function setSourceObject($class) {
 		$this->sourceObject = $class;

forms/UploadField.php

Doc Comments +2 added lines, -3 removed lines

@@ -211,7 +211,6 @@ 
 	 * @param string $title The field label.
 	 * @param SS_List $items If no items are defined, the field will try to auto-detect an existing relation on
 	 *                       @link $record}, with the same name as the field name.
-	 * @param Form $form Reference to the container form
 	 */
 	public function __construct($name, $title = null, SS_List $items = null) {
 
@@ -1078,7 +1077,7 @@ 
 
 	/**
 	 * @param SS_HTTPRequest $request
-	 * @return UploadField_ItemHandler
+	 * @return UploadField_SelectHandler
 	 */
 	public function handleSelect(SS_HTTPRequest $request) {
 		if(!$this->canAttachExisting()) return $this->httpError(403);
@@ -1434,7 +1433,7 @@ 
 	 * Action to handle editing of a single file
 	 *
 	 * @param SS_HTTPRequest $request
-	 * @return ViewableData_Customised
+	 * @return HTMLText
 	 */
 	public function edit(SS_HTTPRequest $request) {
 		// Check form field state