From 12755711e5e0d6ea25a992f892c4112a7118a2a1 Mon Sep 17 00:00:00 2001 From: Aaron Schulz Date: Fri, 29 Mar 2019 11:40:59 -0700 Subject: [PATCH] jobqueue: clean up __construct() signature for some job classes The $params value should always be an array Change-Id: If7a0350b64241c7b5bbbdbc7bc90379a93484013 --- includes/jobqueue/jobs/DeletePageJob.php | 2 +- includes/jobqueue/jobs/UserGroupExpiryJob.php | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) 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; } -- 2.20.1