Merge "Perform a permission check on the title when changing the page language"

[lhc/web/wiklou.git] / includes / changes / ChangesList.php

diff --git a/includes/changes/ChangesList.php b/includes/changes/ChangesList.php
index 04f815a..cac4769 100644 (file)
--- a/includes/changes/ChangesList.php
+++ b/includes/changes/ChangesList.php
@@ -41,6 +41,9 @@ class ChangesList extends ContextSource {
        protected $rclistOpen;
        protected $rcMoveIndex;
 
+       /** @var callable */
+       protected $changeLinePrefixer;
+
        /** @var BagOStuff */
        protected $watchMsgCache;
 
@@ -102,7 +105,7 @@ class ChangesList extends ContextSource {
         *
         * @since 1.27
         *
-        * @param RecentChange $rc Passed by reference
+        * @param RecentChange &$rc Passed by reference
         * @param bool $watched (default false)
         * @param int $linenumber (default null)
         *
@@ -169,12 +172,14 @@ class ChangesList extends ContextSource {
         * @return array of classes
         */
        protected function getHTMLClasses( $rc, $watched ) {
-               $classes = [];
+               $classes = [ self::CSS_CLASS_PREFIX . 'line' ];
                $logType = $rc->mAttribs['rc_log_type'];
 
                if ( $logType ) {
+                       $classes[] = self::CSS_CLASS_PREFIX . 'log';
                        $classes[] = Sanitizer::escapeClass( self::CSS_CLASS_PREFIX . 'log-' . $logType );
                } else {
+                       $classes[] = self::CSS_CLASS_PREFIX . 'edit';
                        $classes[] = Sanitizer::escapeClass( self::CSS_CLASS_PREFIX . 'ns' .
                                $rc->mAttribs['rc_namespace'] . '-' . $rc->mAttribs['rc_title'] );
                        $classes[] = Sanitizer::escapeClass( self::CSS_CLASS_PREFIX . 'ns-' .
@@ -362,7 +367,7 @@ class ChangesList extends ContextSource {
        }
 
        /**
-        * @param string $s HTML to update
+        * @param string &$s HTML to update
         * @param mixed $rc_timestamp
         */
        public function insertDateHeader( &$s, $rc_timestamp ) {
@@ -379,7 +384,7 @@ class ChangesList extends ContextSource {
        }
 
        /**
-        * @param string $s HTML to update
+        * @param string &$s HTML to update
         * @param Title $title
         * @param string $logtype
         */
@@ -392,8 +397,8 @@ class ChangesList extends ContextSource {
        }
 
        /**
-        * @param string $s HTML to update
-        * @param RecentChange $rc
+        * @param string &$s HTML to update
+        * @param RecentChange &$rc
         * @param bool|null $unpatrolled Unused variable, since 1.27.
         */
        public function insertDiffHist( &$s, &$rc, $unpatrolled = null ) {
@@ -442,7 +447,7 @@ class ChangesList extends ContextSource {
        }
 
        /**
-        * @param string $s Article link will be appended to this string, in place.
+        * @param string &$s Article link will be appended to this string, in place.
         * @param RecentChange $rc
         * @param bool $unpatrolled
         * @param bool $watched
@@ -453,7 +458,7 @@ class ChangesList extends ContextSource {
        }
 
        /**
-        * @param RecentChange $rc
+        * @param RecentChange &$rc
         * @param bool $unpatrolled
         * @param bool $watched
         * @return string HTML
@@ -508,7 +513,7 @@ class ChangesList extends ContextSource {
        /**
         * Insert time timestamp string from $rc into $s
         *
-        * @param string $s HTML to update
+        * @param string &$s HTML to update
         * @param RecentChange $rc
         */
        public function insertTimestamp( &$s, $rc ) {
@@ -620,8 +625,8 @@ class ChangesList extends ContextSource {
 
        /** Inserts a rollback link
         *
-        * @param string $s
-        * @param RecentChange $rc
+        * @param string &$s
+        * @param RecentChange &$rc
         */
        public function insertRollback( &$s, &$rc ) {
                if ( $rc->mAttribs['rc_type'] == RC_EDIT
@@ -658,9 +663,9 @@ class ChangesList extends ContextSource {
        }
 
        /**
-        * @param string $s
-        * @param RecentChange $rc
-        * @param array $classes
+        * @param string &$s
+        * @param RecentChange &$rc
+        * @param array &$classes
         */
        public function insertTags( &$s, &$rc, &$classes ) {
                if ( empty( $rc->mAttribs['ts_tags'] ) ) {
@@ -678,7 +683,7 @@ class ChangesList extends ContextSource {
 
        /**
         * @param RecentChange $rc
-        * @param array $classes
+        * @param array &$classes
         * @return string
         * @since 1.26
         */
@@ -766,4 +771,15 @@ class ChangesList extends ContextSource {
 
                return $attrs;
        }
+
+       /**
+        * Sets the callable that generates a change line prefix added to the beginning of each line.
+        *
+        * @param callable $prefixer Callable to run that generates the change line prefix.
+        *     Takes three parameters: a RecentChange object, a ChangesList object,
+        *     and whether the current entry is a grouped entry.
+        */
+       public function setChangeLinePrefixer( callable $prefixer ) {
+               $this->changeLinePrefixer = $prefixer;
+       }
 }