Completed
Pull Request — master (#5846)
by Hamish
19:13 queued 08:18
created
core/Convert.php 1 patch
Doc Comments   +4 added lines, -3 removed lines patch added patch discarded remove patch
@@ -50,7 +50,7 @@  discard block
 block discarded – undo
50 50
 	 *
51 51
 	 * @param array|string $val String to escape, or array of strings
52 52
 	 *
53
-	 * @return array|string
53
+	 * @return string
54 54
 	 */
55 55
 	public static function raw2htmlname($val) {
56 56
 		if(is_array($val)) {
@@ -176,7 +176,7 @@  discard block
 block discarded – undo
176 176
 	 * table, or column name. Supports encoding of multi identfiers separated by
177 177
 	 * a delimiter (e.g. ".")
178 178
 	 *
179
-	 * @param string|array $identifier The identifier to escape. E.g. 'SiteTree.Title' or list of identifiers
179
+	 * @param string $identifier The identifier to escape. E.g. 'SiteTree.Title' or list of identifiers
180 180
 	 * to be joined via the separator.
181 181
 	 * @param string $separator The string that delimits subsequent identifiers
182 182
 	 * @return string The escaped identifier. E.g. '"SiteTree"."Title"'
@@ -257,6 +257,7 @@  discard block
 block discarded – undo
257 257
 	 * call this function directly, Please use {@link Convert::xml2array()}
258 258
 	 *
259 259
 	 * @param SimpleXMLElement
260
+	 * @param SimpleXMLElement $xml
260 261
 	 *
261 262
 	 * @return mixed
262 263
 	 */
@@ -450,7 +451,7 @@  discard block
 block discarded – undo
450 451
 	 * - IDField => idField
451 452
 	 * - iDField => iDField
452 453
 	 *
453
-	 * @param $str
454
+	 * @param string $str
454 455
 	 * @return string
455 456
 	 */
456 457
 	public static function upperCamelToLowerCamel($str) {
Please login to merge, or discard this patch.
filesystem/FileMigrationHelper.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -89,7 +89,7 @@
 block discarded – undo
89 89
 	/**
90 90
 	 * Get list of File dataobjects to import
91 91
 	 *
92
-	 * @return DataList
92
+	 * @return SilverStripe\ORM\DataList
93 93
 	 */
94 94
 	protected function getFileQuery() {
95 95
 		// Select all records which have a Filename value, but not FileFilename.
Please login to merge, or discard this patch.
filesystem/Folder.php 1 patch
Doc Comments   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -47,7 +47,7 @@  discard block
 block discarded – undo
47 47
 	 * Find the given folder or create it as a database record
48 48
 	 *
49 49
 	 * @param string $folderPath Directory path relative to assets root
50
-	 * @return Folder|null
50
+	 * @return null|SilverStripe\ORM\DataObject
51 51
 	 */
52 52
 	public static function find_or_make($folderPath) {
53 53
 		// replace leading and trailing slashes
@@ -149,7 +149,7 @@  discard block
 block discarded – undo
149 149
 	/**
150 150
 	 * Returns all children of this folder
151 151
 	 *
152
-	 * @return DataList
152
+	 * @return SilverStripe\ORM\DataList
153 153
 	 */
154 154
 	public function myChildren() {
155 155
 		return File::get()->filter("ParentID", $this->ID);
@@ -200,7 +200,7 @@  discard block
 block discarded – undo
200 200
 	/**
201 201
 	 * Get the children of this folder that are also folders.
202 202
 	 *
203
-	 * @return DataList
203
+	 * @return SilverStripe\ORM\DataList
204 204
 	 */
205 205
 	public function ChildFolders() {
206 206
 		return Folder::get()->filter('ParentID', $this->ID);
Please login to merge, or discard this patch.
filesystem/ImageManipulation.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -513,7 +513,7 @@
 block discarded – undo
513 513
 	/**
514 514
 	 * Get HTML for img containing the icon for this file
515 515
 	 *
516
-	 * @return DBHTMLText
516
+	 * @return DBField
517 517
 	 */
518 518
 	public function IconTag() {
519 519
 		return DBField::create_field(
Please login to merge, or discard this patch.
forms/htmleditor/HTMLEditorField.php 1 patch
Doc Comments   +13 added lines, -6 removed lines patch added patch discarded remove patch
@@ -63,7 +63,7 @@  discard block
 block discarded – undo
63 63
 	/**
64 64
 	 * Assign a new configuration instance or identifier
65 65
 	 *
66
-	 * @param string|HTMLEditorConfig $config
66
+	 * @param string $config
67 67
 	 * @return $this
68 68
 	 */
69 69
 	public function setEditorConfig($config) {
@@ -77,7 +77,7 @@  discard block
 block discarded – undo
77 77
 	 *
78 78
 	 * @param string $name The internal field name, passed to forms.
79 79
 	 * @param string $title The human-readable field label.
80
-	 * @param mixed $value The value of the field.
80
+	 * @param integer $value The value of the field.
81 81
 	 * @param string $config HTMLEditorConfig identifier to be used. Default to the active one.
82 82
 	 */
83 83
 	public function __construct($name, $title = null, $value = '', $config = null) {
@@ -118,6 +118,9 @@  discard block
 block discarded – undo
118 118
 		$record->{$this->name} = $htmlValue->getContent();
119 119
 	}
120 120
 
121
+	/**
122
+	 * @param string|null $value
123
+	 */
121 124
 	public function setValue($value) {
122 125
 		// Regenerate links prior to preview, so that the editor can see them.
123 126
 		$value = Image::regenerate_html_links($value);
@@ -185,6 +188,10 @@  discard block
 block discarded – undo
185 188
 
186 189
 	protected $controller, $name;
187 190
 
191
+	/**
192
+	 * @param Controller|null $controller
193
+	 * @param string $name
194
+	 */
188 195
 	public function __construct($controller, $name) {
189 196
 		parent::__construct();
190 197
 
@@ -203,7 +210,7 @@  discard block
 block discarded – undo
203 210
 	/**
204 211
 	 * Searches the SiteTree for display in the dropdown
205 212
 	 *
206
-	 * @return callback
213
+	 * @return SilverStripe\ORM\DataList
207 214
 	 */
208 215
 	public function siteTreeSearchCallback($sourceObject, $labelField, $search) {
209 216
 		return DataObject::get($sourceObject)->filterAny(array(
@@ -583,7 +590,7 @@  discard block
 block discarded – undo
583 590
 	/**
584 591
 	 * Find all anchors available on the given page.
585 592
 	 *
586
-	 * @return array
593
+	 * @return string
587 594
 	 * @throws SS_HTTPResponse_Exception
588 595
 	 */
589 596
 	public function getanchors() {
@@ -646,7 +653,7 @@  discard block
 block discarded – undo
646 653
 	 * Gets files filtered by a given parent with the allowed extensions
647 654
 	 *
648 655
 	 * @param int $parentID
649
-	 * @return DataList
656
+	 * @return SilverStripe\ORM\DataList
650 657
 	 */
651 658
 	protected function getFiles($parentID = null) {
652 659
 		$exts = $this->getAllowedExtensions();
@@ -1159,7 +1166,7 @@  discard block
 block discarded – undo
1159 1166
 	}
1160 1167
 
1161 1168
 	/**
1162
-	 * @return AdapterInterface
1169
+	 * @return Embed
1163 1170
 	 */
1164 1171
 	public function getEmbed() {
1165 1172
 		return $this->embed;
Please login to merge, or discard this patch.
ORM/Connect/MySQLSchemaManager.php 1 patch
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -115,6 +115,9 @@
 block discarded – undo
115 115
 		$this->query("ALTER TABLE \"$tableName\" $alterations");
116 116
 	}
117 117
 
118
+	/**
119
+	 * @param string $tableName
120
+	 */
118 121
 	public function isView($tableName) {
119 122
 		$info = $this->query("SHOW /*!50002 FULL*/ TABLES LIKE '$tableName'")->record();
120 123
 		return $info && strtoupper($info['Table_type']) == 'VIEW';
Please login to merge, or discard this patch.
ORM/Connect/MySQLStatement.php 1 patch
Doc Comments   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -78,8 +78,8 @@
 block discarded – undo
78 78
 
79 79
 	/**
80 80
 	 * Hook the result-set given into a Query class, suitable for use by SilverStripe.
81
-	 * @param mysqli_stmt $statement The related statement, if present
82
-	 * @param mysqli_result $metadata The metadata for this statement
81
+	 * @param \mysqli_stmt $statement The related statement, if present
82
+	 * @param \mysqli_result $metadata The metadata for this statement
83 83
 	 */
84 84
 	public function __construct($statement, $metadata) {
85 85
 		$this->statement = $statement;
Please login to merge, or discard this patch.
ORM/Connect/PDOConnector.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -324,7 +324,7 @@
 block discarded – undo
324 324
 	 * @param int $errorLevel
325 325
 	 * @param string $sql
326 326
 	 * @param array $parameters
327
-	 * @return \PDOQuery
327
+	 * @return PDOQuery|null
328 328
 	 */
329 329
 	protected function prepareResults($statement, $errorLevel, $sql, $parameters = array()) {
330 330
 
Please login to merge, or discard this patch.
ORM/DatabaseAdmin.php 1 patch
Doc Comments   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -107,7 +107,7 @@  discard block
 block discarded – undo
107 107
 	/**
108 108
 	 * Gets the url to return to after build
109 109
 	 *
110
-	 * @return string|null
110
+	 * @return string
111 111
 	 */
112 112
 	protected function getReturnURL() {
113 113
 		$url = $this->request->getVar('returnURL');
@@ -152,7 +152,7 @@  discard block
 block discarded – undo
152 152
 	/**
153 153
 	 * Returns the timestamp of the time that the database was last built
154 154
 	 *
155
-	 * @return string Returns the timestamp of the time that the database was
155
+	 * @return integer|null Returns the timestamp of the time that the database was
156 156
 	 *                last built
157 157
 	 */
158 158
 	public static function lastBuilt() {
Please login to merge, or discard this patch.