Set method visibility on maintenance scripts

Change-Id: I44c82fbe65e1d002803ce065df6563f06dd39cd4

4 files changed, 22 insertions, 22 deletions

diff --git a/maintenance/language/StatOutputs.php b/maintenance/language/StatOutputs.php
index 723ea62c662f..9979babef9d2 100644
--- a/maintenance/language/StatOutputs.php
+++ b/maintenance/language/StatOutputs.php
@@ -25,7 +25,7 @@
 
 /** A general output object. Need to be overridden */
 class StatsOutput {
-	function formatPercent( $subset, $total, $revert = false, $accuracy = 2 ) {
+	public function formatPercent( $subset, $total, $revert = false, $accuracy = 2 ) {
 		Wikimedia\suppressWarnings();
 		$return = sprintf( '%.' . $accuracy . 'f%%', 100 * $subset / $total );
 		Wikimedia\restoreWarnings();
@@ -34,25 +34,25 @@ class StatsOutput {
 	}
 
 	# Override the following methods
-	function heading() {
+	public function heading() {
 	}
 
-	function footer() {
+	public function footer() {
 	}
 
-	function blockstart() {
+	public function blockstart() {
 	}
 
-	function blockend() {
+	public function blockend() {
 	}
 
-	function element( $in, $heading = false ) {
+	public function element( $in, $heading = false ) {
 	}
 }
 
 /** Outputs WikiText */
 class WikiStatsOutput extends StatsOutput {
-	function heading() {
+	public function heading() {
 		global $wgDummyLanguageCodes;
 		$version = SpecialVersion::getVersion( 'nodb' );
 		echo "'''Statistics are based on:''' <code>" . $version . "</code>\n\n";
@@ -75,23 +75,23 @@ class WikiStatsOutput extends StatsOutput {
 			'border: 1px #AAAAAA solid; border-collapse: collapse; clear:both; width:100%;"' . "\n";
 	}
 
-	function footer() {
+	public function footer() {
 		echo "|}\n";
 	}
 
-	function blockstart() {
+	public function blockstart() {
 		echo "|-\n";
 	}
 
-	function blockend() {
+	public function blockend() {
 		echo '';
 	}
 
-	function element( $in, $heading = false ) {
+	public function element( $in, $heading = false ) {
 		echo ( $heading ? '!' : '|' ) . "$in\n";
 	}
 
-	function formatPercent( $subset, $total, $revert = false, $accuracy = 2 ) {
+	public function formatPercent( $subset, $total, $revert = false, $accuracy = 2 ) {
 		Wikimedia\suppressWarnings();
 		$v = round( 255 * $subset / $total );
 		Wikimedia\restoreWarnings();
@@ -125,22 +125,22 @@ class WikiStatsOutput extends StatsOutput {
 
 /** Output text. To be used on a terminal for example. */
 class TextStatsOutput extends StatsOutput {
-	function element( $in, $heading = false ) {
+	public function element( $in, $heading = false ) {
 		echo $in . "\t";
 	}
 
-	function blockend() {
+	public function blockend() {
 		echo "\n";
 	}
 }
 
 /** csv output. Some people love excel */
 class CsvStatsOutput extends StatsOutput {
-	function element( $in, $heading = false ) {
+	public function element( $in, $heading = false ) {
 		echo $in . ";";
 	}
 
-	function blockend() {
+	public function blockend() {
 		echo "\n";
 	}
 }
diff --git a/maintenance/language/checkLanguage.inc b/maintenance/language/checkLanguage.inc
index 007ced15600e..409022f692e9 100644
--- a/maintenance/language/checkLanguage.inc
+++ b/maintenance/language/checkLanguage.inc
@@ -473,7 +473,7 @@ ENDS;
 	/**
 	 * Output the checks results as wiki text.
 	 */
-	function outputWiki() {
+	private function outputWiki() {
 		$detailText = '';
 		$rows[] = '! Language !! Code !! Total !! ' .
 			implode( ' !! ', array_diff( $this->checks, $this->nonMessageChecks() ) );
diff --git a/maintenance/language/generateCollationData.php b/maintenance/language/generateCollationData.php
index 29083b02ac59..7952aca6ddfb 100644
--- a/maintenance/language/generateCollationData.php
+++ b/maintenance/language/generateCollationData.php
@@ -138,12 +138,12 @@ class GenerateCollationData extends Maintenance {
 		$this->generateFirstChars();
 	}
 
-	function loadUcd() {
+	private function loadUcd() {
 		$uxr = new UcdXmlReader( "{$this->dataDir}/ucd.all.grouped.xml" );
 		$uxr->readChars( [ $this, 'charCallback' ] );
 	}
 
-	function charCallback( $data ) {
+	private function charCallback( $data ) {
 		// Skip non-printable characters,
 		// but do not skip a normal space (U+0020) since
 		// people like to use that as a fake no header symbol.
@@ -193,7 +193,7 @@ class GenerateCollationData extends Maintenance {
 		}
 	}
 
-	function generateFirstChars() {
+	private function generateFirstChars() {
 		$file = fopen( "{$this->dataDir}/allkeys.txt", 'r' );
 		if ( !$file ) {
 			$this->fatalError( "Unable to open allkeys.txt" );
diff --git a/maintenance/language/languages.inc b/maintenance/language/languages.inc
index c6a5c6829985..3a6da243fcb1 100644
--- a/maintenance/language/languages.inc
+++ b/maintenance/language/languages.inc
@@ -62,7 +62,7 @@ class Languages {
 	 * Load the list of languages: all the Messages*.php
 	 * files in the languages directory.
 	 */
-	function __construct() {
+	public function __construct() {
 		Hooks::run( 'LocalisationIgnoredOptionalMessages',
 			[ &$this->mIgnoredMessages, &$this->mOptionalMessages ] );
 
@@ -795,7 +795,7 @@ class ExtensionLanguages extends Languages {
 	 * Load the messages group.
 	 * @param MessageGroup $group The messages group.
 	 */
-	function __construct( MessageGroup $group ) {
+	public function __construct( MessageGroup $group ) {
 		$this->mMessageGroup = $group;
 
 		$this->mIgnoredMessages = $this->mMessageGroup->getIgnored();