Pass the user as an argument to 'isValidPassword' hook callbacks; see docs/hooks...
[lhc/web/wiklou.git] / includes / SpecialWatchlist.php
index 97093d0..924edc2 100644 (file)
@@ -1,81 +1,91 @@
 <?php
 /**
  *
- * @package MediaWiki
- * @subpackage SpecialPage
+ * @addtogroup SpecialPage
  */
 
 /**
  *
  */
-require_once( 'SpecialRecentchanges.php' );
-require_once( 'WatchedItem.php' );
+require_once( dirname(__FILE__) . '/SpecialRecentchanges.php' );
 
 /**
- * constructor
+ * Constructor
+ *
+ * @param $par Parameter passed to the page
  */
 function wfSpecialWatchlist( $par ) {
-       global $wgUser, $wgOut, $wgLang, $wgTitle, $wgMemc, $wgRequest, $wgContLang;;
-       global $wgUseWatchlistCache, $wgWLCacheTimeout, $wgDBname;
+       global $wgUser, $wgOut, $wgLang, $wgRequest, $wgContLang;
        global $wgRCShowWatchingUsers, $wgEnotifWatchlist, $wgShowUpdatedMarker;
+       global $wgEnotifWatchlist;
        $fname = 'wfSpecialWatchlist';
 
-       $wgOut->setPagetitle( wfMsg( 'watchlist' ) );
-       $sub = wfMsg( 'watchlistsub', $wgUser->getName() );
-       $wgOut->setSubtitle( $sub );
-       $wgOut->setRobotpolicy( 'noindex,nofollow' );
-
-       $specialTitle = Title::makeTitle( NS_SPECIAL, 'Watchlist' );
+       $skin = $wgUser->getSkin();
+       $specialTitle = SpecialPage::getTitleFor( 'Watchlist' );
+       $wgOut->setRobotPolicy( 'noindex,nofollow' );
 
+       # Anons don't get a watchlist
        if( $wgUser->isAnon() ) {
-               $wgOut->addWikiText( wfMsg( 'nowatchlist' ) );
+               $wgOut->setPageTitle( wfMsg( 'watchnologin' ) );
+               $llink = $skin->makeKnownLinkObj( SpecialPage::getTitleFor( 'Userlogin' ), wfMsgHtml( 'loginreqlink' ), 'returnto=' . $specialTitle->getPrefixedUrl() );
+               $wgOut->addHtml( wfMsgWikiHtml( 'watchlistanontext', $llink ) );
                return;
        }
+       
+       $wgOut->setPageTitle( wfMsg( 'watchlist' ) );
+       
+       $sub  = wfMsgExt( 'watchlistfor', 'parseinline', $wgUser->getName() );
+       $sub .= '<br />' . WatchlistEditor::buildTools( $wgUser->getSkin() );
+       $wgOut->setSubtitle( $sub );
 
-       # Get query variables
-       $days = $wgRequest->getVal( 'days' );
-       $action = $wgRequest->getVal( 'action' );
-       $remove = $wgRequest->getVal( 'remove' );
-       $hideOwn = $wgRequest->getBool( 'hideOwn' );    
-       $id = $wgRequest->getArray( 'id' );
-
-       $uid = $wgUser->getID();
-       if( $wgRequest->getVal( 'reset' ) && $wgRequest->wasPosted() ) {
+       if( ( $mode = WatchlistEditor::getMode( $wgRequest, $par ) ) !== false ) {
+               $editor = new WatchlistEditor();
+               $editor->execute( $wgUser, $wgOut, $wgRequest, $mode );
+               return;
+       }
+       
+       $uid = $wgUser->getId();
+       if( $wgEnotifWatchlist && $wgRequest->getVal( 'reset' ) && $wgRequest->wasPosted() ) {
                $wgUser->clearAllNotifications( $uid );
+               $wgOut->redirect( $specialTitle->getFullUrl() );
+               return;
        }
 
+       $defaults = array(
+       /* 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',
+       );
 
-       if(($action == 'submit') && isset($remove) && is_array($id)) {
-               $wgOut->addWikiText( wfMsg( 'removingchecked' ) );
-               $wgOut->addHTML( '<p>' );
-               foreach($id as $one) {
-                       $t = Title::newFromURL( $one );
-                       if( !is_null( $t ) ) {
-                               $wl = WatchedItem::fromUserTitle( $wgUser, $t );
-                               if( $wl->removeWatch() === false ) {
-                                       $wgOut->addHTML( "<br />\n" . wfMsg( 'couldntremove', htmlspecialchars($one) ) );
-                               } else {
-                                       $wgOut->addHTML( ' (' . htmlspecialchars($one) . ')' );
-                               }
-                       } else {
-                               $wgOut->addHTML( "<br />\n" . wfMsg( 'iteminvalidname', htmlspecialchars($one) ) );
-                       }
-               }
-               $wgOut->addHTML( "done.</p>\n" );
-       }
+       extract($defaults);
 
-       if ( $wgUseWatchlistCache ) {
-               $memckey = "$wgDBname:watchlist:id:" . $wgUser->getId();
-               $cache_s = @$wgMemc->get( $memckey );
-               if( $cache_s ){
-                       $wgOut->addWikiText( wfMsg('wlsaved') );
-                       $wgOut->addHTML( $cache_s );
-                       return;
-               }
+       # Extract variables from the request, falling back to user preferences or
+       # other default values if these don't exist
+       $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' );
+       if( !is_null( $nameSpace ) ) {
+               $nameSpace = intval( $nameSpace );
+               $nameSpaceClause = " AND rc_namespace = $nameSpace";
+       } else {
+               $nameSpace = '';
+               $nameSpaceClause = '';
        }
 
-       $dbr =& wfGetDB( DB_SLAVE );
-       extract( $dbr->tableNames( 'page', 'revision', 'watchlist', 'recentchanges' ) );
+       $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";
        $res = $dbr->query( $sql, $fname );
@@ -83,74 +93,45 @@ function wfSpecialWatchlist( $par ) {
 
 #      Patch *** A1 *** (see A2 below)
 #      adjust for page X, talk:page X, which are both stored separately, but treated together
-#      $nitems = $s->n / 2;
-       $nitems = $s->n;
+       $nitems = floor($s->n / 2);
+#      $nitems = $s->n;
 
        if($nitems == 0) {
                $wgOut->addWikiText( wfMsg( 'nowatchlist' ) );
                return;
        }
 
-       if ( is_null( $days ) ) {
-               $big = 1000;
+       if( is_null($days) || !is_numeric($days) ) {
+               $big = 1000; /* The magical big */
                if($nitems > $big) {
                        # Set default cutoff shorter
-                       $days = (12.0 / 24.0); # 12 hours...
+                       $days = $defaults['days'] = (12.0 / 24.0); # 12 hours...
                } else {
-                       $days = 3; # longer cutoff for shortlisters
+                       $days = $defaults['days']; # default cutoff for shortlisters
                }
        } else {
                $days = floatval($days);
        }
 
+       // Dump everything here
+       $nondefaults = array();
+
+       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 ) {
-               $docutoff = '';
-               $cutoff = false;
-               $npages = wfMsg( 'watchlistall1' );
+               $andcutoff = '';
        } else {
-               $docutoff = "AND rev_timestamp > '" .
-                 ( $cutoff = $dbr->timestamp( time() - intval( $days * 86400 ) ) )
-                 . "'";
-               $sql = "SELECT COUNT(*) AS n FROM $page, $revision  WHERE rev_timestamp>'$cutoff' AND page_id=rev_page";
+               $andcutoff = "AND rc_timestamp > '".$dbr->timestamp( time() - intval( $days * 86400 ) )."'";
+               /*
+               $sql = "SELECT COUNT(*) AS n FROM $page, $revision  WHERE rev_timestamp>'$cutoff' AND page_id=rev_page";
                $res = $dbr->query( $sql, $fname );
                $s = $dbr->fetchObject( $res );
                $npages = $s->n;
-
-       }
-
-       if($wgRequest->getBool('edit') || $par == 'edit' ) {
-               $wgOut->addWikiText( wfMsg( 'watchlistcontains', $wgLang->formatNum( $nitems ) ) .
-                       "\n\n" . wfMsg( 'watcheditlist' ) );
-
-               $wgOut->addHTML( '<form action=\'' .
-                       $specialTitle->escapeLocalUrl( 'action=submit' ) .
-                       "' method='post'>\n" .
-                       "<ul>\n" );
-
-#              Patch A2
-#              The following was proposed by KTurner 07.11.2004 to T.Gries
-#              $sql = "SELECT distinct (wl_namespace & ~1),wl_title FROM $watchlist WHERE wl_user=$uid";
-               $sql = "SELECT wl_namespace,wl_title FROM $watchlist WHERE wl_user=$uid";
-
-               $res = $dbr->query( $sql, $fname );
-               $sk = $wgUser->getSkin();
-               while( $s = $dbr->fetchObject( $res ) ) {
-                       $t = Title::makeTitle( $s->wl_namespace, $s->wl_title );
-                       if( is_null( $t ) ) {
-                               $wgOut->addHTML( '<!-- bad title "' . htmlspecialchars( $s->wl_title ) . '" in namespace ' . IntVal( $s->wl_namespace ) . " -->\n" );
-                       } else {
-                               $t = $t->getPrefixedText();
-                               $wgOut->addHTML( '<li><input type="checkbox" name="id[]" value="' . htmlspecialchars($t) . '" />' .
-                                       $sk->makeLink( $t, $t ) .
-                                       "</li>\n" );
-                       }
-               }
-               $wgOut->addHTML( "</ul>\n" .
-                       "<input type='submit' name='remove' value=\"" .
-                       htmlspecialchars( wfMsg( "removechecked" ) ) . "\" />\n" .
-                       "</form>\n" );
-
-               return;
+               */
        }
 
        # If the watchlist is relatively short, it's simplest to zip
@@ -160,39 +141,38 @@ function wfSpecialWatchlist( $par ) {
        # through the time-sorted page list checking for watched items.
 
        # Up estimate of watched items by 15% to compensate for talk pages...
-       if( $cutoff && ( $nitems*1.15 > $npages ) ) {
-               $x = 'rev_timestamp';
-               $y = wfMsg( 'watchmethod-recent' );
-               # TG patch: here we do not consider pages and their talk pages equivalent - why should we ?
-               # The change results in talk-pages not automatically included in watchlists, when their parent page is included
-               # $z = "wl_namespace=cur_namespace & ~1";
-               $z = 'wl_namespace=page_namespace';
-       } else {
-               $x = 'page_timestamp';
-               $y = wfMsg( 'watchmethod-list' );
-               # TG patch: here we do not consider pages and their talk pages equivalent - why should we ?
-               # The change results in talk-pages not automatically included in watchlists, when their parent page is included
-               # $z = "(wl_namespace=cur_namespace OR wl_namespace+1=cur_namespace)";
-               $z = 'wl_namespace=page_namespace';
-       }
 
-       $andHideOwn = $hideOwn ? "AND (rev_user <> $uid)" : '';
+       # Toggles
+       $andHideOwn = $hideOwn ? "AND (rc_user <> $uid)" : '';
+       $andHideBots = $hideBots ? "AND (rc_bot = 0)" : '';
+       $andHideMinor = $hideMinor ? 'AND rc_minor = 0' : '';
 
        # Show watchlist header
        $header = '';
        if( $wgUser->getOption( 'enotifwatchlistpages' ) && $wgEnotifWatchlist) {
                $header .= wfMsg( 'wlheader-enotif' ) . "\n";
        }
-       if ( $wgShowUpdatedMarker ) {
+       if ( $wgEnotifWatchlist && $wgShowUpdatedMarker ) {
                $header .= wfMsg( 'wlheader-showupdated' ) . "\n";
        }
 
-       $header .= wfMsg( 'watchdetails', $wgLang->formatNum( $nitems ), 
-               $wgLang->formatNum( $npages ), $y,
-               $specialTitle->getFullUrl( 'edit=yes' ) );
+  # Toggle watchlist content (all recent edits or just the latest)
+       if( $wgUser->getOption( 'extendwatchlist' )) {
+               $andLatest='';
+               $limitWatchlist = 'LIMIT ' . intval( $wgUser->getOption( 'wllimit' ) );
+       } else {
+               $andLatest= 'AND rc_this_oldid=page_latest';
+               $limitWatchlist = '';
+       }
+
+       $header .= wfMsgExt( 'watchlist-details', array( 'parsemag' ), $wgLang->formatNum( $nitems ) );
        $wgOut->addWikiText( $header );
-       
-       if ( $wgShowUpdatedMarker ) {
+
+       # Show a message about slave lag, if applicable
+       if( ( $lag = $dbr->getLag() ) > 0 )
+               $wgOut->showLagWarning( $lag );
+
+       if ( $wgEnotifWatchlist && $wgShowUpdatedMarker ) {
                $wgOut->addHTML( '<form action="' .
                        $specialTitle->escapeLocalUrl() .
                        '" method="post"><input type="submit" name="dummy" value="' .
@@ -201,53 +181,101 @@ function wfSpecialWatchlist( $par ) {
                        "\n\n" );
        }
 
-       $use_index = $dbr->useIndexClause( $x );
-       $sql = "SELECT
-  page_namespace,page_title,rev_comment, page_id,
-  rev_user,rev_user_text,rev_timestamp,rev_minor_edit,page_is_new,wl_notificationtimestamp
-  FROM $watchlist,$page,$revision  $use_index
-  WHERE wl_user=$uid
-  $andHideOwn
-  AND $z
-  AND wl_title=page_title
-  AND page_latest=rev_id
-  $docutoff
-  ORDER BY rev_timestamp DESC";
-
+       $sql = "SELECT *
+         FROM $watchlist,$recentchanges,$page
+         WHERE wl_user=$uid
+         AND wl_namespace=rc_namespace
+         AND wl_title=rc_title
+         AND rc_cur_id=page_id
+         $andcutoff
+         $andLatest
+         $andHideOwn
+         $andHideBots
+         $andHideMinor
+         $nameSpaceClause
+         ORDER BY rc_timestamp DESC
+         $limitWatchlist";
 
        $res = $dbr->query( $sql, $fname );
        $numRows = $dbr->numRows( $res );
-       if($days >= 1)
-               $note = wfMsg( 'rcnote', $wgLang->formatNum( $numRows ), $wgLang->formatNum( $days ) );
-       elseif($days > 0)
-               $note = wfMsg( 'wlnote', $wgLang->formatNum( $numRows ), $wgLang->formatNum( round($days*24) ) );
-       else
-               $note = '';
-       $wgOut->addHTML( "\n<hr />\n{$note}\n<br />" );
-       $note = wlCutoffLinks( $days );
-       $wgOut->addHTML( "{$note}\n" );
-       
-       $sk = $wgUser->getSkin();
-       $s = $sk->makeKnownLink(
-               $wgContLang->specialPage( 'Watchlist' ),
-               (0 == $hideOwn) ? wfMsg( 'wlhide' ) : wfMsg( 'wlshow' ),        
-               'hideOwn=' . $wgLang->formatNum( 1-$hideOwn ) );
-               
-       $note = wfMsg( "wlhideshowown", $s );
-       $wgOut->addHTML( "\n<br />{$note}\n<br />" );
+
+       /* Start bottom header */
+       $wgOut->addHTML( "<hr />\n" );
+
+       if($days >= 1) {
+               $wgOut->addWikiText( wfMsgExt( 'rcnote', array( 'parseinline' ), $wgLang->formatNum( $numRows ),
+                       $wgLang->formatNum( $days ), $wgLang->timeAndDate( wfTimestampNow(), true ) ) . '<br />' , false );
+       } elseif($days > 0) {
+               $wgOut->addWikiText( wfMsgExt( 'wlnote', array( 'parseinline' ), $wgLang->formatNum( $numRows ),
+                       $wgLang->formatNum( round($days*24) ) ) . '<br />' , false );
+       }
+
+       $wgOut->addHTML( "\n" . wlCutoffLinks( $days, 'Watchlist', $nondefaults ) . "<br />\n" );
+
+       # Spit out some control panel links
+       $thisTitle = SpecialPage::getTitleFor( 'Watchlist' );
+       $skin = $wgUser->getSkin();
+
+       # Hide/show bot edits
+       $label = $hideBots ? wfMsgHtml( 'watchlist-show-bots' ) : wfMsgHtml( 'watchlist-hide-bots' );
+       $linkBits = wfArrayToCGI( array( 'hideBots' => 1 - (int)$hideBots ), $nondefaults );
+       $links[] = $skin->makeKnownLinkObj( $thisTitle, $label, $linkBits );
        
-       if ( $numRows == 0 ) {
-               $wgOut->addHTML( '<p><i>' . wfMsg( 'watchnochange' ) . '</i></p>' );
+       # Hide/show own edits
+       $label = $hideOwn ? wfMsgHtml( 'watchlist-show-own' ) : wfMsgHtml( 'watchlist-hide-own' );
+       $linkBits = wfArrayToCGI( array( 'hideOwn' => 1 - (int)$hideOwn ), $nondefaults );
+       $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
+       $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;
        }
 
-       $sk = $wgUser->getSkin();
-       $list =& new ChangesList( $sk );
+       /* 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
-               $rc = RecentChange::newFromCurRow( $obj );
+               # Make RC entry
+               $rc = RecentChange::newFromRow( $obj );
                $rc->counter = $counter++;
 
                if ( $wgShowUpdatedMarker ) {
@@ -258,8 +286,8 @@ function wfSpecialWatchlist( $par ) {
                }
 
                if ($wgRCShowWatchingUsers && $wgUser->getOption( 'shownumberswatching' )) {
-                       $sql3 = "SELECT COUNT(*) AS n FROM $watchlist WHERE wl_title='" .wfStrencode($obj->cur_title). "' AND wl_namespace='{$obj->cur_namespace}'" ;
-                       $res3 = $dbr->query( $sql3, DB_READ, $fname );
+                       $sql3 = "SELECT COUNT(*) AS n FROM $watchlist WHERE wl_title='" .$dbr->strencode($obj->page_title). "' AND wl_namespace='{$obj->page_namespace}'" ;
+                       $res3 = $dbr->query( $sql3, $fname );
                        $x = $dbr->fetchObject( $res3 );
                        $rc->numberofWatchingusers = $x->n;
                } else {
@@ -273,50 +301,67 @@ function wfSpecialWatchlist( $par ) {
        $dbr->freeResult( $res );
        $wgOut->addHTML( $s );
 
-       if ( $wgUseWatchlistCache ) {
-               $wgMemc->set( $memckey, $s, $wgWLCacheTimeout);
-       }
-
 }
 
-
-function wlHoursLink( $h, $page ) {
+function wlHoursLink( $h, $page, $options = array() ) {
        global $wgUser, $wgLang, $wgContLang;
        $sk = $wgUser->getSkin();
        $s = $sk->makeKnownLink(
          $wgContLang->specialPage( $page ),
          $wgLang->formatNum( $h ),
-         'days=' . ($h / 24.0) );
+         wfArrayToCGI( array('days' => ($h / 24.0)), $options ) );
        return $s;
 }
 
-
-function wlDaysLink( $d, $page ) {
+function wlDaysLink( $d, $page, $options = array() ) {
        global $wgUser, $wgLang, $wgContLang;
        $sk = $wgUser->getSkin();
        $s = $sk->makeKnownLink(
          $wgContLang->specialPage( $page ),
-         ($d ? $wgLang->formatNum( $d ) : wfMsg( 'watchlistall2' ) ), "days=$d" );
+         ($d ? $wgLang->formatNum( $d ) : wfMsgHtml( 'watchlistall2' ) ),
+         wfArrayToCGI( array('days' => $d), $options ) );
        return $s;
 }
 
-function wlCutoffLinks( $days, $page = 'Watchlist' )
-{
+/**
+ * Returns html
+ */
+function wlCutoffLinks( $days, $page = 'Watchlist', $options = array() ) {
        $hours = array( 1, 2, 6, 12 );
        $days = array( 1, 3, 7 );
-       $cl = '';
        $i = 0;
        foreach( $hours as $h ) {
-               $hours[$i++] = wlHoursLink( $h, $page );
+               $hours[$i++] = wlHoursLink( $h, $page, $options );
        }
        $i = 0;
        foreach( $days as $d ) {
-               $days[$i++] = wlDaysLink( $d, $page );
+               $days[$i++] = wlDaysLink( $d, $page, $options );
        }
-       return wfMsg ('wlshowlast',
+       return wfMsgExt('wlshowlast',
+               array('parseinline', 'replaceafter'),
                implode(' | ', $hours),
                implode(' | ', $days),
-               wlDaysLink( 0, $page ) );
+               wlDaysLink( 0, $page, $options ) );
 }
 
-?>
+/**
+ * Count the number of items on a user's watchlist
+ *
+ * @param $talk Include talk pages
+ * @return integer
+ */
+function wlCountItems( &$user, $talk = true ) {
+       $dbr = wfGetDB( DB_SLAVE, 'watchlist' );
+
+       # Fetch the raw count
+       $res = $dbr->select( 'watchlist', 'COUNT(*) AS count', array( 'wl_user' => $user->mId ), 'wlCountItems' );
+       $row = $dbr->fetchObject( $res );
+       $count = $row->count;
+       $dbr->freeResult( $res );
+
+       # Halve to remove talk pages if needed
+       if( !$talk )
+               $count = floor( $count / 2 );
+
+       return( $count );
+}
\ No newline at end of file