Bump development Monolog version

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

diff --git a/includes/changes/ChangesList.php b/includes/changes/ChangesList.php
index 249f53f..9ac6c32 100644 (file)
--- a/includes/changes/ChangesList.php
+++ b/includes/changes/ChangesList.php
@@ -76,6 +76,21 @@ class ChangesList extends ContextSource {
                }
        }
 
+       /**
+        * Format a line
+        *
+        * @since 1.27
+        *
+        * @param RecentChange $rc Passed by reference
+        * @param bool $watched (default false)
+        * @param int $linenumber (default null)
+        *
+        * @return string|bool
+        */
+       public function recentChangesLine( &$rc, $watched = false, $linenumber = null ) {
+               throw new RuntimeException( 'recentChangesLine should be implemented' );
+       }
+
        /**
         * Sets the list to use a "<li class='watchlist-(namespace)-(page)'>" tag
         * @param bool $value
@@ -305,7 +320,11 @@ class ChangesList extends ContextSource {
         */
        public function insertDiffHist( &$s, &$rc, $unpatrolled ) {
                # Diff link
-               if ( $rc->mAttribs['rc_type'] == RC_NEW || $rc->mAttribs['rc_type'] == RC_LOG ) {
+               if (
+                       $rc->mAttribs['rc_type'] == RC_NEW ||
+                       $rc->mAttribs['rc_type'] == RC_LOG ||
+                       $rc->mAttribs['rc_type'] == RC_CATEGORIZE
+               ) {
                        $diffLink = $this->message['diff'];
                } elseif ( !self::userCan( $rc, Revision::DELETED_TEXT, $this->getUser() ) ) {
                        $diffLink = $this->message['diff'];
@@ -323,17 +342,22 @@ class ChangesList extends ContextSource {
                                $query
                        );
                }
-               $diffhist = $diffLink . $this->message['pipe-separator'];
-               # History link
-               $diffhist .= Linker::linkKnown(
-                       $rc->getTitle(),
-                       $this->message['hist'],
-                       array(),
-                       array(
-                               'curid' => $rc->mAttribs['rc_cur_id'],
-                               'action' => 'history'
-                       )
-               );
+               if ( $rc->mAttribs['rc_type'] == RC_CATEGORIZE ) {
+                       $diffhist = $diffLink . $this->message['pipe-separator'] . $this->message['hist'];
+               } else {
+                       $diffhist = $diffLink . $this->message['pipe-separator'];
+                       # History link
+                       $diffhist .= Linker::linkKnown(
+                               $rc->getTitle(),
+                               $this->message['hist'],
+                               array(),
+                               array(
+                                       'curid' => $rc->mAttribs['rc_cur_id'],
+                                       'action' => 'history'
+                               )
+                       );
+               }
+
                // @todo FIXME: Hard coded ". .". Is there a message for this? Should there be?
                $s .= $this->msg( 'parentheses' )->rawParams( $diffhist )->escaped() .
                        ' <span class="mw-changeslist-separator">. .</span> ';
@@ -456,28 +480,6 @@ class ChangesList extends ContextSource {
                }
        }
 
-       /**
-        * @param RCCacheEntry $rc
-        * @param array $query array of key/value pairs to append as a query string
-        * @return string
-        * @since 1.26
-        */
-       public function getDiffHistLinks( RCCacheEntry $rc, array $query ) {
-               $pageTitle = $rc->getTitle();
-               if ( intval( $rc->getAttribute( 'rc_type' ) ) === RC_CATEGORIZE ) {
-                       $pageTitle = Title::newFromID( $rc->getAttribute( 'rc_cur_id' ) );
-               }
-
-               $retVal = ' ' . $this->msg( 'parentheses' )
-               ->rawParams( $rc->difflink . $this->message['pipe-separator'] . Linker::linkKnown(
-                               $pageTitle,
-                               $this->message['hist'],
-                               array(),
-                               $query
-                       ) )->escaped();
-               return $retVal;
-       }
-
        /**
         * Check whether to enable recent changes patrol features
         *
@@ -657,12 +659,14 @@ class ChangesList extends ContextSource {
         * Determines whether a revision is linked to this change; this may not be the case
         * when the categorization wasn't done by an edit but a conditional parser function
         *
+        * @since 1.27
+        *
         * @param RecentChange|RCCacheEntry $rcObj
         * @return bool
         */
        protected function isCategorizationWithoutRevision( $rcObj ) {
                return intval( $rcObj->getAttribute( 'rc_type' ) ) === RC_CATEGORIZE
-               && intval( $rcObj->getAttribute( 'rc_this_oldid' ) ) === 0;
+                       && intval( $rcObj->getAttribute( 'rc_this_oldid' ) ) === 0;
        }
 
 }