silverstripe / silverstripe-framework

forms/HiddenField.php

Doc Comments +1 added lines, -1 removed lines

@@ -10,7 +10,7 @@
 	/**
 	 * @param array $properties
 	 *
-	 * @return HTMLText
+	 * @return string
 	 */
 	public function FieldHolder($properties = array()) {
 		return $this->Field($properties);

forms/HtmlEditorSanitiser.php

Doc Comments +2 added lines, -1 removed lines

@@ -39,7 +39,7 @@ 
 	/**
 	 * Given a TinyMCE pattern (close to unix glob style), create a regex that does the match
 	 *
-	 * @param $str - The TinyMCE pattern
+	 * @param string|null $str - The TinyMCE pattern
 	 * @return string - The equivalent regex
 	 */
 	protected function patternToRegex($str) {
@@ -180,6 +180,7 @@ 
 	/**
 	 * Given an attribute name, return the rule structure for that attribute
 	 * @param string $name - The attribute name
+	 * @param stdClass $elementRule
 	 * @return stdClass - The attribute rule
 	 */
 	protected function getRuleForAttribute($elementRule, $name) {

forms/InlineFormAction.php

Doc Comments +1 added lines, -1 removed lines

@@ -14,7 +14,7 @@
 
 	/**
 	 * Create a new action button.
-	 * @param action The method to call when the button is clicked
+	 * @param action string method to call when the button is clicked
 	 * @param title The label on the button
 	 * @param extraClass A CSS class to apply to the button in addition to 'action'
 	 */

forms/NestedForm.php

Doc Comments +1 added lines, -1 removed lines

@@ -11,7 +11,7 @@
 
 	/**
 	 * Represent the given form in a tabular style
-	 * @param form The form to decorate.
+	 * @param form Form form to decorate.
 	 */
 	public function __construct(Form $form) {
 		$this->form = $form;

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