Completed
Pull Request — master (#5886)
by Damian
11:53
created
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.
ORM/DataList.php 1 patch
Doc Comments   +10 added lines, -10 removed lines patch added patch discarded remove patch
@@ -157,7 +157,7 @@  discard block
 block discarded – undo
157 157
 	 * Returns a new DataList instance with the specified query parameter assigned
158 158
 	 *
159 159
 	 * @param string|array $keyOrArray Either the single key to set, or an array of key value pairs to set
160
-	 * @param mixed $val If $keyOrArray is not an array, this is the value to set
160
+	 * @param string $val If $keyOrArray is not an array, this is the value to set
161 161
 	 * @return DataList
162 162
 	 */
163 163
 	public function setDataQueryParam($keyOrArray, $val = null) {
@@ -520,7 +520,7 @@  discard block
 block discarded – undo
520 520
 	 * Check if the given field specification could be interpreted as an unquoted relation name
521 521
 	 *
522 522
 	 * @param string $field
523
-	 * @return bool
523
+	 * @return integer
524 524
 	 */
525 525
 	protected function isValidRelationName($field) {
526 526
 		return preg_match('/^[A-Z0-9._]+$/i', $field);
@@ -635,7 +635,7 @@  discard block
 block discarded – undo
635 635
 	 * @param int $order A numerical index to control the order that joins are added to the query; lower order values
636 636
 	 * will cause the query to appear first. The default is 20, and joins created automatically by the
637 637
 	 * ORM have a value of 10.
638
-	 * @param array $parameters Any additional parameters if the join is a parameterised subquery
638
+	 * @param string[] $parameters Any additional parameters if the join is a parameterised subquery
639 639
 	 * @return DataList
640 640
 	 */
641 641
 	public function innerJoin($table, $onClause, $alias = null, $order = 20, $parameters = array()) {
@@ -653,7 +653,7 @@  discard block
 block discarded – undo
653 653
 	 * @param int $order A numerical index to control the order that joins are added to the query; lower order values
654 654
 	 * will cause the query to appear first. The default is 20, and joins created automatically by the
655 655
 	 * ORM have a value of 10.
656
-	 * @param array $parameters Any additional parameters if the join is a parameterised subquery
656
+	 * @param string[] $parameters Any additional parameters if the join is a parameterised subquery
657 657
 	 * @return DataList
658 658
 	 */
659 659
 	public function leftJoin($table, $onClause, $alias = null, $order = 20, $parameters = array()) {
@@ -780,7 +780,7 @@  discard block
 block discarded – undo
780 780
 	 * Return the maximum value of the given field in this DataList
781 781
 	 *
782 782
 	 * @param string $fieldName
783
-	 * @return mixed
783
+	 * @return string
784 784
 	 */
785 785
 	public function max($fieldName) {
786 786
 		return $this->dataQuery->max($fieldName);
@@ -790,7 +790,7 @@  discard block
 block discarded – undo
790 790
 	 * Return the minimum value of the given field in this DataList
791 791
 	 *
792 792
 	 * @param string $fieldName
793
-	 * @return mixed
793
+	 * @return string
794 794
 	 */
795 795
 	public function min($fieldName) {
796 796
 		return $this->dataQuery->min($fieldName);
@@ -800,7 +800,7 @@  discard block
 block discarded – undo
800 800
 	 * Return the average value of the given field in this DataList
801 801
 	 *
802 802
 	 * @param string $fieldName
803
-	 * @return mixed
803
+	 * @return string
804 804
 	 */
805 805
 	public function avg($fieldName) {
806 806
 		return $this->dataQuery->avg($fieldName);
@@ -810,7 +810,7 @@  discard block
 block discarded – undo
810 810
 	 * Return the sum of the values of the given field in this DataList
811 811
 	 *
812 812
 	 * @param string $fieldName
813
-	 * @return mixed
813
+	 * @return string
814 814
 	 */
815 815
 	public function sum($fieldName) {
816 816
 		return $this->dataQuery->sum($fieldName);
@@ -862,7 +862,7 @@  discard block
 block discarded – undo
862 862
 	/**
863 863
 	 * Restrict the columns to fetch into this DataList
864 864
 	 *
865
-	 * @param array $queriedColumns
865
+	 * @param string[] $queriedColumns
866 866
 	 * @return DataList
867 867
 	 */
868 868
 	public function setQueriedColumns($queriedColumns) {
@@ -1135,7 +1135,7 @@  discard block
 block discarded – undo
1135 1135
 	/**
1136 1136
 	 * Returns item stored in list with index $key
1137 1137
 	 *
1138
-	 * @param mixed $key
1138
+	 * @param integer $key
1139 1139
 	 * @return DataObject
1140 1140
 	 */
1141 1141
 	public function offsetGet($key) {
Please login to merge, or discard this patch.