* Fix namespace selection after submit of Special:Newpages
[lhc/web/wiklou.git] / includes / SpecialNewpages.php
index cdf1b10..220e77f 100644 (file)
@@ -1,20 +1,21 @@
 <?php
 /**
  *
- * @package MediaWiki
- * @subpackage SpecialPage
+ * @addtogroup SpecialPage
  */
 
 /**
- *
- * @package MediaWiki
- * @subpackage SpecialPage
+ * implements Special:Newpages
+ * @addtogroup SpecialPage
  */
 class NewPagesPage extends QueryPage {
+
        var $namespace;
+       var $username = '';
 
-       function NewPagesPage( $namespace = NS_MAIN ) {
+       function NewPagesPage( $namespace = NS_MAIN, $username = '' ) {
                $this->namespace = $namespace;
+               $this->username = $username;
        }
 
        function getName() {
@@ -26,11 +27,29 @@ class NewPagesPage extends QueryPage {
                return false;
        }
 
+       function makeUserWhere( &$dbo ) {
+               $title = Title::makeTitleSafe( NS_USER, $this->username );
+               if( $title ) {
+                       return ' AND rc_user_text = ' . $dbo->addQuotes( $title->getText() );
+               } else {
+                       return '';
+               }
+       }
+
+       private function makeNamespaceWhere() {
+               return $this->namespace !== 'all'
+                       ? ' AND rc_namespace = ' . intval( $this->namespace )
+                       : '';
+       }
+
        function getSQL() {
                global $wgUser, $wgUseRCPatrol;
                $usepatrol = ( $wgUseRCPatrol && $wgUser->isAllowed( 'patrol' ) ) ? 1 : 0;
-               $dbr =& wfGetDB( DB_SLAVE );
-               extract( $dbr->tableNames( 'recentchanges', 'page', 'text' ) );
+               $dbr = wfGetDB( DB_SLAVE );
+               list( $recentchanges, $page ) = $dbr->tableNamesN( 'recentchanges', 'page' );
+
+               $nsfilter = $this->makeNamespaceWhere();
+               $uwhere = $this->makeUserWhere( $dbr );
 
                # FIXME: text will break with compression
                return
@@ -38,9 +57,9 @@ class NewPagesPage extends QueryPage {
                                rc_namespace AS namespace,
                                rc_title AS title,
                                rc_cur_id AS cur_id,
-                               rc_user AS user,
+                               rc_user AS \"user\",
                                rc_user_text AS user_text,
-                               rc_comment as comment,
+                               rc_comment as \"comment\",
                                rc_timestamp AS timestamp,
                                rc_timestamp AS value,
                                '{$usepatrol}' as usepatrol,
@@ -50,7 +69,9 @@ class NewPagesPage extends QueryPage {
                                page_latest as rev_id
                        FROM $recentchanges,$page
                        WHERE rc_cur_id=page_id AND rc_new=1
-                       AND rc_namespace=" . $this->namespace . " AND page_is_redirect=0";
+                       {$nsfilter}
+                       AND page_is_redirect = 0
+                       {$uwhere}";
        }
        
        function preprocessResults( &$dbo, &$res ) {
@@ -66,32 +87,37 @@ class NewPagesPage extends QueryPage {
                        $dbo->dataSeek( $res, 0 );
        }
 
+       /**
+        * 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
+        */
        function formatResult( $skin, $result ) {
-               global $wgLang, $wgContLang, $wgUser, $wgUseRCPatrol;
-               $u = $result->user;
-               $ut = $result->user_text;
-               $dirmark = $wgContLang->getDirMark(); // To keep text in correct order
-
-               $length = wfMsgExt( 'nbytes', array('parsemag', 'escape'),
-                       $wgLang->formatNum( $result->length ) );
-               $d = $wgLang->timeanddate( $result->timestamp, true );
-
-               # Since there is no diff link, we need to give users a way to
-               # mark the article as patrolled if it isn't already
-               $ns = $wgContLang->getNsText( $result->namespace );
-               if( $wgUseRCPatrol && !is_null( $result->usepatrol ) && $result->usepatrol && $result->patrolled == 0 && $wgUser->isAllowed( 'patrol' ) ) {
-                       $link = $skin->makeKnownLink( $ns . ':' . $result->title, '', "rcid={$result->rcid}" );
-               } else {
-                       $link = $skin->makeKnownLink( $ns . ':' . $result->title, '' );
-               }
-
-               $userLink = $skin->userLink( $u, $ut );
-               $userTools = $skin->userToolLinks( $u, $ut );
+               global $wgLang, $wgContLang;
+               $dm = $wgContLang->getDirMark();
+
+               $title = Title::makeTitleSafe( $result->namespace, $result->title );
+               $time = $wgLang->timeAndDate( $result->timestamp, true );
+               $plink = $skin->makeKnownLinkObj( $title, '', $this->patrollable( $result ) ? 'rcid=' . $result->rcid : '' );
+               $hist = $skin->makeKnownLinkObj( $title, wfMsgHtml( 'hist' ), 'action=history' );
+               $length = wfMsgExt( 'nbytes', array( 'parsemag', 'escape' ), $wgLang->formatNum( htmlspecialchars( $result->length ) ) );
+               $ulink = $skin->userLink( $result->user, $result->user_text ) . ' ' . $skin->userToolLinks( $result->user, $result->user_text );
+               $comment = $skin->commentBlock( $result->comment );
+
+               return "{$time} {$dm}{$plink} ({$hist}) {$dm}[{$length}] {$dm}{$ulink} {$comment}";
+       }
 
-               $s = "{$d} {$dirmark}{$link} {$dirmark}({$length}) . . " .
-                       "{$dirmark}{$userLink}{$dirmark}{$userTools}";
-               $s .= $dirmark . $skin->commentBlock( $result->comment );
-               return $s;
+       /**
+        * Should a specific result row provide "patrollable" links?
+        *
+        * @param $result Result row
+        * @return bool
+        */
+       function patrollable( $result ) {
+               global $wgUser, $wgUseRCPatrol;
+               return $wgUseRCPatrol && $wgUser->isAllowed( 'patrol' ) && !$result->patrolled;
        }
 
        function feedItemDesc( $row ) {
@@ -107,34 +133,44 @@ class NewPagesPage extends QueryPage {
        }
        
        /**
-        * Show a namespace selection form for filtering
+        * Show a form for filtering namespace and username
         *
         * @return string
         */     
        function getPageHeader() {
-               $thisTitle = Title::makeTitle( NS_SPECIAL, $this->getName() );
-               $form  = wfOpenElement( 'form', array(
-                       'method' => 'post',
-                       'action' => $thisTitle->getLocalUrl() ) );
-               $form .= wfElement( 'label', array( 'for' => 'namespace' ),
-                       wfMsg( 'namespace' ) ) . ' ';
-               $form .= HtmlNamespaceSelector( $this->namespace );
-               # Preserve the offset and limit
-               $form .= wfElement( 'input', array(
-                       'type' => 'hidden',
-                       'name' => 'offset',
-                       'value' => $this->offset ) );
-               $form .= wfElement( 'input', array(
-                       'type' => 'hidden',
-                       'name' => 'limit',
-                       'value' => $this->limit ) );
-               $form .= wfElement( 'input', array(
-                       'type' => 'submit',
-                       'name' => 'submit',
-                       'id' => 'submit',
-                       'value' => wfMsg( 'allpagessubmit' ) ) );
-               $form .= wfCloseElement( 'form' );
-               return( $form );
+               global $wgScript, $wgContLang;
+               $align = $wgContLang->isRTL() ? 'left' : 'right';
+               $self = SpecialPage::getTitleFor( $this->getName() );
+               $form = Xml::openElement( 'form', array( 'method' => 'get', 'action' => $wgScript ) ) .
+                       Xml::hidden( 'title', $self->getPrefixedDBkey() ) .
+                       Xml::openElement( 'table' ) .
+                       "<tr>
+                               <td align=\"$align\">" .
+                                       Xml::label( wfMsg( 'namespace' ), 'namespace' ) .
+                               "</td>
+                               <td>" .
+                                       Xml::namespaceSelector( intval( $this->namespace ), 'all' ) .
+                               "</td>
+                       </tr>
+                       <tr>
+                               <td align=\"$align\">" .
+                                       Xml::label( wfMsg( 'newpages-username' ), 'mw-np-username' ) .
+                               "</td>
+                               <td>" .
+                                       Xml::input( 'username', 30, $this->username, array( 'id' => 'mw-np-username' ) ) .
+                               "</td>
+                       </tr>
+                       <tr>
+                               <td></td>
+                               <td>" .
+                                       Xml::submitButton( wfMsg( 'allpagessubmit' ) ) .
+                               "</td>
+                       </tr>" .
+                       Xml::closeElement( 'table' ) .
+                       Xml::hidden( 'offset', $this->offset ) .
+                       Xml::hidden( 'limit', $this->limit ) .
+                       Xml::closeElement( 'form' );
+               return $form;
        }
        
        /**
@@ -143,7 +179,7 @@ class NewPagesPage extends QueryPage {
         * @return array
         */
        function linkParameters() {
-               return( array( 'namespace' => $this->namespace ) );
+               return( array( 'namespace' => $this->namespace, 'username' => $this->username ) );
        }
        
 }
@@ -156,6 +192,7 @@ function wfSpecialNewpages($par, $specialPage) {
 
        list( $limit, $offset ) = wfCheckLimits();
        $namespace = NS_MAIN;
+       $username = '';
 
        if ( $par ) {
                $bits = preg_split( '/\s*,\s*/', trim( $par ) );
@@ -165,6 +202,7 @@ function wfSpecialNewpages($par, $specialPage) {
                        if ( is_numeric( $bit ) )
                                $limit = $bit;
 
+                       $m = array();
                        if ( preg_match( '/^limit=(\d+)$/', $bit, $m ) )
                                $limit = intval($m[1]);
                        if ( preg_match( '/^offset=(\d+)$/', $bit, $m ) )
@@ -177,17 +215,17 @@ function wfSpecialNewpages($par, $specialPage) {
                        }
                }
        } else {
-               if( $ns = $wgRequest->getInt( 'namespace', 0 ) )
+               if( $ns = $wgRequest->getText( 'namespace', NS_MAIN ) )
                        $namespace = $ns;
+               if( $un = $wgRequest->getText( 'username' ) )
+                       $username = $un;
        }
        
        if ( ! isset( $shownavigation ) )
                $shownavigation = ! $specialPage->including();
 
-       $npp = new NewPagesPage( $namespace );
+       $npp = new NewPagesPage( $namespace, $username );
 
        if ( ! $npp->doFeed( $wgRequest->getVal( 'feed' ), $limit ) )
                $npp->doQuery( $offset, $limit, $shownavigation );
 }
-
-?>