Completed
Pull Request — master (#6337)
by Daniel
08:58
created
src/Assets/Upload.php 1 patch
Doc Comments   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -114,7 +114,7 @@  discard block
 block discarded – undo
114 114
 	 * Set a different instance than {@link Upload_Validator}
115 115
 	 * for this upload session.
116 116
 	 *
117
-	 * @param object $validator
117
+	 * @param Upload_Validator $validator
118 118
 	 */
119 119
 	public function setValidator($validator) {
120 120
 		$this->validator = $validator;
@@ -363,7 +363,7 @@  discard block
 block discarded – undo
363 363
 	/**
364 364
 	 * Determines wether previous operations caused an error.
365 365
 	 *
366
-	 * @return boolean
366
+	 * @return integer
367 367
 	 */
368 368
 	public function isError() {
369 369
 		return (count($this->errors));
Please login to merge, or discard this patch.
src/Control/Controller.php 1 patch
Doc Comments   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -142,6 +142,7 @@  discard block
 block discarded – undo
142 142
 	 * {@inheritdoc}
143 143
 	 *
144 144
 	 * Also set the URLParams
145
+	 * @param HTTPRequest $request
145 146
 	 */
146 147
 	public function setRequest($request) {
147 148
 		$return = parent::setRequest($request);
@@ -501,7 +502,7 @@  discard block
 block discarded – undo
501 502
 	 *
502 503
 	 * @param array $params
503 504
 	 *
504
-	 * @return string
505
+	 * @return DBHTMLText
505 506
 	 */
506 507
 	public function render($params = null) {
507 508
 		$template = $this->getViewer($this->getAction());
@@ -664,7 +665,7 @@  discard block
 block discarded – undo
664 665
 	 * Tests whether a redirection has been requested. If redirect() has been called, it will return
665 666
 	 * the URL redirected to. Otherwise, it will return null.
666 667
 	 *
667
-	 * @return null|string
668
+	 * @return boolean
668 669
 	 */
669 670
 	public function redirectedTo() {
670 671
 		return $this->getResponse() && $this->getResponse()->getHeader('Location');
@@ -695,7 +696,6 @@  discard block
 block discarded – undo
695 696
 	 *
696 697
 	 * Caution: All parameters are expected to be URI-encoded already.
697 698
 	 *
698
-	 * @param string|array $arg,.. One or more link segments, or list of link segments as an array
699 699
 	 * @return string
700 700
 	 */
701 701
 	public static function join_links($arg = null) {
Please login to merge, or discard this patch.
src/Control/CookieJar.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -147,7 +147,7 @@
 block discarded – undo
147 147
 	 * @see http://uk3.php.net/manual/en/function.setcookie.php
148 148
 	 *
149 149
 	 * @param string $name The name of the cookie
150
-	 * @param string|array $value The value for the cookie to hold
150
+	 * @param false|string $value The value for the cookie to hold
151 151
 	 * @param int $expiry The number of days until expiry
152 152
 	 * @param string $path The path to save the cookie on (falls back to site base)
153 153
 	 * @param string $domain The domain to make the cookie available on
Please login to merge, or discard this patch.
src/Control/Director.php 1 patch
Doc Comments   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -534,7 +534,7 @@  discard block
 block discarded – undo
534 534
 	 * Returns the domain part of the URL 'http://www.mysite.com'. Returns FALSE is this environment
535 535
 	 * variable isn't set.
536 536
 	 *
537
-	 * @return bool|string
537
+	 * @return string|false
538 538
 	 */
539 539
 	public static function protocolAndHost() {
540 540
 		$alternate = Config::inst()->get('Director', 'alternate_base_url');
@@ -869,7 +869,7 @@  discard block
 block discarded – undo
869 869
 	/**
870 870
 	 * Returns true if the given file exists. Filename should be relative to the site root.
871 871
 	 *
872
-	 * @param $file
872
+	 * @param string $file
873 873
 	 *
874 874
 	 * @return bool
875 875
 	 */
@@ -1087,7 +1087,7 @@  discard block
 block discarded – undo
1087 1087
 	 * Can also be checked with {@link Director::isDev()}, {@link Director::isTest()}, and
1088 1088
 	 * {@link Director::isLive()}.
1089 1089
 	 *
1090
-	 * @return bool|string
1090
+	 * @return string|false
1091 1091
 	 */
1092 1092
 	public static function get_environment_type() {
1093 1093
 		if(Director::isLive()) {
Please login to merge, or discard this patch.
src/Control/Email/Email.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -390,7 +390,7 @@
 block discarded – undo
390 390
 	}
391 391
 
392 392
 	/**
393
-	 * @return Email|ViewableData_Customised
393
+	 * @return ViewableData
394 394
 	 */
395 395
 	protected function templateData() {
396 396
 		if($this->template_data) {
Please login to merge, or discard this patch.
src/Control/Email/Mailer.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -321,7 +321,7 @@
 block discarded – undo
321 321
 	/**
322 322
 	 * Encode an array of parts using multipart
323 323
 	 *
324
-	 * @param array $parts List of parts
324
+	 * @param string[] $parts List of parts
325 325
 	 * @param string $contentType Content-type of parts
326 326
 	 * @param array $headers Existing headers to include in response
327 327
 	 * @return array Array with two items, the body followed by headers
Please login to merge, or discard this patch.
src/Control/HTTPRequest.php 1 patch
Doc Comments   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -427,9 +427,9 @@
 block discarded – undo
427 427
 	 * it's only advisable to send small files through this method.
428 428
 	 *
429 429
 	 * @static
430
-	 * @param $fileData
430
+	 * @param string $fileData
431 431
 	 * @param $fileName
432
-	 * @param null $mimeType
432
+	 * @param string $mimeType
433 433
 	 * @return SS_HTTPResponse
434 434
 	 */
435 435
 	public static function send_file($fileData, $fileName, $mimeType = null) {
Please login to merge, or discard this patch.
src/Control/RequestHandler.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -289,7 +289,7 @@
 block discarded – undo
289 289
 	 *
290 290
 	 * Must not raise HTTPResponse_Exceptions - instead it should return
291 291
 	 *
292
-	 * @param $request
292
+	 * @param HTTPRequest $request
293 293
 	 * @param $action
294 294
 	 * @return HTTPResponse
295 295
 	 */
Please login to merge, or discard this patch.
src/Control/RSS/RSSFeed.php 1 patch
Doc Comments   +1 added lines patch added patch discarded remove patch
@@ -227,6 +227,7 @@
 block discarded – undo
227 227
 	 * via the standard template inclusion process.
228 228
 	 *
229 229
 	 * @param string
230
+	 * @param string|null $template
230 231
 	 */
231 232
 	public function setTemplate($template) {
232 233
 		$this->template = $template;
Please login to merge, or discard this patch.