X-Git-Url: http://git.heureux-cyclage.org/?a=blobdiff_plain;f=includes%2Fspecials%2FSpecialNewpages.php;h=ecec87de5375ec83c74c8d47d8a228ed37eca261;hb=e7e268afa9750c0089309e6f9291f1b3ab70c7f8;hp=886c41a25488c181b925c88691bc80eab2c00acb;hpb=d24c7e193c358f0d7ca310404c7a7c90acd363ac;p=lhc%2Fweb%2Fwiklou.git diff --git a/includes/specials/SpecialNewpages.php b/includes/specials/SpecialNewpages.php index 886c41a254..ecec87de53 100644 --- a/includes/specials/SpecialNewpages.php +++ b/includes/specials/SpecialNewpages.php @@ -1,41 +1,73 @@ includable( true ); } protected function setup( $par ) { - global $wgRequest, $wgUser, $wgEnableNewpagesUserFilter; + global $wgEnableNewpagesUserFilter; // Options $opts = new FormOptions(); $this->opts = $opts; // bind $opts->add( 'hideliu', false ); - $opts->add( 'hidepatrolled', $wgUser->getBoolOption( 'newpageshidepatrolled' ) ); + $opts->add( 'hidepatrolled', $this->getUser()->getBoolOption( 'newpageshidepatrolled' ) ); $opts->add( 'hidebots', false ); $opts->add( 'hideredirs', true ); - $opts->add( 'limit', (int)$wgUser->getOption( 'rclimit' ) ); + $opts->add( 'limit', (int)$this->getUser()->getOption( 'rclimit' ) ); $opts->add( 'offset', '' ); $opts->add( 'namespace', '0' ); $opts->add( 'username', '' ); $opts->add( 'feed', '' ); $opts->add( 'tagfilter', '' ); + $this->customFilters = array(); + wfRunHooks( 'SpecialNewPagesFilters', array( $this, &$this->customFilters ) ); + foreach( $this->customFilters as $key => $params ) { + $opts->add( $key, $params['default'] ); + } + // Set values - $opts->fetchValuesFromRequest( $wgRequest ); + $opts->fetchValuesFromRequest( $this->getRequest() ); if ( $par ) $this->parseParams( $par ); // Validate @@ -43,38 +75,43 @@ class SpecialNewpages extends SpecialPage { if( !$wgEnableNewpagesUserFilter ) { $opts->setValue( 'username', '' ); } - - // Store some objects - $this->skin = $wgUser->getSkin(); } protected function parseParams( $par ) { - global $wgLang; $bits = preg_split( '/\s*,\s*/', trim( $par ) ); foreach ( $bits as $bit ) { - if ( 'shownav' == $bit ) + if ( 'shownav' == $bit ) { $this->showNavigation = true; - if ( 'hideliu' === $bit ) + } + if ( 'hideliu' === $bit ) { $this->opts->setValue( 'hideliu', true ); - if ( 'hidepatrolled' == $bit ) + } + if ( 'hidepatrolled' == $bit ) { $this->opts->setValue( 'hidepatrolled', true ); - if ( 'hidebots' == $bit ) + } + if ( 'hidebots' == $bit ) { $this->opts->setValue( 'hidebots', true ); - if ( 'showredirs' == $bit ) + } + if ( 'showredirs' == $bit ) { $this->opts->setValue( 'hideredirs', false ); - if ( is_numeric( $bit ) ) + } + if ( is_numeric( $bit ) ) { $this->opts->setValue( 'limit', intval( $bit ) ); + } $m = array(); - if ( preg_match( '/^limit=(\d+)$/', $bit, $m ) ) - $this->opts->setValue( 'limit', intval($m[1]) ); + if ( preg_match( '/^limit=(\d+)$/', $bit, $m ) ) { + $this->opts->setValue( 'limit', intval( $m[1] ) ); + } // PG offsets not just digits! - if ( preg_match( '/^offset=([^=]+)$/', $bit, $m ) ) - $this->opts->setValue( 'offset', intval($m[1]) ); - if ( preg_match( '/^username=(.*)$/', $bit, $m ) ) + if ( preg_match( '/^offset=([^=]+)$/', $bit, $m ) ) { + $this->opts->setValue( 'offset', intval( $m[1] ) ); + } + if ( preg_match( '/^username=(.*)$/', $bit, $m ) ) { $this->opts->setValue( 'username', $m[1] ); + } if ( preg_match( '/^namespace=(.*)$/', $bit, $m ) ) { - $ns = $wgLang->getNsIndex( $m[1] ); + $ns = $this->getLanguage()->getNsIndex( $m[1] ); if( $ns !== false ) { $this->opts->setValue( 'namespace', $ns ); } @@ -85,11 +122,11 @@ class SpecialNewpages extends SpecialPage { /** * Show a form for filtering namespace and username * - * @param string $par - * @return string + * @param $par String + * @return String */ public function execute( $par ) { - global $wgLang, $wgUser, $wgOut; + $out = $this->getOutput(); $this->setHeaders(); $this->outputHeader(); @@ -101,11 +138,12 @@ class SpecialNewpages extends SpecialPage { // Settings $this->form(); - $this->setSyndicated(); $feedType = $this->opts->getValue( 'feed' ); if( $feedType ) { return $this->feed( $feedType ); } + + $out->setFeedAppendQuery( wfArrayToCGI( $this->opts->getAllValues() ) ); } $pager = new NewPagesPager( $this, $this->opts ); @@ -114,15 +152,17 @@ class SpecialNewpages extends SpecialPage { if( $pager->getNumRows() ) { $navigation = ''; - if ( $this->showNavigation ) $navigation = $pager->getNavigationBar(); - $wgOut->addHTML( $navigation . $pager->getBody() . $navigation ); + if ( $this->showNavigation ) { + $navigation = $pager->getNavigationBar(); + } + $out->addHTML( $navigation . $pager->getBody() . $navigation ); } else { - $wgOut->addWikiMsg( 'specialpage-empty' ); + $out->addWikiMsg( 'specialpage-empty' ); } } protected function filterLinks() { - global $wgGroupPermissions, $wgUser, $wgLang; + global $wgGroupPermissions; // show/hide links $showhide = array( wfMsgHtml( 'show' ), wfMsgHtml( 'hide' ) ); @@ -134,37 +174,43 @@ class SpecialNewpages extends SpecialPage { 'hidebots' => 'rcshowhidebots', 'hideredirs' => 'whatlinkshere-hideredirs' ); + foreach ( $this->customFilters as $key => $params ) { + $filters[$key] = $params['msg']; + } // Disable some if needed - if ( $wgGroupPermissions['*']['createpage'] !== true ) - unset($filters['hideliu']); - - if ( !$wgUser->useNPPatrol() ) - unset($filters['hidepatrolled']); + # @todo FIXME: Throws E_NOTICEs if not set; and doesn't obey hooks etc. + if ( $wgGroupPermissions['*']['createpage'] !== true ) { + unset( $filters['hideliu'] ); + } + if ( !$this->getUser()->useNPPatrol() ) { + unset( $filters['hidepatrolled'] ); + } $links = array(); $changed = $this->opts->getChangedValues(); - unset($changed['offset']); // Reset offset if query type changes + unset( $changed['offset'] ); // Reset offset if query type changes $self = $this->getTitle(); foreach ( $filters as $key => $msg ) { - $onoff = 1 - $this->opts->getValue($key); - $link = $this->skin->link( $self, $showhide[$onoff], array(), - array( $key => $onoff ) + $changed + $onoff = 1 - $this->opts->getValue( $key ); + $link = Linker::link( $self, $showhide[$onoff], array(), + array( $key => $onoff ) + $changed ); $links[$key] = wfMsgHtml( $msg, $link ); } - return $wgLang->pipeList( $links ); + return $this->getLanguage()->pipeList( $links ); } protected function form() { - global $wgOut, $wgEnableNewpagesUserFilter, $wgScript; + global $wgEnableNewpagesUserFilter, $wgScript; // Consume values $this->opts->consumeValue( 'offset' ); // don't carry offset, DWIW $namespace = $this->opts->consumeValue( 'namespace' ); $username = $this->opts->consumeValue( 'username' ); + $tagFilterVal = $this->opts->consumeValue( 'tagfilter' ); // Check username input validity $ut = Title::makeTitleSafe( NS_USER, $username ); @@ -173,143 +219,179 @@ class SpecialNewpages extends SpecialPage { // Store query values in hidden fields so that form submission doesn't lose them $hidden = array(); foreach ( $this->opts->getUnconsumedValues() as $key => $value ) { - $hidden[] = Xml::hidden( $key, $value ); + $hidden[] = Html::hidden( $key, $value ); } $hidden = implode( "\n", $hidden ); - $tagFilter = ChangeTags::buildTagFilterSelector( $this->opts['tagfilter'] ); - if ($tagFilter) + $tagFilter = ChangeTags::buildTagFilterSelector( $tagFilterVal ); + if ( $tagFilter ) { list( $tagFilterLabel, $tagFilterSelector ) = $tagFilter; + } $form = Xml::openElement( 'form', array( 'action' => $wgScript ) ) . - Xml::hidden( 'title', $this->getTitle()->getPrefixedDBkey() ) . + Html::hidden( 'title', $this->getTitle()->getPrefixedDBkey() ) . Xml::fieldset( wfMsg( 'newpages' ) ) . Xml::openElement( 'table', array( 'id' => 'mw-newpages-table' ) ) . - " - " . + ' + ' . Xml::label( wfMsg( 'namespace' ), 'namespace' ) . - " - " . + ' + ' . Xml::namespaceSelector( $namespace, 'all' ) . - " - " . ( $tagFilter ? ( - " - " . + ' + ' . ( $tagFilter ? ( + ' + ' . $tagFilterLabel . - " - " . + ' + ' . $tagFilterSelector . - " - " ) : '' ) . - ($wgEnableNewpagesUserFilter ? - " - " . + ' + ' ) : '' ) . + ( $wgEnableNewpagesUserFilter ? + ' + ' . Xml::label( wfMsg( 'newpages-username' ), 'mw-np-username' ) . - " - " . + ' + ' . Xml::input( 'username', 30, $userText, array( 'id' => 'mw-np-username' ) ) . - " - " : "" ) . - " - " . + ' + ' : '' ) . + ' + ' . Xml::submitButton( wfMsg( 'allpagessubmit' ) ) . - " - " . - " + ' + ' . + ' - " . + ' . $this->filterLinks() . - " - " . + ' + ' . Xml::closeElement( 'table' ) . Xml::closeElement( 'fieldset' ) . $hidden . Xml::closeElement( 'form' ); - $wgOut->addHTML( $form ); - } - - protected function setSyndicated() { - global $wgOut; - $queryParams = array( - 'namespace' => $this->opts->getValue( 'namespace' ), - 'username' => $this->opts->getValue( 'username' ) - ); - $wgOut->setSyndicated( true ); - $wgOut->setFeedAppendQuery( wfArrayToCGI( $queryParams ) ); + $this->getOutput()->addHTML( $form ); } /** * Format a row, providing the timestamp, links to the page/history, size, user links, and a comment * - * @param $skin Skin to use * @param $result Result row - * @return string + * @return String */ public function formatRow( $result ) { - global $wgLang, $wgContLang, $wgUser; + # Revision deletion works on revisions, so we should cast one + $row = array( + 'comment' => $result->rc_comment, + 'deleted' => $result->rc_deleted, + 'user_text' => $result->rc_user_text, + 'user' => $result->rc_user, + ); + $rev = new Revision( $row ); $classes = array(); - - $dm = $wgContLang->getDirMark(); - - $title = Title::makeTitleSafe( $result->rc_namespace, $result->rc_title ); - $time = $wgLang->timeAndDate( $result->rc_timestamp, true ); - $query = $this->patrollable( $result ) ? "rcid={$result->rc_id}&redirect=no" : 'redirect=no'; - $plink = $this->skin->makeKnownLinkObj( $title, '', $query ); - $hist = $this->skin->makeKnownLinkObj( $title, wfMsgHtml( 'hist' ), 'action=history' ); - $length = wfMsgExt( 'nbytes', array( 'parsemag', 'escape' ), - $wgLang->formatNum( $result->length ) ); - $ulink = $this->skin->userLink( $result->rc_user, $result->rc_user_text ) . ' ' . - $this->skin->userToolLinks( $result->rc_user, $result->rc_user_text ); - $comment = $this->skin->commentBlock( $result->rc_comment ); - - if ( $this->patrollable( $result ) ) + + $lang = $this->getLanguage(); + $dm = $lang->getDirMark(); + + $title = Title::newFromRow( $result ); + $spanTime = Html::element( 'span', array( 'class' => 'mw-newpages-time' ), + $lang->timeanddate( $result->rc_timestamp, true ) + ); + $time = Linker::linkKnown( + $title, + $spanTime, + array(), + array( 'oldid' => $result->rc_this_oldid ), + array() + ); + + $query = array( 'redirect' => 'no' ); + + if( $this->patrollable( $result ) ) { + $query['rcid'] = $result->rc_id; + } + + $plink = Linker::linkKnown( + $title, + null, + array( 'class' => 'mw-newpages-pagename' ), + $query, + array( 'known' ) // Set explicitly to avoid the default of 'known','noclasses'. This breaks the colouration for stubs + ); + $histLink = Linker::linkKnown( + $title, + wfMsgHtml( 'hist' ), + array(), + array( 'action' => 'history' ) + ); + $hist = Html::rawElement( 'span', array( 'class' => 'mw-newpages-history' ), wfMsg( 'parentheses', $histLink ) ); + + $length = Html::element( 'span', array( 'class' => 'mw-newpages-length' ), + '[' . $this->msg( 'nbytes' )->numParams( $result->length )->text() . ']' + ); + + $ulink = Linker::revUserTools( $rev ); + $comment = Linker::revComment( $rev ); + + if ( $this->patrollable( $result ) ) { $classes[] = 'not-patrolled'; + } - # Tags, if any. - list( $tagDisplay, $newClasses ) = ChangeTags::formatSummaryRow( $result->ts_tags, 'newpages' ); - $classes = array_merge( $classes, $newClasses ); + # Add a class for zero byte pages + if ( $result->length == 0 ) { + $classes[] = 'mw-newpages-zero-byte-page'; + } - $css = count($classes) ? ' class="'.implode( " ", $classes).'"' : ''; + # Tags, if any. check for including due to bug 23293 + if ( !$this->including() ) { + list( $tagDisplay, $newClasses ) = ChangeTags::formatSummaryRow( $result->ts_tags, 'newpages' ); + $classes = array_merge( $classes, $newClasses ); + } else { + $tagDisplay = ''; + } + + $css = count( $classes ) ? ' class="' . implode( ' ', $classes ) . '"' : ''; - return "{$time} {$dm}{$plink} ({$hist}) {$dm}[{$length}] {$dm}{$ulink} {$comment} {$tagDisplay}\n"; + return "{$time} {$dm}{$plink} {$hist} {$dm}{$length} {$dm}{$ulink} {$comment} {$tagDisplay}\n"; } /** * Should a specific result row provide "patrollable" links? * * @param $result Result row - * @return bool + * @return Boolean */ protected function patrollable( $result ) { - global $wgUser; - return ( $wgUser->useNPPatrol() && !$result->rc_patrolled ); + return ( $this->getUser()->useNPPatrol() && !$result->rc_patrolled ); } /** * Output a subscription feed listing recent edits to this page. - * @param string $type + * + * @param $type String */ protected function feed( $type ) { global $wgFeed, $wgFeedClasses, $wgFeedLimit; if ( !$wgFeed ) { - global $wgOut; - $wgOut->addWikiMsg( 'feed-unavailable' ); + $this->getOutput()->addWikiMsg( 'feed-unavailable' ); return; } if( !isset( $wgFeedClasses[$type] ) ) { - global $wgOut; - $wgOut->addWikiMsg( 'feed-invalid' ); + $this->getOutput()->addWikiMsg( 'feed-invalid' ); return; } $feed = new $wgFeedClasses[$type]( $this->feedTitle(), wfMsgExt( 'tagline', 'parsemag' ), - $this->getTitle()->getFullUrl() ); + $this->getTitle()->getFullUrl() + ); $pager = new NewPagesPager( $this, $this->opts ); $limit = $this->opts->getValue( 'limit' ); @@ -317,7 +399,7 @@ class SpecialNewpages extends SpecialPage { $feed->outHeader(); if( $pager->getNumRows() > 0 ) { - while( $row = $pager->mResult->fetchObject() ) { + foreach ( $pager->mResult as $row ) { $feed->outItem( $this->feedItem( $row ) ); } } @@ -325,10 +407,9 @@ class SpecialNewpages extends SpecialPage { } protected function feedTitle() { - global $wgContLanguageCode, $wgSitename; - $page = SpecialPage::getPage( 'Newpages' ); - $desc = $page->getDescription(); - return "$wgSitename - $desc [$wgContLanguageCode]"; + global $wgLanguageCode, $wgSitename; + $desc = $this->getDescription(); + return "$wgSitename - $desc [$wgLanguageCode]"; } protected function feedItem( $row ) { @@ -343,9 +424,10 @@ class SpecialNewpages extends SpecialPage { $title->getFullURL(), $date, $this->feedItemAuthor( $row ), - $comments); + $comments + ); } else { - return NULL; + return null; } } @@ -357,7 +439,7 @@ class SpecialNewpages extends SpecialPage { $revision = Revision::newFromId( $row->rev_id ); if( $revision ) { return '

' . htmlspecialchars( $revision->getUserText() ) . wfMsgForContent( 'colon-separator' ) . - htmlspecialchars( FeedItem::stripComment( $revision->getComment() ) ) . + htmlspecialchars( FeedItem::stripComment( $revision->getComment() ) ) . "

\n
\n
" . nl2br( htmlspecialchars( $revision->getText() ) ) . "
"; } @@ -370,23 +452,21 @@ class SpecialNewpages extends SpecialPage { */ class NewPagesPager extends ReverseChronologicalPager { // Stored opts - protected $opts, $mForm; + protected $opts; + + /** + * @var HtmlForm + */ + protected $mForm; function __construct( $form, FormOptions $opts ) { - parent::__construct(); + parent::__construct( $form->getContext() ); $this->mForm = $form; $this->opts = $opts; } - function getTitle() { - static $title = null; - if ( $title === null ) - $title = $this->mForm->getTitle(); - return $title; - } - function getQueryInfo() { - global $wgEnableNewpagesUserFilter, $wgGroupPermissions, $wgUser; + global $wgEnableNewpagesUserFilter, $wgGroupPermissions; $conds = array(); $conds['rc_new'] = 1; @@ -412,7 +492,7 @@ class NewPagesPager extends ReverseChronologicalPager { $conds['rc_user'] = 0; } # If this user cannot see patrolled edits or they are off, don't do dumb queries! - if( $this->opts->getValue( 'hidepatrolled' ) && $wgUser->useNPPatrol() ) { + if( $this->opts->getValue( 'hidepatrolled' ) && $this->getUser()->useNPPatrol() ) { $conds['rc_patrolled'] = 0; } if( $this->opts->getValue( 'hidebots' ) ) { @@ -423,27 +503,39 @@ class NewPagesPager extends ReverseChronologicalPager { $conds['page_is_redirect'] = 0; } + // Allow changes to the New Pages query + $tables = array( 'recentchanges', 'page' ); + $fields = array( + 'rc_namespace', 'rc_title', 'rc_cur_id', 'rc_user', 'rc_user_text', + 'rc_comment', 'rc_timestamp', 'rc_patrolled','rc_id', 'rc_deleted', + 'page_len AS length', 'page_latest AS rev_id', 'ts_tags', 'rc_this_oldid', + 'page_namespace', 'page_title' + ); + $join_conds = array( 'page' => array( 'INNER JOIN', 'page_id=rc_cur_id' ) ); + + wfRunHooks( 'SpecialNewpagesConditions', + array( &$this, $this->opts, &$conds, &$tables, &$fields, &$join_conds ) ); + $info = array( - 'tables' => array( 'recentchanges', 'page' ), - 'fields' => 'rc_namespace,rc_title, rc_cur_id, rc_user,rc_user_text,rc_comment, - rc_timestamp,rc_patrolled,rc_id,page_len as length, page_latest as rev_id, ts_tags', - 'conds' => $conds, - 'options' => array( 'USE INDEX' => array('recentchanges' => $rcIndexes) ), - 'join_conds' => array( - 'page' => array('INNER JOIN', 'page_id=rc_cur_id'), - ), + 'tables' => $tables, + 'fields' => $fields, + 'conds' => $conds, + 'options' => array( 'USE INDEX' => array( 'recentchanges' => $rcIndexes ) ), + 'join_conds' => $join_conds ); - ## Empty array for fields, it'll be set by us anyway. + // Empty array for fields, it'll be set by us anyway. $fields = array(); - ## Modify query for tags - ChangeTags::modifyDisplayQuery( $info['tables'], - $fields, - $info['conds'], - $info['join_conds'], - $info['options'], - $this->opts['tagfilter'] ); + // Modify query for tags + ChangeTags::modifyDisplayQuery( + $info['tables'], + $fields, + $info['conds'], + $info['join_conds'], + $info['options'], + $this->opts['tagfilter'] + ); return $info; } @@ -459,16 +551,16 @@ class NewPagesPager extends ReverseChronologicalPager { function getStartBody() { # Do a batch existence check on pages $linkBatch = new LinkBatch(); - while( $row = $this->mResult->fetchObject() ) { + foreach ( $this->mResult as $row ) { $linkBatch->add( NS_USER, $row->rc_user_text ); $linkBatch->add( NS_USER_TALK, $row->rc_user_text ); $linkBatch->add( $row->rc_namespace, $row->rc_title ); } $linkBatch->execute(); - return "'; } }