Set method visibility for Job::run implementation

Change-Id: Iee7ccd8765542f243d3555abbe805d75d4b1ea2c

.phpcs.xml

@@ -33,7 +33,6 @@
 		<exclude-pattern>includes/htmlform/</exclude-pattern>
 		<exclude-pattern>includes/import/</exclude-pattern>
 		<exclude-pattern>includes/installer/</exclude-pattern>
-		<exclude-pattern>includes/jobqueue/</exclude-pattern>
 		<exclude-pattern>includes/language/</exclude-pattern>
 		<exclude-pattern>includes/libs/</exclude-pattern>
 		<exclude-pattern>includes/logging/</exclude-pattern>

includes/jobqueue/jobs/DeleteLinksJob.php

@@ -37,7 +37,7 @@ class DeleteLinksJob extends Job {
 		$this->removeDuplicates = true;
 	}
 
-	function run() {
+	public function run() {
 		if ( is_null( $this->title ) ) {
 			$this->setLastError( "deleteLinks: Invalid title" );
 			return false;

includes/jobqueue/jobs/DoubleRedirectJob.php

@@ -96,7 +96,7 @@ class DoubleRedirectJob extends Job {
 	/**
 	 * @return bool
 	 */
-	function run() {
+	public function run() {
 		if ( !$this->redirTitle ) {
 			$this->setLastError( 'Invalid title' );
 
@@ -239,7 +239,7 @@ class DoubleRedirectJob extends Job {
 	 *
 	 * @return User|bool
 	 */
-	function getUser() {
+	private function getUser() {
 		if ( !self::$user ) {
 			$username = wfMessage( 'double-redirect-fixer' )->inContentLanguage()->text();
 			self::$user = User::newFromName( $username );

includes/jobqueue/jobs/EmaillingJob.php

@@ -32,7 +32,7 @@ class EmaillingJob extends Job {
 		parent::__construct( 'sendMail', Title::newMainPage(), $params );
 	}
 
-	function run() {
+	public function run() {
 		$status = UserMailer::send(
 			$this->params['to'],
 			$this->params['from'],

includes/jobqueue/jobs/EnotifNotifyJob.php

@@ -31,7 +31,7 @@ class EnotifNotifyJob extends Job {
 		parent::__construct( 'enotifNotify', $title, $params );
 	}
 
-	function run() {
+	public function run() {
 		$enotif = new EmailNotification();
 		// Get the user from ID (rename safe). Anons are 0, so defer to name.
 		if ( isset( $this->params['editorID'] ) && $this->params['editorID'] ) {

includes/jobqueue/jobs/HTMLCacheUpdateJob.php

@@ -68,7 +68,7 @@ class HTMLCacheUpdateJob extends Job {
 		);
 	}
 
-	function run() {
+	public function run() {
 		global $wgUpdateRowsPerJob, $wgUpdateRowsPerQuery;
 
 		if ( isset( $this->params['table'] ) && !isset( $this->params['pages'] ) ) {

includes/jobqueue/jobs/RefreshLinksJob.php

@@ -84,7 +84,7 @@ class RefreshLinksJob extends Job {
 		return $job;
 	}
 
-	function run() {
+	public function run() {
 		$ok = true;
 
 		// Job to update all (or a range of) backlink pages for a page