From: Chad Horohoe <demon@users.mediawiki.org>
Date: Tue, 26 Oct 2010 12:02:23 +0000 (+0000)
Subject: Clarify some visibilities based on actual usage
X-Git-Tag: 1.31.0-rc.0~34307
X-Git-Url: http://git.cyclocoop.org/%24self?a=commitdiff_plain;h=e9db0e2f1db56b4f367a690183776e33144ff991;p=lhc%2Fweb%2Fwiklou.git

Clarify some visibilities based on actual usage
---

diff --git a/includes/installer/WebInstaller.php b/includes/installer/WebInstaller.php
index c9ef7cbb52..340b006d00 100644
--- a/includes/installer/WebInstaller.php
+++ b/includes/installer/WebInstaller.php
@@ -460,7 +460,7 @@ class WebInstaller extends CoreInstaller {
 	 *
 	 * @param $currentPageName String
 	 */
-	public function startPageWrapper( $currentPageName ) {
+	private function startPageWrapper( $currentPageName ) {
 		$s = "<div class=\"config-page-wrapper\">\n" .
 			"<div class=\"config-page-list\"><ul>\n";
 		$lastHappy = -1;
@@ -501,7 +501,7 @@ class WebInstaller extends CoreInstaller {
 	 *
 	 * @return string
 	 */
-	public function getPageListItem( $pageName, $enabled, $currentPageName ) {
+	private function getPageListItem( $pageName, $enabled, $currentPageName ) {
 		$s = "<li class=\"config-page-list-item\">";
 		$name = wfMsg( 'config-page-' . strtolower( $pageName ) );
 
@@ -545,7 +545,7 @@ class WebInstaller extends CoreInstaller {
 	/**
 	 * Output some stuff after a page is finished.
 	 */
-	public function endPageWrapper() {
+	private function endPageWrapper() {
 		$this->output->addHTMLNoFlush(
 			"</div>\n" .
 			"<br style=\"clear:both\"/>\n" .
@@ -928,7 +928,7 @@ class WebInstaller extends CoreInstaller {
 	/**
 	 * Helper for Installer::docLink()
 	 */
-	public function getDocUrl( $page ) {
+	protected function getDocUrl( $page ) {
 		$url = "{$_SERVER['PHP_SELF']}?page=" . urlencode( $page );
 
 		if ( in_array( $this->currentPageName, $this->pageSequence ) ) {