Make sure BlockHideName really defaults to 0, not false. Fixes bug 10007.
[lhc/web/wiklou.git] / includes / SpecialWatchlist.php
index f149bcb..403ee8f 100644 (file)
@@ -1,28 +1,26 @@
 <?php
 /**
  *
- * @package MediaWiki
- * @subpackage SpecialPage
+ * @addtogroup SpecialPage
  */
 
 /**
  *
  */
-require_once( 'SpecialRecentchanges.php' );
+require_once( dirname(__FILE__) . '/SpecialRecentchanges.php' );
 
 /**
  * Constructor
- * @todo Document $par parameter.
- * @param $par String: FIXME
+ *
+ * @param $par Parameter passed to the page
  */
 function wfSpecialWatchlist( $par ) {
-       global $wgUser, $wgOut, $wgLang, $wgMemc, $wgRequest, $wgContLang;
-       global $wgUseWatchlistCache, $wgWLCacheTimeout;
+       global $wgUser, $wgOut, $wgLang, $wgRequest, $wgContLang;
        global $wgRCShowWatchingUsers, $wgEnotifWatchlist, $wgShowUpdatedMarker;
        global $wgEnotifWatchlist;
        $fname = 'wfSpecialWatchlist';
 
-       $skin =& $wgUser->getSkin();
+       $skin = $wgUser->getSkin();
        $specialTitle = SpecialPage::getTitleFor( 'Watchlist' );
        $wgOut->setRobotPolicy( 'noindex,nofollow' );
 
@@ -45,6 +43,7 @@ function wfSpecialWatchlist( $par ) {
        /* float */ 'days' => floatval( $wgUser->getOption( 'watchlistdays' ) ), /* 3.0 or 0.5, watch further below */
        /* bool  */ 'hideOwn' => (int)$wgUser->getBoolOption( 'watchlisthideown' ),
        /* bool  */ 'hideBots' => (int)$wgUser->getBoolOption( 'watchlisthidebots' ),
+       /* bool */ 'hideMinor' => (int)$wgUser->getBoolOption( 'watchlisthideminor' ),
        /* ?     */ 'namespace' => 'all',
        );
 
@@ -55,11 +54,13 @@ function wfSpecialWatchlist( $par ) {
        $prefs['days'    ] = floatval( $wgUser->getOption( 'watchlistdays' ) );
        $prefs['hideown' ] = $wgUser->getBoolOption( 'watchlisthideown' );
        $prefs['hidebots'] = $wgUser->getBoolOption( 'watchlisthidebots' );
+       $prefs['hideminor'] = $wgUser->getBoolOption( 'watchlisthideminor' );
 
        # Get query variables
        $days     = $wgRequest->getVal(  'days', $prefs['days'] );
        $hideOwn  = $wgRequest->getBool( 'hideOwn', $prefs['hideown'] );
        $hideBots = $wgRequest->getBool( 'hideBots', $prefs['hidebots'] );
+       $hideMinor = $wgRequest->getBool( 'hideMinor', $prefs['hideminor'] );
 
        # Get namespace value, if supplied, and prepare a WHERE fragment
        $nameSpace = $wgRequest->getIntOrNull( 'namespace' );
@@ -84,35 +85,25 @@ function wfSpecialWatchlist( $par ) {
         # Deleting items from watchlist
        if(($action == 'submit') && isset($remove) && is_array($id)) {
                $wgOut->addWikiText( wfMsg( 'removingchecked' ) );
-               $wgOut->addHTML( '<p>' );
+               $wgOut->addHTML( "<ul id=\"mw-unwatch-list\">\n" );
                foreach($id as $one) {
                        $t = Title::newFromURL( $one );
                        if( !is_null( $t ) ) {
                                $wl = WatchedItem::fromUserTitle( $wgUser, $t );
                                if( $wl->removeWatch() === false ) {
-                                       $wgOut->addHTML( wfMsg( 'couldntremove', htmlspecialchars($one) ) . "<br />\n" );
+                                       $wgOut->addHTML( '<li class="mw-unwatch-failure">' . wfMsg( 'couldntremove', htmlspecialchars($one) ) . "</li>\n" );
                                } else {
                                        wfRunHooks('UnwatchArticle', array(&$wgUser, new Article($t)));
-                                       $wgOut->addHTML( '(' . htmlspecialchars($one) . ')<br />' );
+                                       $wgOut->addHTML( '<li class="mw-unwatch-success">[[' . htmlspecialchars($one) . "]]</li>\n" );
                                }
                        } else {
-                               $wgOut->addHTML( wfMsg( 'iteminvalidname', htmlspecialchars($one) ) . "<br />\n" );
+                               $wgOut->addHTML( '<li class="mw-unwatch-invalid">' . wfMsg( 'iteminvalidname', htmlspecialchars($one) ) . "</li>\n" );
                        }
                }
-               $wgOut->addHTML( "</p>\n<p>" . wfMsg( 'wldone' ) . "</p>\n" );
+               $wgOut->addHTML( "</ul>\n<p>" . wfMsg( 'wldone' ) . "</p>\n" );
        }
 
-       if ( $wgUseWatchlistCache ) {
-               $memckey = wfMemcKey( 'watchlist', 'id', $wgUser->getId() );
-               $cache_s = @$wgMemc->get( $memckey );
-               if( $cache_s ){
-                       $wgOut->addWikiText( wfMsg('wlsaved') );
-                       $wgOut->addHTML( $cache_s );
-                       return;
-               }
-       }
-
-       $dbr =& wfGetDB( DB_SLAVE );
+       $dbr = wfGetDB( DB_SLAVE, 'watchlist' );
        list( $page, $watchlist, $recentchanges ) = $dbr->tableNamesN( 'page', 'watchlist', 'recentchanges' );
 
        $sql = "SELECT COUNT(*) AS n FROM $watchlist WHERE wl_user=$uid";
@@ -147,6 +138,7 @@ function wfSpecialWatchlist( $par ) {
        wfAppendToArrayIfNotDefault('days'     , $days         , $defaults, $nondefaults);
        wfAppendToArrayIfNotDefault('hideOwn'  , (int)$hideOwn , $defaults, $nondefaults);
        wfAppendToArrayIfNotDefault('hideBots' , (int)$hideBots, $defaults, $nondefaults);
+       wfAppendToArrayIfNotDefault( 'hideMinor', (int)$hideMinor, $defaults, $nondefaults );
        wfAppendToArrayIfNotDefault('namespace', $nameSpace    , $defaults, $nondefaults);
 
        if ( $days <= 0 ) {
@@ -165,7 +157,7 @@ function wfSpecialWatchlist( $par ) {
 
        /* Edit watchlist form */
        if($wgRequest->getBool('edit') || $par == 'edit' ) {
-               $wgOut->addWikiText( wfMsg( 'watchlistcontains', $wgLang->formatNum( $nitems ) ) .
+               $wgOut->addWikiText( wfMsgExt( 'watchlistcontains', array( 'parseinline' ), $wgLang->formatNum( $nitems ) ) .
                        "\n\n" . wfMsg( 'watcheditlist' ) );
 
                $wgOut->addHTML( '<form action=\'' .
@@ -249,6 +241,7 @@ function wfSpecialWatchlist( $par ) {
        # Toggles
        $andHideOwn = $hideOwn ? "AND (rc_user <> $uid)" : '';
        $andHideBots = $hideBots ? "AND (rc_bot = 0)" : '';
+       $andHideMinor = $hideMinor ? 'AND rc_minor = 0' : '';
 
        # Show watchlist header
        $header = '';
@@ -264,7 +257,8 @@ function wfSpecialWatchlist( $par ) {
                $andLatest='';
                $limitWatchlist = 'LIMIT ' . intval( $wgUser->getOption( 'wllimit' ) );
        } else {
-               $andLatest= 'AND rc_this_oldid=page_latest';
+       # Top log Ids for a page are not stored
+               $andLatest= 'AND (rc_this_oldid=page_latest OR rc_type=' . RC_LOG . ') ';
                $limitWatchlist = '';
        }
 
@@ -273,6 +267,10 @@ function wfSpecialWatchlist( $par ) {
                $wgLang->formatNum( $npages ), '',
                $specialTitle->getFullUrl( 'edit=yes' ) );
        $wgOut->addWikiText( $header );
+       
+       # Show a message about slave lag, if applicable
+       if( ( $lag = $dbr->getLag() ) > 0 )
+               $wgOut->showLagWarning( $lag );
 
        if ( $wgEnotifWatchlist && $wgShowUpdatedMarker ) {
                $wgOut->addHTML( '<form action="' .
@@ -293,6 +291,7 @@ function wfSpecialWatchlist( $par ) {
          $andLatest
          $andHideOwn
          $andHideBots
+         $andHideMinor
          $nameSpaceClause
          ORDER BY rc_timestamp DESC
          $limitWatchlist";
@@ -304,10 +303,10 @@ function wfSpecialWatchlist( $par ) {
        $wgOut->addHTML( "<hr />\n" );
 
        if($days >= 1) {
-               $wgOut->addWikiText( wfMsg( 'rcnote', $wgLang->formatNum( $numRows ),
+               $wgOut->addWikiText( wfMsgExt( 'rcnote', array( 'parseinline' ), $wgLang->formatNum( $numRows ),
                        $wgLang->formatNum( $days ), $wgLang->timeAndDate( wfTimestampNow(), true ) ) . '<br />' , false );
        } elseif($days > 0) {
-               $wgOut->addWikiText( wfMsg( 'wlnote', $wgLang->formatNum( $numRows ),
+               $wgOut->addWikiText( wfMsgExt( 'wlnote', array( 'parseinline' ), $wgLang->formatNum( $numRows ),
                        $wgLang->formatNum( round($days*24) ) ) . '<br />' , false );
        }
 
@@ -317,47 +316,65 @@ function wfSpecialWatchlist( $par ) {
        $thisTitle = SpecialPage::getTitleFor( 'Watchlist' );
        $skin = $wgUser->getSkin();
 
-       # Problems encountered using the fancier method
-       $label = $hideBots ? wfMsgHtml( 'show' ) : wfMsgHtml( 'hide' );
+       # Hide/show bot edits
+       $label = $hideBots ? wfMsgHtml( 'watchlist-show-bots' ) : wfMsgHtml( 'watchlist-hide-bots' );
        $linkBits = wfArrayToCGI( array( 'hideBots' => 1 - (int)$hideBots ), $nondefaults );
-       $link = $skin->makeKnownLinkObj( $thisTitle, $label, $linkBits );
-       $links[] = wfMsgHtml( 'wlhideshowbots', $link );
-
-       $label = $hideOwn ? wfMsgHtml( 'show' ) : wfMsgHtml( 'hide' );
+       $links[] = $skin->makeKnownLinkObj( $thisTitle, $label, $linkBits );
+       
+       # Hide/show own edits
+       $label = $hideOwn ? wfMsgHtml( 'watchlist-show-own' ) : wfMsgHtml( 'watchlist-hide-own' );
        $linkBits = wfArrayToCGI( array( 'hideOwn' => 1 - (int)$hideOwn ), $nondefaults );
-       $link = $skin->makeKnownLinkObj( $thisTitle, $label, $linkBits );
-       $links[] = wfMsgHtml( 'wlhideshowown', $link );
+       $links[] = $skin->makeKnownLinkObj( $thisTitle, $label, $linkBits );
+
+       # Hide/show minor edits
+       $label = $hideMinor ? wfMsgHtml( 'watchlist-show-minor' ) : wfMsgHtml( 'watchlist-hide-minor' );
+       $linkBits = wfArrayToCGI( array( 'hideMinor' => 1 - (int)$hideMinor ), $nondefaults );
+       $links[] = $skin->makeKnownLinkObj( $thisTitle, $label, $linkBits );
 
        $wgOut->addHTML( implode( ' | ', $links ) );
 
        # Form for namespace filtering
-       $wgOut->addHTML( "\n" .
-               wfOpenElement( 'form', array(
-                               'method' => 'post',
-                               'action' => $thisTitle->getLocalURL(),
-                       ) ) .
-               wfMsgExt( 'namespace', array( 'parseinline') ) .
-               HTMLnamespaceselector( $nameSpace, '' ) . "\n" .
-               ( $hideOwn ? wfHidden('hideown', 1)."\n" : '' ) .
-               ( $hideBots ? wfHidden('hidebots', 1)."\n" : '' ) .
-               wfHidden( 'days', $days ) . "\n" .
-               wfSubmitButton( wfMsgExt( 'allpagessubmit', array( 'escape') ) ) . "\n" .
-               wfCloseElement( 'form' ) . "\n"
-       );
-
-       if ( $numRows == 0 ) {
-               $wgOut->addWikitext( "<br />" . wfMsg( 'watchnochange' ), false );
+       $form  = Xml::openElement( 'form', array( 'method' => 'post', 'action' => $thisTitle->getLocalUrl() ) );
+       $form .= '<p>';
+       $form .= Xml::label( wfMsg( 'namespace' ), 'namespace' ) . '&nbsp;';
+       $form .= Xml::namespaceSelector( $nameSpace, '' ) . '&nbsp;';
+       $form .= Xml::submitButton( wfMsg( 'allpagessubmit' ) ) . '</p>';
+       $form .= Xml::hidden( 'days', $days );
+       if( $hideOwn )
+               $form .= Xml::hidden( 'hideOwn', 1 );
+       if( $hideBots )
+               $form .= Xml::hidden( 'hideBots', 1 );
+       if( $hideMinor )
+               $form .= Xml::hidden( 'hideMinor', 1 );
+       $form .= Xml::closeElement( 'form' );
+       $wgOut->addHtml( $form );
+
+       # If there's nothing to show, stop here
+       if( $numRows == 0 ) {
+               $wgOut->addWikiText( wfMsgNoTrans( 'watchnochange' ) );
                return;
        }
 
        /* End bottom header */
 
+       /* Do link batch query */
+       $linkBatch = new LinkBatch;
+       while ( $row = $dbr->fetchObject( $res ) ) {
+               $userNameUnderscored = str_replace( ' ', '_', $row->rc_user_text );
+               if ( $row->rc_user != 0 ) {
+                       $linkBatch->add( NS_USER, $userNameUnderscored );
+               }
+               $linkBatch->add( NS_USER_TALK, $userNameUnderscored );
+       }
+       $linkBatch->execute();
+       $dbr->dataSeek( $res, 0 );
+
        $list = ChangesList::newFromUser( $wgUser );
 
        $s = $list->beginRecentChangesList();
        $counter = 1;
        while ( $obj = $dbr->fetchObject( $res ) ) {
-               # Make fake RC entry
+               # Make RC entry
                $rc = RecentChange::newFromRow( $obj );
                $rc->counter = $counter++;
 
@@ -384,10 +401,6 @@ function wfSpecialWatchlist( $par ) {
        $dbr->freeResult( $res );
        $wgOut->addHTML( $s );
 
-       if ( $wgUseWatchlistCache ) {
-               $wgMemc->set( $memckey, $s, $wgWLCacheTimeout);
-       }
-
 }
 
 function wlHoursLink( $h, $page, $options = array() ) {
@@ -438,7 +451,7 @@ function wlCutoffLinks( $days, $page = 'Watchlist', $options = array() ) {
  * @return integer
  */
 function wlCountItems( &$user, $talk = true ) {
-       $dbr =& wfGetDB( DB_SLAVE );
+       $dbr = wfGetDB( DB_SLAVE, 'watchlist' );
 
        # Fetch the raw count
        $res = $dbr->select( 'watchlist', 'COUNT(*) AS count', array( 'wl_user' => $user->mId ), 'wlCountItems' );
@@ -474,7 +487,7 @@ function wlHandleClear( &$out, &$request, $par ) {
                        # See if we're clearing or confirming
                        if( $request->wasPosted() && $wgUser->matchEditToken( $request->getText( 'token' ), 'clearwatchlist' ) ) {
                                # Clearing, so do it and report the result
-                               $dbw =& wfGetDB( DB_MASTER );
+                               $dbw = wfGetDB( DB_MASTER );
                                $dbw->delete( 'watchlist', array( 'wl_user' => $wgUser->mId ), 'wlHandleClear' );
                                $out->addWikiText( wfMsgExt( 'watchlistcleardone', array( 'parsemag', 'escape'), $wgLang->formatNum( $count ) ) );
                                $out->returnToMain();