Revert r35848 per Brion's WONTFIX of bug 14536: "This would just mean that there...
[lhc/web/wiklou.git] / includes / SpecialRecentchanges.php
index 84444e6..cd1dac0 100644 (file)
@@ -1,7 +1,7 @@
 <?php
 /**
- *
- * @addtogroup SpecialPage
+ * @file
+ * @ingroup SpecialPage
  */
 
 /**
@@ -16,7 +16,6 @@ function wfSpecialRecentchanges( $par, $specialPage ) {
        global $wgUser, $wgOut, $wgRequest, $wgUseRCPatrol;
        global $wgRCShowWatchingUsers, $wgShowUpdatedMarker;
        global $wgAllowCategorizedRecentChanges ;
-       $fname = 'wfSpecialRecentchanges';
 
        # Get query parameters
        $feedFormat = $wgRequest->getVal( 'feed' );
@@ -52,14 +51,11 @@ function wfSpecialRecentchanges( $par, $specialPage ) {
 
        /* order of selection: url > preferences > default */
        $hideminor = $wgRequest->getBool( 'hideminor', $wgUser->getOption( 'hideminor') ? true : $defaults['hideminor'] );
-       
+
        # As a feed, use limited settings only
        if( $feedFormat ) {
                global $wgFeedLimit;
-               if( $limit > $wgFeedLimit ) {
-                       $limit = $wgFeedLimit;
-               }
-
+               $limit = min( $wgFeedLimit, $limit );
        } else {
 
                $namespace = $wgRequest->getIntOrNull( 'namespace' );
@@ -87,7 +83,7 @@ function wfSpecialRecentchanges( $par, $specialPage ) {
                                if ( is_numeric( $bit ) ) {
                                        $limit = $bit;
                                }
-                               
+
                                $m = array();
                                if ( preg_match( '/^limit=(\d+)$/', $bit, $m ) ) {
                                        $limit = $m[1];
@@ -102,11 +98,8 @@ function wfSpecialRecentchanges( $par, $specialPage ) {
 
        if ( $limit < 0 || $limit > 5000 ) $limit = $defaults['limit'];
 
-
        # Database connection and caching
        $dbr = wfGetDB( DB_SLAVE );
-       list( $recentchanges, $watchlist ) = $dbr->tableNamesN( 'recentchanges', 'watchlist' );
-
 
        $cutoff_unixtime = time() - ( $days * 86400 );
        $cutoff_unixtime = $cutoff_unixtime - ($cutoff_unixtime % 86400);
@@ -122,7 +115,7 @@ function wfSpecialRecentchanges( $par, $specialPage ) {
 
        # Get last modified date, for client caching
        # Don't use this if we are using the patrol feature, patrol changes don't update the timestamp
-       $lastmod = $dbr->selectField( 'recentchanges', 'MAX(rc_timestamp)', false, $fname );
+       $lastmod = $dbr->selectField( 'recentchanges', 'MAX(rc_timestamp)', false, __FUNCTION__ );
        if ( $feedFormat || !$wgUseRCPatrol ) {
                if( $lastmod && $wgOut->checkLastModified( $lastmod ) ){
                        # Client cache fresh and headers sent, nothing more to do.
@@ -132,53 +125,87 @@ function wfSpecialRecentchanges( $par, $specialPage ) {
 
        # It makes no sense to hide both anons and logged-in users
        # Where this occurs, force anons to be shown
-       if( $hideanons && $hideliu )
-               $hideanons = false;
+       $forcebot = false;
+       if( $hideanons && $hideliu ){
+               # Check if the user wants to show bots only
+               if( $hidebots ){
+                       $hideanons = 0;
+               } else {
+                       $forcebot = true;
+                       $hidebots = 0;
+               }
+       }
 
        # Form WHERE fragments for all the options
        $hidem  = $hideminor ? 'AND rc_minor = 0' : '';
        $hidem .= $hidebots ? ' AND rc_bot = 0' : '';
-       $hidem .= $hideliu ? ' AND rc_user = 0' : '';
-       $hidem .= ( $wgUseRCPatrol && $hidepatrolled ) ? ' AND rc_patrolled = 0' : '';
-       $hidem .= $hideanons ? ' AND rc_user != 0' : '';
-       
+       $hidem .= $hideliu && !$forcebot ? ' AND rc_user = 0' : '';
+       $hidem .= ($wgUser->useRCPatrol() && $hidepatrolled ) ? ' AND rc_patrolled = 0' : '';
+       $hidem .= $hideanons && !$forcebot ? ' AND rc_user != 0' : '';
+       $hidem .= $forcebot ? ' AND rc_bot = 1' : '';
+
        if( $hidemyself ) {
-               if( $wgUser->getID() ) {
-                       $hidem .= ' AND rc_user != ' . $wgUser->getID();
+               if( $wgUser->getId() ) {
+                       $hidem .= ' AND rc_user != ' . $wgUser->getId();
                } else {
                        $hidem .= ' AND rc_user_text != ' . $dbr->addQuotes( $wgUser->getName() );
                }
        }
 
+       // JOIN on watchlist for users
+       $uid = $wgUser->getId();
+       if( $uid ) {
+               $tables = array( 'recentchanges', 'watchlist' );
+               $join_conds = array( 'watchlist' => array('LEFT JOIN',"wl_user={$uid} AND wl_title=rc_title AND wl_namespace=rc_namespace") );
+       } else {
+               $tables = array( 'recentchanges' );
+               $join_conds = array();
+       }
+       
        # Namespace filtering
-       $hidem .= is_null( $namespace ) ?  '' : ' AND rc_namespace' . ($invert ? '!=' : '=') . $namespace;
-
-       // This is the big thing!
-
-       $uid = $wgUser->getID();
-
-       // Perform query
-       $forceclause = $dbr->useIndexClause("rc_timestamp");
-       $sql2 = "SELECT * FROM $recentchanges $forceclause".
-         ($uid ? "LEFT OUTER JOIN $watchlist ON wl_user={$uid} AND wl_title=rc_title AND wl_namespace=rc_namespace " : "") .
-         "WHERE rc_timestamp >= '{$cutoff}' {$hidem} " .
-         "ORDER BY rc_timestamp DESC";
-       $sql2 = $dbr->limitResult($sql2, $limit, 0);
-       $res = $dbr->query( $sql2, $fname );
-
+       $hidem .= is_null($namespace) ?  '' : ' AND rc_namespace' . ($invert ? '!=' : '=') . $namespace;
+       
+       // Is there either one namespace selected or excluded?
+       // Also, if this is "all" or main namespace, just use timestamp index.
+       if( is_null($namespace) || $invert || $namespace == NS_MAIN ) {
+               $res = $dbr->select( $tables, '*',
+                       array( "rc_timestamp >= '{$cutoff}' {$hidem}" ),
+                       __METHOD__,
+                       array( 'ORDER BY' => 'rc_timestamp DESC', 'LIMIT' => $limit, 
+                               'USE INDEX' => array('recentchanges' => 'rc_timestamp') ),
+                       $join_conds );
+       // We have a new_namespace_time index! UNION over new=(0,1) and sort result set!
+       } else {
+               // New pages
+               $sqlNew = $dbr->selectSQLText( $tables, '*',
+                       array( 'rc_new' => 1,
+                               "rc_timestamp >= '{$cutoff}' {$hidem}" ),
+                       __METHOD__,
+                       array( 'ORDER BY' => 'rc_timestamp DESC', 'LIMIT' => $limit, 
+                               'USE INDEX' =>  array('recentchanges' => 'new_name_timestamp') ),
+                       $join_conds );
+               // Old pages
+               $sqlOld = $dbr->selectSQLText( $tables, '*',
+                       array( 'rc_new' => 0,
+                               "rc_timestamp >= '{$cutoff}' {$hidem}" ),
+                       __METHOD__,
+                       array( 'ORDER BY' => 'rc_timestamp DESC', 'LIMIT' => $limit, 
+                               'USE INDEX' =>  array('recentchanges' => 'new_name_timestamp') ),
+                       $join_conds );
+               # Join the two fast queries, and sort the result set
+               $sql = "($sqlNew) UNION ($sqlOld) ORDER BY rc_timestamp DESC LIMIT $limit";
+               $res = $dbr->query( $sql, __METHOD__ );
+       }
+       
        // Fetch results, prepare a batch link existence check query
        $rows = array();
        $batch = new LinkBatch;
        while( $row = $dbr->fetchObject( $res ) ){
                $rows[] = $row;
                if ( !$feedFormat ) {
-                       // User page link
-                       $title = Title::makeTitleSafe( NS_USER, $row->rc_user_text );
-                       $batch->addObj( $title );
-
-                       // User talk
-                       $title = Title::makeTitleSafe( NS_USER_TALK, $row->rc_user_text );
-                       $batch->addObj( $title );
+                       // User page and talk links
+                       $batch->add( NS_USER, $row->rc_user_text  );
+                       $batch->add( NS_USER_TALK, $row->rc_user_text  );
                }
 
        }
@@ -223,7 +250,7 @@ function wfSpecialRecentchanges( $par, $specialPage ) {
                $wgOut->setSyndicated( true );
 
                $list = ChangesList::newFromUser( $wgUser );
-               
+
                if ( $wgAllowCategorizedRecentChanges ) {
                        $categories = trim ( $wgRequest->getVal ( 'categories' , "" ) ) ;
                        $categories = str_replace ( "|" , "\n" , $categories ) ;
@@ -233,6 +260,10 @@ function wfSpecialRecentchanges( $par, $specialPage ) {
 
                $s = $list->beginRecentChangesList();
                $counter = 1;
+
+               $showWatcherCount = $wgRCShowWatchingUsers && $wgUser->getOption( 'shownumberswatching' );
+               $watcherCache = array();
+
                foreach( $rows as $obj ){
                        if( $limit == 0) {
                                break;
@@ -251,13 +282,19 @@ function wfSpecialRecentchanges( $par, $specialPage ) {
                                        $rc->notificationtimestamp = false;
                                }
 
-                               if ($wgRCShowWatchingUsers && $wgUser->getOption( 'shownumberswatching' )) {
-                                       $sql3 = "SELECT COUNT(*) AS n FROM $watchlist WHERE wl_title='" . $dbr->strencode($obj->rc_title) ."' AND wl_namespace=$obj->rc_namespace" ;
-                                       $res3 = $dbr->query( $sql3, 'wfSpecialRecentChanges');
-                                       $x = $dbr->fetchObject( $res3 );
-                                       $rc->numberofWatchingusers = $x->n;
-                               } else {
-                                       $rc->numberofWatchingusers = 0;
+                               $rc->numberofWatchingusers = 0; // Default
+                               if ($showWatcherCount && $obj->rc_namespace >= 0) {
+                                       if (!isset($watcherCache[$obj->rc_namespace][$obj->rc_title])) {
+                                               $watcherCache[$obj->rc_namespace][$obj->rc_title] =
+                                                       $dbr->selectField( 'watchlist',
+                                                               'COUNT(*)',
+                                                               array(
+                                                                       'wl_namespace' => $obj->rc_namespace,
+                                                                       'wl_title' => $obj->rc_title,
+                                                               ),
+                                                               __METHOD__ . '-watchers' );
+                                       }
+                                       $rc->numberofWatchingusers = $watcherCache[$obj->rc_namespace][$obj->rc_title];
                                }
                                $s .= $list->recentChangesLine( $rc, !empty( $obj->wl_user ) );
                                --$limit;
@@ -269,8 +306,10 @@ function wfSpecialRecentchanges( $par, $specialPage ) {
 }
 
 function rcFilterByCategories ( &$rows , $categories , $any ) {
-       require_once ( 'Categoryfinder.php' ) ;
-       
+       if( empty( $categories ) ) {
+               return;
+       }
+
        # Filter categories
        $cats = array () ;
        foreach ( $categories AS $cat ) {
@@ -278,12 +317,12 @@ function rcFilterByCategories ( &$rows , $categories , $any ) {
                if ( $cat == "" ) continue ;
                $cats[] = $cat ;
        }
-       
+
        # Filter articles
        $articles = array () ;
        $a2r = array () ;
        foreach ( $rows AS $k => $r ) {
-               $nt = Title::newFromText ( $r->rc_title , $r->rc_namespace ) ;
+               $nt = Title::makeTitle( $r->rc_namespace, $r->rc_title );
                $id = $nt->getArticleID() ;
                if ( $id == 0 ) continue ; # Page might have been deleted...
                if ( !in_array ( $id , $articles ) ) {
@@ -294,16 +333,16 @@ function rcFilterByCategories ( &$rows , $categories , $any ) {
                }
                $a2r[$id][] = $k ;
        }
-       
+
        # Shortcut?
        if ( count ( $articles ) == 0 OR count ( $cats ) == 0 )
                return ;
-       
+
        # Look up
        $c = new Categoryfinder ;
        $c->seed ( $articles , $cats , $any ? "OR" : "AND" ) ;
        $match = $c->run () ;
-       
+
        # Filter
        $newrows = array () ;
        foreach ( $match AS $id ) {
@@ -318,6 +357,13 @@ function rcFilterByCategories ( &$rows , $categories , $any ) {
 function rcOutputFeed( $rows, $feedFormat, $limit, $hideminor, $lastmod ) {
        global $messageMemc, $wgFeedCacheTimeout;
        global $wgFeedClasses, $wgTitle, $wgSitename, $wgContLanguageCode;
+       global $wgFeed;
+
+       if ( !$wgFeed ) {
+               global $wgOut;
+               $wgOut->addWikiMsg( 'feed-unavailable' );
+               return;
+       }
 
        if( !isset( $wgFeedClasses[$feedFormat] ) ) {
                wfHttpError( 500, "Internal Server Error", "Unsupported feed type." );
@@ -334,6 +380,14 @@ function rcOutputFeed( $rows, $feedFormat, $limit, $hideminor, $lastmod ) {
                htmlspecialchars( wfMsgForContent( 'recentchanges-feed-description' ) ),
                $wgTitle->getFullUrl() );
 
+       //purge cache if requested
+       global $wgRequest, $wgUser;
+       $purge = $wgRequest->getVal( 'action' ) == 'purge';
+       if ( $purge && $wgUser->isAllowed('purge') ) {
+               $messageMemc->delete( $timekey );
+               $messageMemc->delete( $key );
+       }
+
        /**
         * Bumping around loading up diffs can be pretty slow, so where
         * possible we want to cache the feed output so the next visitor
@@ -375,9 +429,12 @@ function rcOutputFeed( $rows, $feedFormat, $limit, $hideminor, $lastmod ) {
        return true;
 }
 
+/**
+ * @todo document
+ * @param $rows Database resource with recentchanges rows
+ */
 function rcDoOutputFeed( $rows, &$feed ) {
-       $fname = 'rcDoOutputFeed';
-       wfProfileIn( $fname );
+       wfProfileIn( __METHOD__ );
 
        $feed->outHeader();
 
@@ -402,38 +459,40 @@ function rcDoOutputFeed( $rows, &$feed ) {
                $item = new FeedItem(
                        $title->getPrefixedText(),
                        rcFormatDiff( $obj ),
-                       $title->getFullURL(),
+                       $title->getFullURL( 'diff=' . $obj->rc_this_oldid . '&oldid=prev' ),
                        $obj->rc_timestamp,
-                       $obj->rc_user_text,
+                       ($obj->rc_deleted & Revision::DELETED_USER) ? wfMsgHtml('rev-deleted-user') : $obj->rc_user_text,
                        $talkpage->getFullURL()
                        );
                $feed->outItem( $item );
        }
        $feed->outFooter();
-       wfProfileOut( $fname );
+       wfProfileOut( __METHOD__ );
 }
 
 /**
  *
  */
-function rcCountLink( $lim, $d, $page='Recentchanges', $more='' ) {
+function rcCountLink( $lim, $d, $page='Recentchanges', $more='', $active = false ) {
        global $wgUser, $wgLang, $wgContLang;
        $sk = $wgUser->getSkin();
        $s = $sk->makeKnownLink( $wgContLang->specialPage( $page ),
          ($lim ? $wgLang->formatNum( "{$lim}" ) : wfMsg( 'recentchangesall' ) ), "{$more}" .
-         ($d ? "days={$d}&" : '') . 'limit='.$lim );
+         ($d ? "days={$d}&" : '') . 'limit='.$lim, '', '',
+         $active ? 'style="font-weight: bold;"' : '' );
        return $s;
 }
 
 /**
  *
  */
-function rcDaysLink( $lim, $d, $page='Recentchanges', $more='' ) {
+function rcDaysLink( $lim, $d, $page='Recentchanges', $more='', $active = false ) {
        global $wgUser, $wgLang, $wgContLang;
        $sk = $wgUser->getSkin();
        $s = $sk->makeKnownLink( $wgContLang->specialPage( $page ),
          ($d ? $wgLang->formatNum( "{$d}" ) : wfMsg( 'recentchangesall' ) ), $more.'days='.$d .
-         ($lim ? '&limit='.$lim : '') );
+         ($lim ? '&limit='.$lim : ''), '', '',
+         $active ? 'style="font-weight: bold;"' : '' );
        return $s;
 }
 
@@ -442,25 +501,38 @@ function rcDaysLink( $lim, $d, $page='Recentchanges', $more='' ) {
  */
 function rcDayLimitLinks( $days, $limit, $page='Recentchanges', $more='', $doall = false, $minorLink = '',
        $botLink = '', $liuLink = '', $patrLink = '', $myselfLink = '' ) {
+       global $wgRCLinkLimits, $wgRCLinkDays;
        if ($more != '') $more .= '&';
-       $cl = rcCountLink( 50, $days, $page, $more ) . ' | ' .
-         rcCountLink( 100, $days, $page, $more  ) . ' | ' .
-         rcCountLink( 250, $days, $page, $more  ) . ' | ' .
-         rcCountLink( 500, $days, $page, $more  ) .
-         ( $doall ? ( ' | ' . rcCountLink( 0, $days, $page, $more ) ) : '' );
-       $dl = rcDaysLink( $limit, 1, $page, $more  ) . ' | ' .
-         rcDaysLink( $limit, 3, $page, $more  ) . ' | ' .
-         rcDaysLink( $limit, 7, $page, $more  ) . ' | ' .
-         rcDaysLink( $limit, 14, $page, $more  ) . ' | ' .
-         rcDaysLink( $limit, 30, $page, $more  ) .
-         ( $doall ? ( ' | ' . rcDaysLink( $limit, 0, $page, $more ) ) : '' );
+       
+       # Sort data for display and make sure it's unique after we've added user data.
+       # FIXME: why does this piss around with globals like this? Why is $limit added on globally?
+       $wgRCLinkLimits[] = $limit;
+       $wgRCLinkDays[] = $days;
+       sort($wgRCLinkLimits);
+       sort($wgRCLinkDays);
+       $wgRCLinkLimits = array_unique($wgRCLinkLimits);
+       $wgRCLinkDays = array_unique($wgRCLinkDays);
+       
+       $cl = array();
+       foreach( $wgRCLinkLimits as $countLink ) {
+               $cl[] = rcCountLink( $countLink, $days, $page, $more, $countLink == $limit );
+       }
+       if( $doall ) $cl[] = rcCountLink( 0, $days, $page, $more );
+       $cl = implode( ' | ', $cl);
+       
+       $dl = array();
+       foreach( $wgRCLinkDays as $daysLink ) {
+               $dl[] = rcDaysLink( $limit, $daysLink, $page, $more, $daysLink == $days );
+       }
+       if( $doall ) $dl[] = rcDaysLink( $limit, 0, $page, $more );
+       $dl = implode( ' | ', $dl);
        
        $linkParts = array( 'minorLink' => 'minor', 'botLink' => 'bots', 'liuLink' => 'liu', 'patrLink' => 'patr', 'myselfLink' => 'mine' );
        foreach( $linkParts as $linkVar => $linkMsg ) {
                if( $$linkVar != '' )
                        $links[] = wfMsgHtml( 'rcshowhide' . $linkMsg, $$linkVar );
        }
-       
+
        $shm = implode( ' | ', $links );
        $note = wfMsg( 'rclinks', $cl, $dl, $shm );
        return $note;
@@ -473,11 +545,12 @@ function rcDayLimitLinks( $days, $limit, $page='Recentchanges', $more='', $doall
  * @param $override
  * @param $options
  */
-function makeOptionsLink( $title, $override, $options ) {
+function makeOptionsLink( $title, $override, $options, $active = false ) {
        global $wgUser, $wgContLang;
        $sk = $wgUser->getSkin();
        return $sk->makeKnownLink( $wgContLang->specialPage( 'Recentchanges' ),
-               htmlspecialchars( $title ), wfArrayToCGI( $override, $options ) );
+               htmlspecialchars( $title ), wfArrayToCGI( $override, $options ), '', '',
+               $active ? 'style="font-weight: bold;"' : '' );
 }
 
 /**
@@ -486,7 +559,7 @@ function makeOptionsLink( $title, $override, $options ) {
  * @param $nondefaults
  */
 function rcOptionsPanel( $defaults, $nondefaults ) {
-       global $wgLang, $wgUseRCPatrol;
+       global $wgLang, $wgUser, $wgRCLinkLimits, $wgRCLinkDays;
 
        $options = $nondefaults + $defaults;
 
@@ -500,19 +573,25 @@ function rcOptionsPanel( $defaults, $nondefaults ) {
                        $wgLang->formatNum( $options['days'] ),
                        $wgLang->timeAndDate( wfTimestampNow(), true ) );
 
+       # Sort data for display and make sure it's unique after we've added user data.
+       $wgRCLinkLimits[] = $options['limit'];
+       $wgRCLinkDays[] = $options['days'];
+       sort($wgRCLinkLimits);
+       sort($wgRCLinkDays);
+       $wgRCLinkLimits = array_unique($wgRCLinkLimits);
+       $wgRCLinkDays = array_unique($wgRCLinkDays);
+       
        // limit links
-       $options_limit = array(50, 100, 250, 500);
-       foreach( $options_limit as $value ) {
+       foreach( $wgRCLinkLimits as $value ) {
                $cl[] = makeOptionsLink( $wgLang->formatNum( $value ),
-                       array( 'limit' => $value ), $nondefaults) ;
+                       array( 'limit' => $value ), $nondefaults, $value == $options['limit'] ) ;
        }
        $cl = implode( ' | ', $cl);
 
        // day links, reset 'from' to none
-       $options_days = array(1, 3, 7, 14, 30);
-       foreach( $options_days as $value ) {
+       foreach( $wgRCLinkDays as $value ) {
                $dl[] = makeOptionsLink( $wgLang->formatNum( $value ),
-                       array( 'days' => $value, 'from' => ''  ), $nondefaults) ;
+                       array( 'days' => $value, 'from' => '' ), $nondefaults, $value == $options['days'] ) ;
        }
        $dl = implode( ' | ', $dl);
 
@@ -531,16 +610,16 @@ function rcOptionsPanel( $defaults, $nondefaults ) {
                array( 'hidepatrolled' => 1-$options['hidepatrolled'] ), $nondefaults);
        $myselfLink = makeOptionsLink( $showhide[1-$options['hidemyself']],
                array( 'hidemyself' => 1-$options['hidemyself'] ), $nondefaults);
-               
+
        $links[] = wfMsgHtml( 'rcshowhideminor', $minorLink );
        $links[] = wfMsgHtml( 'rcshowhidebots', $botLink );
        $links[] = wfMsgHtml( 'rcshowhideanons', $anonsLink );
        $links[] = wfMsgHtml( 'rcshowhideliu', $liuLink );
-       if( $wgUseRCPatrol )
+       if( $wgUser->useRCPatrol() )
                $links[] = wfMsgHtml( 'rcshowhidepatr', $patrLink );
        $links[] = wfMsgHtml( 'rcshowhidemine', $myselfLink );
        $hl = implode( ' | ', $links );
-               
+
        // show from this onward link
        $now = $wgLang->timeanddate( wfTimestampNow(), true );
        $tl =  makeOptionsLink( $now, array( 'from' => wfTimestampNow()), $nondefaults );
@@ -572,7 +651,7 @@ function rcNamespaceForm( $namespace, $invert, $nondefaults, $categories_any ) {
        $namespaceselect = HTMLnamespaceselector($namespace, '');
        $submitbutton = '<input type="submit" value="' . wfMsgHtml( 'allpagessubmit' ) . "\" />\n";
        $invertbox = "<input type='checkbox' name='invert' value='1' id='nsinvert'" . ( $invert ? ' checked="checked"' : '' ) . ' />';
-       
+
        if ( $wgAllowCategorizedRecentChanges ) {
                $categories = trim ( $wgRequest->getVal ( 'categories' , "" ) ) ;
                $cb_arr = array( 'type' => 'checkbox', 'name' => 'categories_any', 'value' => "1" ) ;
@@ -586,7 +665,7 @@ function rcNamespaceForm( $namespace, $invert, $nondefaults, $categories_any ) {
        } else {
                $catbox = "" ;
        }
-       
+
        $out = "<div class='namespacesettings'><form method='get' action='{$wgScript}'>\n";
 
        foreach ( $nondefaults as $key => $value ) {
@@ -608,25 +687,46 @@ function rcNamespaceForm( $namespace, $invert, $nondefaults, $categories_any ) {
  * Format a diff for the newsfeed
  */
 function rcFormatDiff( $row ) {
+       global $wgUser;
+
        $titleObj = Title::makeTitle( $row->rc_namespace, $row->rc_title );
        $timestamp = wfTimestamp( TS_MW, $row->rc_timestamp );
+       $actiontext = '';
+       if( $row->rc_type == RC_LOG ) {
+               if( $row->rc_deleted & LogPage::DELETED_ACTION ) {
+                       $actiontext = wfMsgHtml('rev-deleted-event');
+               } else {
+                       $actiontext = LogPage::actionText( $row->rc_log_type, $row->rc_log_action,
+                               $titleObj, $wgUser->getSkin(), LogPage::extractParams($row->rc_params,true,true) );
+               }
+       }
        return rcFormatDiffRow( $titleObj,
                $row->rc_last_oldid, $row->rc_this_oldid,
                $timestamp,
-               $row->rc_comment );
+               ($row->rc_deleted & Revision::DELETED_COMMENT) ? wfMsgHtml('rev-deleted-comment') : $row->rc_comment,
+               $actiontext );
 }
 
-function rcFormatDiffRow( $title, $oldid, $newid, $timestamp, $comment ) {
+function rcFormatDiffRow( $title, $oldid, $newid, $timestamp, $comment, $actiontext='' ) {
        global $wgFeedDiffCutoff, $wgContLang, $wgUser;
-       $fname = 'rcFormatDiff';
-       wfProfileIn( $fname );
+       wfProfileIn( __FUNCTION__ );
 
        $skin = $wgUser->getSkin();
+       # log enties
+       if( $actiontext ) {
+               $comment = "$actiontext $comment";
+       }
        $completeText = '<p>' . $skin->formatComment( $comment ) . "</p>\n";
 
-       if( $title->getNamespace() >= 0 && $title->userCan( 'read' ) ) {
+       //NOTE: Check permissions for anonymous users, not current user.
+       //      No "privileged" version should end up in the cache.
+       //      Most feed readers will not log in anway.
+       $anon = new User();
+       $accErrors = $title->getUserPermissionsErrors( 'read', $anon, true );
+
+       if( $title->getNamespace() >= 0 && !$accErrors ) {
                if( $oldid ) {
-                       wfProfileIn( "$fname-dodiff" );
+                       wfProfileIn( __FUNCTION__."-dodiff" );
 
                        $de = new DifferenceEngine( $title, $oldid, $newid );
                        #$diffText = $de->getDiff( wfMsg( 'revisionasof',
@@ -636,7 +736,7 @@ function rcFormatDiffRow( $title, $oldid, $newid, $timestamp, $comment ) {
                                wfMsg( 'previousrevision' ), // hack
                                wfMsg( 'revisionasof',
                                        $wgContLang->timeanddate( $timestamp ) ) );
-                               
+
 
                        if ( strlen( $diffText ) > $wgFeedDiffCutoff ) {
                                // Omit large diffs
@@ -656,7 +756,7 @@ function rcFormatDiffRow( $title, $oldid, $newid, $timestamp, $comment ) {
                                $diffText = UtfNormal::cleanUp( $diffText );
                                $diffText = rcApplyDiffStyle( $diffText );
                        }
-                       wfProfileOut( "$fname-dodiff" );
+                       wfProfileOut( __FUNCTION__."-dodiff" );
                } else {
                        $rev = Revision::newFromId( $newid );
                        if( is_null( $rev ) ) {
@@ -670,7 +770,7 @@ function rcFormatDiffRow( $title, $oldid, $newid, $timestamp, $comment ) {
                $completeText .= $diffText;
        }
 
-       wfProfileOut( $fname );
+       wfProfileOut( __FUNCTION__ );
        return $completeText;
 }
 
@@ -685,21 +785,19 @@ function rcFormatDiffRow( $title, $oldid, $newid, $timestamp, $comment ) {
  */
 function rcApplyDiffStyle( $text ) {
        $styles = array(
-               'diff'             => 'background-color: white;',
-               'diff-otitle'      => 'background-color: white;',
-               'diff-ntitle'      => 'background-color: white;',
-               'diff-addedline'   => 'background: #cfc; font-size: smaller;',
-               'diff-deletedline' => 'background: #ffa; font-size: smaller;',
-               'diff-context'     => 'background: #eee; font-size: smaller;',
+               'diff'             => 'background-color: white; color:black;',
+               'diff-otitle'      => 'background-color: white; color:black;',
+               'diff-ntitle'      => 'background-color: white; color:black;',
+               'diff-addedline'   => 'background: #cfc; color:black; font-size: smaller;',
+               'diff-deletedline' => 'background: #ffa; color:black; font-size: smaller;',
+               'diff-context'     => 'background: #eee; color:black; font-size: smaller;',
                'diffchange'       => 'color: red; font-weight: bold; text-decoration: none;',
        );
-       
+
        foreach( $styles as $class => $style ) {
                $text = preg_replace( "/(<[^>]+)class=(['\"])$class\\2([^>]*>)/",
                        "\\1style=\"$style\"\\3", $text );
        }
-       
+
        return $text;
 }
-
-?>