@@ -942,7 +942,7 @@ discard block |
||
| 942 | 942 | /** |
| 943 | 943 | * Returns true if the given file exists. Filename should be relative to the site root. |
| 944 | 944 | * |
| 945 | - * @param $file |
|
| 945 | + * @param string $file |
|
| 946 | 946 | * |
| 947 | 947 | * @return bool |
| 948 | 948 | */ |
@@ -1175,7 +1175,7 @@ discard block |
||
| 1175 | 1175 | * Can also be checked with {@link Director::isDev()}, {@link Director::isTest()}, and |
| 1176 | 1176 | * {@link Director::isLive()}. |
| 1177 | 1177 | * |
| 1178 | - * @return bool|string |
|
| 1178 | + * @return string|false |
|
| 1179 | 1179 | */ |
| 1180 | 1180 | public static function get_environment_type() |
| 1181 | 1181 | { |
@@ -117,7 +117,7 @@ discard block |
||
| 117 | 117 | * @param array $attributes an array of attributes to include on the link. |
| 118 | 118 | * @param string $iconClass |
| 119 | 119 | * |
| 120 | - * @return boolean The result of the operation. |
|
| 120 | + * @return boolean|null The result of the operation. |
|
| 121 | 121 | */ |
| 122 | 122 | public static function add_link($code, $menuTitle, $url, $priority = -1, $attributes = null, $iconClass = null) |
| 123 | 123 | { |
@@ -140,7 +140,7 @@ discard block |
||
| 140 | 140 | * @param array $attributes an array of attributes to include on the link. |
| 141 | 141 | * @param string $iconClass |
| 142 | 142 | * |
| 143 | - * @return bool Success |
|
| 143 | + * @return boolean|null Success |
|
| 144 | 144 | */ |
| 145 | 145 | public static function add_menu_item( |
| 146 | 146 | $code, |
@@ -322,7 +322,7 @@ discard block |
||
| 322 | 322 | * @param array $attributes an array of attributes to include on the link. |
| 323 | 323 | * @param string $iconClass |
| 324 | 324 | * |
| 325 | - * @return bool Success |
|
| 325 | + * @return boolean|null Success |
|
| 326 | 326 | */ |
| 327 | 327 | public static function replace_menu_item( |
| 328 | 328 | $code, |
@@ -264,7 +264,7 @@ discard block |
||
| 264 | 264 | /** |
| 265 | 265 | * Gets the combined configuration of all LeafAndMain subclasses required by the client app. |
| 266 | 266 | * |
| 267 | - * @return array |
|
| 267 | + * @return string |
|
| 268 | 268 | * |
| 269 | 269 | * WARNING: Experimental API |
| 270 | 270 | */ |
@@ -691,7 +691,7 @@ discard block |
||
| 691 | 691 | * |
| 692 | 692 | * @param string $url |
| 693 | 693 | * @param int $code |
| 694 | - * @return HTTPResponse|string |
|
| 694 | + * @return string|null |
|
| 695 | 695 | */ |
| 696 | 696 | public function redirect($url, $code = 302) |
| 697 | 697 | { |
@@ -1006,7 +1006,7 @@ discard block |
||
| 1006 | 1006 | * {@link SSViewer::get_templates_by_class()} |
| 1007 | 1007 | * |
| 1008 | 1008 | * @param string $suffix |
| 1009 | - * @return array |
|
| 1009 | + * @return string |
|
| 1010 | 1010 | */ |
| 1011 | 1011 | public function getTemplatesWithSuffix($suffix) |
| 1012 | 1012 | { |
@@ -2159,7 +2159,7 @@ discard block |
||
| 2159 | 2159 | } |
| 2160 | 2160 | |
| 2161 | 2161 | /** |
| 2162 | - * @return String |
|
| 2162 | + * @return DBField |
|
| 2163 | 2163 | */ |
| 2164 | 2164 | public function Locale() |
| 2165 | 2165 | { |
@@ -92,6 +92,9 @@ |
||
| 92 | 92 | return $return; |
| 93 | 93 | } |
| 94 | 94 | |
| 95 | + /** |
|
| 96 | + * @param boolean $index |
|
| 97 | + */ |
|
| 95 | 98 | function f($index) { |
| 96 | 99 | return stripslashes($this->row[$index]); |
| 97 | 100 | } |
@@ -92,6 +92,9 @@ |
||
| 92 | 92 | return $return; |
| 93 | 93 | } |
| 94 | 94 | |
| 95 | + /** |
|
| 96 | + * @param boolean $index |
|
| 97 | + */ |
|
| 95 | 98 | function f($index) { |
| 96 | 99 | return stripslashes($this->row[$index]); |
| 97 | 100 | } |
@@ -42,6 +42,9 @@ |
||
| 42 | 42 | ] |
| 43 | 43 | ); |
| 44 | 44 | |
| 45 | + /** |
|
| 46 | + * @param string $root |
|
| 47 | + */ |
|
| 45 | 48 | public function __construct($root = null, $writeFlags = LOCK_EX, $linkHandling = self::DISALLOW_LINKS) |
| 46 | 49 | { |
| 47 | 50 | // Get root path, and ensure that this exists and is safe |
@@ -89,6 +89,9 @@ discard block |
||
| 89 | 89 | $this->sequentialSaveableSet = null; |
| 90 | 90 | } |
| 91 | 91 | |
| 92 | + /** |
|
| 93 | + * @param FormField[] $list |
|
| 94 | + */ |
|
| 92 | 95 | protected function collateDataFields(&$list, $saveableOnly = false) |
| 93 | 96 | { |
| 94 | 97 | if (!isset($list)) { |
@@ -198,7 +201,7 @@ discard block |
||
| 198 | 201 | * Removes a number of fields from a Tab/TabSet within this FieldList. |
| 199 | 202 | * |
| 200 | 203 | * @param string $tabName The name of the Tab or TabSet field |
| 201 | - * @param array $fields A list of fields, e.g. array('Name', 'Email') |
|
| 204 | + * @param string[] $fields A list of fields, e.g. array('Name', 'Email') |
|
| 202 | 205 | */ |
| 203 | 206 | public function removeFieldsFromTab($tabName, $fields) |
| 204 | 207 | { |
@@ -625,7 +628,7 @@ discard block |
||
| 625 | 628 | } |
| 626 | 629 | |
| 627 | 630 | /** |
| 628 | - * @param $field |
|
| 631 | + * @param CompositeField $field |
|
| 629 | 632 | * @return $this |
| 630 | 633 | */ |
| 631 | 634 | public function setContainerField($field) |
@@ -707,6 +710,7 @@ discard block |
||
| 707 | 710 | * the children collection. Doesn't work recursively. |
| 708 | 711 | * |
| 709 | 712 | * @param string|FormField |
| 713 | + * @param string $field |
|
| 710 | 714 | * @return int Position in children collection (first position starts with 0). |
| 711 | 715 | * Returns FALSE if the field can't be found. |
| 712 | 716 | */ |
@@ -77,7 +77,7 @@ |
||
| 77 | 77 | * |
| 78 | 78 | * @param GridField |
| 79 | 79 | * |
| 80 | - * @return array |
|
| 80 | + * @return string[] |
|
| 81 | 81 | */ |
| 82 | 82 | public function getActions($gridField) |
| 83 | 83 | { |
@@ -77,7 +77,7 @@ |
||
| 77 | 77 | |
| 78 | 78 | |
| 79 | 79 | /** |
| 80 | - * @return mixed|string |
|
| 80 | + * @return string |
|
| 81 | 81 | */ |
| 82 | 82 | public function Value() |
| 83 | 83 | { |