* Not showing " : " in recentchanges if the rcComment is empty
[lhc/web/wiklou.git] / includes / SpecialWatchlist.php
index 234b9df..4d83271 100644 (file)
@@ -8,28 +8,28 @@
 /**
  *
  */
-require_once( "SpecialRecentchanges.php" );
-require_once( "WatchedItem.php" );
+require_once( 'SpecialRecentchanges.php' );
+require_once( 'WatchedItem.php' );
 
 /**
  * constructor
  */
-function wfSpecialWatchlist() {
-       global $wgUser, $wgOut, $wgLang, $wgTitle, $wgMemc, $wgRequest;
+function wfSpecialWatchlist( $par ) {
+       global $wgUser, $wgOut, $wgLang, $wgTitle, $wgMemc, $wgRequest, $wgContLang;;
        global $wgUseWatchlistCache, $wgWLCacheTimeout, $wgDBname;
-       global $wgEnotif, $wgShowUpdatedMarker, $wgRCShowWatchingUsers;
-       $fname = "wfSpecialWatchlist";
+       global $wgRCShowWatchingUsers, $wgEnotifWatchlist, $wgShowUpdatedMarker;
+       global $wgEnotifWatchlist;
+       $fname = 'wfSpecialWatchlist';
 
-       $wgOut->setPagetitle( wfMsg( "watchlist" ) );
-       $sub = wfMsg( "watchlistsub", $wgUser->getName() );
+       $wgOut->setPagetitle( wfMsg( 'watchlist' ) );
+       $sub = wfMsg( 'watchlistsub', $wgUser->getName() );
        $wgOut->setSubtitle( $sub );
-       $wgOut->setRobotpolicy( "noindex,nofollow" );
+       $wgOut->setRobotpolicy( 'noindex,nofollow' );
 
-       $specialTitle = Title::makeTitle( NS_SPECIAL, "Watchlist" );
+       $specialTitle = Title::makeTitle( NS_SPECIAL, 'Watchlist' );
 
-       $uid = $wgUser->getID();
-       if( $uid == 0 ) {
-               $wgOut->addHTML( wfMsg( "nowatchlist" ) );
+       if( $wgUser->isAnon() ) {
+               $wgOut->addWikiText( wfMsg( 'nowatchlist' ) );
                return;
        }
 
@@ -37,37 +37,39 @@ function wfSpecialWatchlist() {
        $days = $wgRequest->getVal( 'days' );
        $action = $wgRequest->getVal( 'action' );
        $remove = $wgRequest->getVal( 'remove' );
+       $hideOwn = $wgRequest->getBool( 'hideOwn' );    
        $id = $wgRequest->getArray( 'id' );
 
-       $wgOut->addHTML( wfMsg( "email_notification_infotext" ) );
-
-       if( $wgRequest->getVal( 'reset' ) == 'all' ) {
+       $uid = $wgUser->getID();
+       if( $wgEnotifWatchlist && $wgRequest->getVal( 'reset' ) && $wgRequest->wasPosted() ) {
                $wgUser->clearAllNotifications( $uid );
        }
 
-       if(($action == "submit") && isset($remove) && is_array($id)) {
-               $wgOut->addHTML( wfMsg( "removingchecked" ) );
+
+       if(($action == 'submit') && isset($remove) && is_array($id)) {
+               $wgOut->addWikiText( wfMsg( 'removingchecked' ) );
+               $wgOut->addHTML( '<p>' );
                foreach($id as $one) {
                        $t = Title::newFromURL( $one );
-                       if($t->getDBkey() != "") {
+                       if( !is_null( $t ) ) {
                                $wl = WatchedItem::fromUserTitle( $wgUser, $t );
                                if( $wl->removeWatch() === false ) {
-                                       $wgOut->addHTML( "<br />\n" . wfMsg( "couldntremove", htmlspecialchars($one) ) );
+                                       $wgOut->addHTML( "<br />\n" . wfMsg( 'couldntremove', htmlspecialchars($one) ) );
                                } else {
-                                       $wgOut->addHTML( " (" . htmlspecialchars($one) . ")" );
+                                       $wgOut->addHTML( ' (' . htmlspecialchars($one) . ')' );
                                }
                        } else {
-                               $wgOut->addHTML( "<br />\n" . wfMsg( "iteminvalidname", htmlspecialchars($one) ) );
+                               $wgOut->addHTML( "<br />\n" . wfMsg( 'iteminvalidname', htmlspecialchars($one) ) );
                        }
                }
-               $wgOut->addHTML( "done.\n<p>" );
+               $wgOut->addHTML( "done.</p>\n" );
        }
 
        if ( $wgUseWatchlistCache ) {
                $memckey = "$wgDBname:watchlist:id:" . $wgUser->getId();
                $cache_s = @$wgMemc->get( $memckey );
                if( $cache_s ){
-                       $wgOut->addHTML( wfMsg("wlsaved") );
+                       $wgOut->addWikiText( wfMsg('wlsaved') );
                        $wgOut->addHTML( $cache_s );
                        return;
                }
@@ -86,10 +88,10 @@ function wfSpecialWatchlist() {
        $nitems = $s->n;
 
        if($nitems == 0) {
-        $wgOut->addHTML( wfMsg( "nowatchlist" ) );
-        return;
+               $wgOut->addWikiText( wfMsg( 'nowatchlist' ) );
+               return;
        }
-       
+
        if ( is_null( $days ) ) {
                $big = 1000;
                if($nitems > $big) {
@@ -105,7 +107,7 @@ function wfSpecialWatchlist() {
        if ( $days <= 0 ) {
                $docutoff = '';
                $cutoff = false;
-               $npages = wfMsg( "all" );
+               $npages = wfMsg( 'watchlistall1' );
        } else {
                $docutoff = "AND rev_timestamp > '" .
                  ( $cutoff = $dbr->timestamp( time() - intval( $days * 86400 ) ) )
@@ -117,14 +119,13 @@ function wfSpecialWatchlist() {
 
        }
 
-       if(isset($_REQUEST['magic'])) {
-               $wgOut->addHTML( wfMsg( "watchlistcontains", $wgLang->formatNum( $nitems ) ) .
-                       "<p>" . wfMsg( "watcheditlist" ) . "</p>\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" );
+               $wgOut->addHTML( '<form action=\'' .
+                       $specialTitle->escapeLocalUrl( 'action=submit' ) .
+                       "' method='post'>\n" );
 
 #              Patch A2
 #              The following was proposed by KTurner 07.11.2004 to T.Gries
@@ -133,21 +134,42 @@ function wfSpecialWatchlist() {
 
                $res = $dbr->query( $sql, $fname );
                $sk = $wgUser->getSkin();
+
+               $list = array();
                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" );
+                       $list[$s->wl_namespace][] = $s->wl_title;
+               }
+               
+               // TODO: Display a TOC
+               foreach($list as $ns => $titles) {
+                       if (Namespace::isTalk($ns))
+                               continue;
+                       if ($ns != NS_MAIN) 
+                               $wgOut->addHTML( '<h2>' . $wgContLang->getFormattedNsText( $ns ) . '</h2>' );
+                       $wgOut->addHTML( '<ul>' );
+                       foreach($titles as $title) {
+                               $t = Title::makeTitle( $ns, $title );
+                               if( is_null( $t ) ) {
+                                       $wgOut->addHTML(
+                                               '<!-- bad title "' .
+                                               htmlspecialchars( $s->wl_title ) . '" in namespace ' . $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>' );
                }
-               $wgOut->addHTML( "</ul>\n" .
+               $wgOut->addHTML(
                        "<input type='submit' name='remove' value=\"" .
                        htmlspecialchars( wfMsg( "removechecked" ) ) . "\" />\n" .
-                       "</form>\n" );
+                       "</form>\n"
+               );
 
                return;
        }
@@ -160,32 +182,53 @@ function wfSpecialWatchlist() {
 
        # 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" );
+               $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";
+               $z = 'wl_namespace=page_namespace';
        } else {
-               $x = "page_timestamp";
-               $y = wfMsg( "watchmethod-list" );
+               $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";
+               $z = 'wl_namespace=page_namespace';
        }
 
+       $andHideOwn = $hideOwn ? "AND (rev_user <> $uid)" : '';
+
+       # Show watchlist header
+       $header = '';
+       if( $wgUser->getOption( 'enotifwatchlistpages' ) && $wgEnotifWatchlist) {
+               $header .= wfMsg( 'wlheader-enotif' ) . "\n";
+       }
+       if ( $wgEnotifWatchlist && $wgShowUpdatedMarker ) {
+               $header .= wfMsg( 'wlheader-showupdated' ) . "\n";
+       }
 
-       $wgOut->addHTML( "<i>" . wfMsg( "watchdetails",
-               $wgLang->formatNum( $nitems ), $wgLang->formatNum( $npages ), $y,
-               $specialTitle->escapeLocalUrl( "magic=yes" ) ) . "</i><br />\n" );
+       $header .= wfMsg( 'watchdetails', $wgLang->formatNum( $nitems / 2 ), 
+               $wgLang->formatNum( $npages ), $y,
+               $specialTitle->getFullUrl( 'edit=yes' ) );
+       $wgOut->addWikiText( $header );
+       
+       if ( $wgEnotifWatchlist && $wgShowUpdatedMarker ) {
+               $wgOut->addHTML( '<form action="' .
+                       $specialTitle->escapeLocalUrl() .
+                       '" method="post"><input type="submit" name="dummy" value="' .
+                       htmlspecialchars( wfMsg( 'enotif_reset' ) ) .
+                       '" /><input type="hidden" name="reset" value="all" /></form>' .
+                       "\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
+  rev_user,rev_user_text,rev_timestamp,rev_minor_edit,rev_id,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
@@ -196,17 +239,26 @@ function wfSpecialWatchlist() {
        $res = $dbr->query( $sql, $fname );
        $numRows = $dbr->numRows( $res );
        if($days >= 1)
-               $note = wfMsg( "rcnote", $wgLang->formatNum( $numRows ), $wgLang->formatNum( $days ) );
+               $note = wfMsg( 'rcnote', $wgLang->formatNum( $numRows ), $wgLang->formatNum( $days ) );
        elseif($days > 0)
-               $note = wfMsg( "wlnote", $wgLang->formatNum( $numRows ), $wgLang->formatNum( round($days*24) ) );
+               $note = wfMsg( 'wlnote', $wgLang->formatNum( $numRows ), $wgLang->formatNum( round($days*24) ) );
        else
-               $note = "";
+               $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 />" );
+       
        if ( $numRows == 0 ) {
-               $wgOut->addHTML( "<p><i>" . wfMsg( "watchnochange" ) . "</i></p>" );
+               $wgOut->addHTML( '<p><i>' . wfMsg( 'watchnochange' ) . '</i></p>' );
                return;
        }
 
@@ -219,14 +271,15 @@ function wfSpecialWatchlist() {
                $rc = RecentChange::newFromCurRow( $obj );
                $rc->counter = $counter++;
 
-               if ($wgShowUpdatedMarker && $wgUser->getOption( 'showupdated' )) {
-                       $rc->notificationtimestamp = $obj->wl_notificationtimestamp;
+               if ( $wgShowUpdatedMarker ) {
+                       $updated = $obj->wl_notificationtimestamp;
                } else {
-                       $rc->notificationtimestamp = false;
+                       // Same visual appearance as MW 1.4
+                       $updated = true;
                }
 
                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}'" ;
+                       $sql3 = "SELECT COUNT(*) AS n FROM $watchlist WHERE wl_title='" .wfStrencode($obj->page_title). "' AND wl_namespace='{$obj->page_namespace}'" ;
                        $res3 = $dbr->query( $sql3, DB_READ, $fname );
                        $x = $dbr->fetchObject( $res3 );
                        $rc->numberofWatchingusers = $x->n;
@@ -234,7 +287,7 @@ function wfSpecialWatchlist() {
                        $rc->numberofWatchingusers = 0;
                }
 
-               $s .= $list->recentChangesLine( $rc, true);
+               $s .= $list->recentChangesLine( $rc, $updated );
        }
        $s .= $list->endRecentChangesList();
 
@@ -254,7 +307,7 @@ function wlHoursLink( $h, $page ) {
        $s = $sk->makeKnownLink(
          $wgContLang->specialPage( $page ),
          $wgLang->formatNum( $h ),
-         "days=" . ($h / 24.0) );
+         'days=' . ($h / 24.0) );
        return $s;
 }
 
@@ -264,15 +317,15 @@ function wlDaysLink( $d, $page ) {
        $sk = $wgUser->getSkin();
        $s = $sk->makeKnownLink(
          $wgContLang->specialPage( $page ),
-         ($d ? $wgLang->formatNum( $d ) : wfMsg( "all" ) ), "days=$d" );
+         ($d ? $wgLang->formatNum( $d ) : wfMsg( 'watchlistall2' ) ), "days=$d" );
        return $s;
 }
 
-function wlCutoffLinks( $days, $page = "Watchlist" )
+function wlCutoffLinks( $days, $page = 'Watchlist' )
 {
        $hours = array( 1, 2, 6, 12 );
        $days = array( 1, 3, 7 );
-       $cl = "";
+       $cl = '';
        $i = 0;
        foreach( $hours as $h ) {
                $hours[$i++] = wlHoursLink( $h, $page );
@@ -281,9 +334,9 @@ function wlCutoffLinks( $days, $page = "Watchlist" )
        foreach( $days as $d ) {
                $days[$i++] = wlDaysLink( $d, $page );
        }
-       return wfMsg ("wlshowlast",
-               implode(" | ", $hours),
-               implode(" | ", $days),
+       return wfMsg ('wlshowlast',
+               implode(' | ', $hours),
+               implode(' | ', $days),
                wlDaysLink( 0, $page ) );
 }