jenkins-bot has submitted this change and it was merged.

Change subject: ReplacementArray: Set explicit visiblity on functions
......................................................................


ReplacementArray: Set explicit visiblity on functions

Also add a type-hint

Change-Id: I78a322dfa11a71e22a3d0b7064b808aec6bfbf83
---
M includes/libs/ReplacementArray.php
1 file changed, 11 insertions(+), 11 deletions(-)

Approvals:
  Addshore: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/includes/libs/ReplacementArray.php 
b/includes/libs/ReplacementArray.php
index 8c8c010..7fdb309 100644
--- a/includes/libs/ReplacementArray.php
+++ b/includes/libs/ReplacementArray.php
@@ -31,18 +31,18 @@
         * The array should have the same form as the replacement array for 
strtr()
         * @param array $data
         */
-       function __construct( $data = array() ) {
+       public function __construct( $data = array() ) {
                $this->data = $data;
        }
 
        /**
         * @return array
         */
-       function __sleep() {
+       public function __sleep() {
                return array( 'data' );
        }
 
-       function __wakeup() {
+       public function __wakeup() {
                $this->fss = false;
        }
 
@@ -50,7 +50,7 @@
         * Set the whole replacement array at once
         * @param array $data
         */
-       function setArray( $data ) {
+       public function setArray( $data ) {
                $this->data = $data;
                $this->fss = false;
        }
@@ -58,7 +58,7 @@
        /**
         * @return array|bool
         */
-       function getArray() {
+       public function getArray() {
                return $this->data;
        }
 
@@ -67,7 +67,7 @@
         * @param string $from
         * @param string $to
         */
-       function setPair( $from, $to ) {
+       public function setPair( $from, $to ) {
                $this->data[$from] = $to;
                $this->fss = false;
        }
@@ -75,7 +75,7 @@
        /**
         * @param array $data
         */
-       function mergeArray( $data ) {
+       public function mergeArray( $data ) {
                $this->data = array_merge( $this->data, $data );
                $this->fss = false;
        }
@@ -83,7 +83,7 @@
        /**
         * @param ReplacementArray $other
         */
-       function merge( $other ) {
+       public function merge( ReplacementArray $other ) {
                $this->data = array_merge( $this->data, $other->data );
                $this->fss = false;
        }
@@ -91,7 +91,7 @@
        /**
         * @param string $from
         */
-       function removePair( $from ) {
+       public function removePair( $from ) {
                unset( $this->data[$from] );
                $this->fss = false;
        }
@@ -99,7 +99,7 @@
        /**
         * @param array $data
         */
-       function removeArray( $data ) {
+       public function removeArray( $data ) {
                foreach ( $data as $from => $to ) {
                        $this->removePair( $from );
                }
@@ -110,7 +110,7 @@
         * @param string $subject
         * @return string
         */
-       function replace( $subject ) {
+       public function replace( $subject ) {
                if ( function_exists( 'fss_prep_replace' ) ) {
                        if ( $this->fss === false ) {
                                $this->fss = fss_prep_replace( $this->data );

-- 
To view, visit https://gerrit.wikimedia.org/r/187405
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I78a322dfa11a71e22a3d0b7064b808aec6bfbf83
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/core
Gerrit-Branch: master
Gerrit-Owner: Legoktm <legoktm.wikipe...@gmail.com>
Gerrit-Reviewer: Addshore <addshorew...@gmail.com>
Gerrit-Reviewer: jenkins-bot <>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to