X-Git-Url: https://git.cyclocoop.org/?a=blobdiff_plain;f=includes%2FSpecialWatchlist.php;h=4ab7c215d6da37301244a2597b4a08dd4d448915;hb=f10d65f0a3b716b7cc90f4aacec388e21f8ef644;hp=a84ad5e0a43d52d74db013d734a57d9c713de634;hpb=90155b8a9719ada6522a320a3315b4e698fdd970;p=lhc%2Fweb%2Fwiklou.git diff --git a/includes/SpecialWatchlist.php b/includes/SpecialWatchlist.php index a84ad5e0a4..4ab7c215d6 100644 --- a/includes/SpecialWatchlist.php +++ b/includes/SpecialWatchlist.php @@ -1,134 +1,197 @@ setPagetitle( wfMsg( "watchlist" ) ); - $sub = wfMsg( "watchlistsub", $wgUser->getName() ); + $wgOut->setPagetitle( wfMsg( 'watchlist' ) ); + $sub = htmlspecialchars( 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; } + $defaults = array( + /* float */ 'days' => 3.0, /* or 0.5, watch further below */ + /* bool */ 'hideOwn' => false, + /* bool */ 'hideBots' => false, + ); + + extract($defaults); + # Get query variables $days = $wgRequest->getVal( 'days' ); + $hideOwn = $wgRequest->getBool( 'hideOwn' ); + $hideBots = $wgRequest->getBool( 'hideBots' ); + + # Watchlist editing $action = $wgRequest->getVal( 'action' ); $remove = $wgRequest->getVal( 'remove' ); - $id = $wgRequest->getVal( 'id' ); + $id = $wgRequest->getArray( 'id' ); + + $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" ) ); + # Deleting items from watchlist + if(($action == 'submit') && isset($remove) && is_array($id)) { + $wgOut->addWikiText( wfMsg( 'removingchecked' ) ); + $wgOut->addHTML( '
' );
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( "
\n" . wfMsg( "couldntremove", htmlspecialchars($one) ) );
+ $wgOut->addHTML( "
\n" . wfMsg( 'couldntremove', htmlspecialchars($one) ) );
} else {
- $wgOut->addHTML( " (" . htmlspecialchars($one) . ")" );
+ $wgOut->addHTML( ' (' . htmlspecialchars($one) . ')' );
}
} else {
- $wgOut->addHTML( "
\n" . wfMsg( "iteminvalidname", htmlspecialchars($one) ) );
+ $wgOut->addHTML( "
\n" . wfMsg( 'iteminvalidname', htmlspecialchars($one) ) );
}
}
- $wgOut->addHTML( "done.\n
" ); + $wgOut->addHTML( "done.
\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; } } $dbr =& wfGetDB( DB_SLAVE ); - extract( $dbr->tableNames( 'cur', 'watchlist', 'recentchanges' ) ); + extract( $dbr->tableNames( 'page', 'revision', 'watchlist', 'recentchanges' ) ); + + $sql = "SELECT COUNT(*) AS n FROM $watchlist WHERE wl_user=$uid"; + $res = $dbr->query( $sql, $fname ); + $s = $dbr->fetchObject( $res ); + + # Patch *** A1 *** (see A2 below) +# adjust for page X, talk:page X, which are both stored separately, but treated together + $nitems = floor($s->n / 2); +# $nitems = $s->n; - $sql = "SELECT COUNT(*) AS n FROM $watchlist WHERE wl_user=$uid"; - $res = $dbr->query( $sql ); - $s = $dbr->fetchObject( $res ); - $nitems = $s->n; if($nitems == 0) { - $wgOut->addHTML( wfMsg( "nowatchlist" ) ); - return; + $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', $hideOwn, $defaults, $nondefaults); + wfAppendToArrayIfNotDefault( 'hideBots', $hideBots, $defaults, $nondefaults); + if ( $days <= 0 ) { $docutoff = ''; $cutoff = false; - $npages = wfMsg( "all" ); + $npages = wfMsg( 'watchlistall1' ); } else { - $docutoff = "AND cur_timestamp > '" . + $docutoff = "AND rev_timestamp > '" . ( $cutoff = $dbr->timestamp( time() - intval( $days * 86400 ) ) ) . "'"; - $sql = "SELECT COUNT(*) AS n FROM $cur WHERE cur_timestamp>'$cutoff'"; - $res = $dbr->query( $sql ); - $s = $dbr->fetchObject( $res ); - $npages = $s->n; + /* + $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; + */ + $npages = 40000 * $days; } - if(isset($_REQUEST['magic'])) { - $wgOut->addHTML( wfMsg( "watchlistcontains", $wgLang->formatNum( $nitems ) ) . - "" . wfMsg( "watcheditlist" ) . "
\n" ); + /* Edit watchlist form */ + if($wgRequest->getBool('edit') || $par == 'edit' ) { + $wgOut->addWikiText( wfMsg( 'watchlistcontains', $wgLang->formatNum( $nitems ) ) . + "\n\n" . wfMsg( 'watcheditlist' ) ); + + $wgOut->addHTML( '\n" ); + $wgOut->addHTML( + "\n" . + "\n" + ); return; } @@ -140,60 +203,124 @@ function wfSpecialWatchlist() { # 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 = "cur_timestamp"; - $y = wfMsg( "watchmethod-recent" ); - $z = "wl_namespace=cur_namespace&65534"; + + $andHideOwn = $hideOwn ? "AND (rc_user <> $uid)" : ''; + if( $wgFilterRobotsWL ) { + $andHideBotsOptional = $hideBots ? "AND (rc_bot = 0)" : ''; } else { - $x = "name_title_timestamp"; - $y = wfMsg( "watchmethod-list" ); - $z = "(wl_namespace=cur_namespace OR wl_namespace+1=cur_namespace)"; + $andHideBotsOptional = "AND rc_this_oldid=page_latest"; + } + + + # Show watchlist header + $header = ''; + if( $wgUser->getOption( 'enotifwatchlistpages' ) && $wgEnotifWatchlist) { + $header .= wfMsg( 'wlheader-enotif' ) . "\n"; + } + if ( $wgEnotifWatchlist && $wgShowUpdatedMarker ) { + $header .= wfMsg( 'wlheader-showupdated' ) . "\n"; } + # TODO: Consider removing the third parameter + $header .= wfMsg( 'watchdetails', $wgLang->formatNum( $nitems ), + $wgLang->formatNum( $npages ), '', + $specialTitle->getFullUrl( 'edit=yes' ) ); + $wgOut->addWikiText( $header ); + + if ( $wgEnotifWatchlist && $wgShowUpdatedMarker ) { + $wgOut->addHTML( '' . + "\n\n" ); + } - $wgOut->addHTML( "" . wfMsg( "watchdetails", - $wgLang->formatNum( $nitems ), $wgLang->formatNum( $npages ), $y, - $specialTitle->escapeLocalUrl( "magic=yes" ) ) . "" );
+
if($days >= 1)
- $note = wfMsg( "rcnote", $wgLang->formatNum( $numRows ), $wgLang->formatNum( $days ) );
+ $wgOut->addWikiText( wfMsg( 'rcnote', $wgLang->formatNum( $numRows ),
+ $wgLang->formatNum( $days ) ) . '
' , false );
elseif($days > 0)
- $note = wfMsg( "wlnote", $wgLang->formatNum( $numRows ), $wgLang->formatNum( round($days*24) ) );
- else
- $note = "";
- $wgOut->addHTML( "\n
" . wfMsg( "watchnochange" ) . "
" ); + $wgOut->addWikitext( "