From: Aaron Schulz <aschulz@wikimedia.org>
Date: Fri, 29 Mar 2019 18:40:59 +0000 (-0700)
Subject: jobqueue: clean up __construct() signature for some job classes
X-Git-Tag: 1.34.0-rc.0~2123^2
X-Git-Url: http://git.cyclocoop.org/%24action?a=commitdiff_plain;h=12755711e5e0d6ea25a992f892c4112a7118a2a1;p=lhc%2Fweb%2Fwiklou.git

jobqueue: clean up __construct() signature for some job classes

The $params value should always be an array

Change-Id: If7a0350b64241c7b5bbbdbc7bc90379a93484013
---

diff --git a/includes/jobqueue/jobs/DeletePageJob.php b/includes/jobqueue/jobs/DeletePageJob.php
index 9b5cef44ea..e6dfae497e 100644
--- a/includes/jobqueue/jobs/DeletePageJob.php
+++ b/includes/jobqueue/jobs/DeletePageJob.php
@@ -4,7 +4,7 @@
  * Class DeletePageJob
  */
 class DeletePageJob extends Job {
-	public function __construct( $title, $params ) {
+	public function __construct( $title, $params = [] ) {
 		parent::__construct( 'deletePage', $title, $params );
 	}
 
diff --git a/includes/jobqueue/jobs/UserGroupExpiryJob.php b/includes/jobqueue/jobs/UserGroupExpiryJob.php
index 0945e58f16..bd0df5b276 100644
--- a/includes/jobqueue/jobs/UserGroupExpiryJob.php
+++ b/includes/jobqueue/jobs/UserGroupExpiryJob.php
@@ -22,7 +22,7 @@
  */
 
 class UserGroupExpiryJob extends Job {
-	public function __construct( $params = false ) {
+	public function __construct( $params = [] ) {
 		parent::__construct( 'userGroupExpiry', Title::newMainPage(), $params );
 		$this->removeDuplicates = true;
 	}