From: Aaron Schulz Date: Wed, 20 Aug 2008 17:50:13 +0000 (+0000) Subject: redo allpages again less obtrusively X-Git-Tag: 1.31.0-rc.0~45770 X-Git-Url: http://git.cyclocoop.org//%27%40script%40/%27?a=commitdiff_plain;h=17c79f9a8e72dcfc5727e03cb606fe45bd290fb6;p=lhc%2Fweb%2Fwiklou.git redo allpages again less obtrusively --- diff --git a/includes/specials/SpecialAllpages.php b/includes/specials/SpecialAllpages.php index 7223e31747..65209eef31 100644 --- a/includes/specials/SpecialAllpages.php +++ b/includes/specials/SpecialAllpages.php @@ -13,7 +13,8 @@ function wfSpecialAllpages( $par=NULL, $specialPage ) { global $wgRequest, $wgOut, $wgContLang; # GET values - $from = $wgRequest->getVal( 'from' ); + $from = $wgRequest->getVal( 'from', null ); + $to = $wgRequest->getVal( 'to', null ); $namespace = $wgRequest->getInt( 'namespace' ); $namespaces = $wgContLang->getNamespaces(); @@ -23,14 +24,14 @@ function wfSpecialAllpages( $par=NULL, $specialPage ) { $wgOut->setPagetitle( ( $namespace > 0 && in_array( $namespace, array_keys( $namespaces) ) ) ? wfMsg( 'allinnamespace', str_replace( '_', ' ', $namespaces[$namespace] ) ) : wfMsg( 'allarticles' ) - ); + ); - if ( isset($par) ) { - $indexPage->showChunk( $namespace, $par, $specialPage->including() ); - } elseif ( isset($from) ) { - $indexPage->showChunk( $namespace, $from, $specialPage->including() ); + if( isset($par) ) { + $indexPage->showChunk( $namespace, $par, $to, $specialPage->including() ); + } elseif( isset($from) && !isset($to) ) { + $indexPage->showChunk( $namespace, $from, $to, $specialPage->including() ); } else { - $indexPage->showToplevel ( $namespace, $specialPage->including() ); + $indexPage->showToplevel( $namespace, $from, $to, $specialPage->including() ); } } @@ -42,7 +43,17 @@ class SpecialAllpages { /** * Maximum number of pages to show on single subpage. */ - protected $maxPerPage = 960; + protected $maxPerPage = 345; + + /** + * Maximum number of pages to show on single index subpage. + */ + protected $maxLineCount = 200; + + /** + * Maximum number of chars to show for an entry. + */ + protected $maxPageLength = 70; /** * Name of this special page. Used to make title objects that reference back @@ -58,46 +69,55 @@ class SpecialAllpages { /** * HTML for the top form * @param integer $namespace A namespace constant (default NS_MAIN). - * @param string $from Article name we are starting listing at. + * @param string $from dbKey we are starting listing at. + * @param string $to dbKey we are ending listing at. */ -function namespaceForm ( $namespace = NS_MAIN, $from = '' ) { - global $wgScript; - $t = SpecialPage::getTitleFor( $this->name ); - - $out = Xml::openElement( 'div', array( 'class' => 'namespaceoptions' ) ); - $out .= Xml::openElement( 'form', array( 'method' => 'get', 'action' => $wgScript ) ); - $out .= Xml::hidden( 'title', $t->getPrefixedText() ); - $out .= Xml::openElement( 'fieldset' ); - $out .= Xml::element( 'legend', null, wfMsg( 'allpages' ) ); - $out .= Xml::openElement( 'table', array( 'id' => 'nsselect', 'class' => 'allpages' ) ); - $out .= " - " . - Xml::label( wfMsg( $this->nsfromMsg ), 'nsfrom' ) . - " - " . - Xml::input( 'from', 20, $from, array( 'id' => 'nsfrom' ) ) . - " - - - " . - Xml::label( wfMsg( 'namespace' ), 'namespace' ) . - " - " . - Xml::namespaceSelector( $namespace, null ) . ' ' . - Xml::submitButton( wfMsg( 'allpagessubmit' ) ) . - " - "; - $out .= Xml::closeElement( 'table' ); - $out .= Xml::closeElement( 'fieldset' ); - $out .= Xml::closeElement( 'form' ); - $out .= Xml::closeElement( 'div' ); - return $out; +function namespaceForm( $namespace = NS_MAIN, $from = '', $to = '' ) { + global $wgScript; + $t = SpecialPage::getTitleFor( $this->name ); + + $out = Xml::openElement( 'div', array( 'class' => 'namespaceoptions' ) ); + $out .= Xml::openElement( 'form', array( 'method' => 'get', 'action' => $wgScript ) ); + $out .= Xml::hidden( 'title', $t->getPrefixedText() ); + $out .= Xml::openElement( 'fieldset' ); + $out .= Xml::element( 'legend', null, wfMsg( 'allpages' ) ); + $out .= Xml::openElement( 'table', array( 'id' => 'nsselect', 'class' => 'allpages' ) ); + $out .= " + " . + Xml::label( wfMsg( 'allpagesfrom' ), 'nsfrom' ) . + " + " . + Xml::input( 'from', 30, str_replace('_',' ',$from), array( 'id' => 'nsfrom' ) ) . + " + + + " . + Xml::label( wfMsg( 'allpagesto' ), 'nsto' ) . + " + " . + Xml::input( 'to', 30, str_replace('_',' ',$to), array( 'id' => 'nsto' ) ) . + " + + + " . + Xml::label( wfMsg( 'namespace' ), 'namespace' ) . + " + " . + Xml::namespaceSelector( $namespace, null ) . ' ' . + Xml::submitButton( wfMsg( 'allpagessubmit' ) ) . + " + "; + $out .= Xml::closeElement( 'table' ); + $out .= Xml::closeElement( 'fieldset' ); + $out .= Xml::closeElement( 'form' ); + $out .= Xml::closeElement( 'div' ); + return $out; } /** * @param integer $namespace (default NS_MAIN) */ -function showToplevel ( $namespace = NS_MAIN, $including = false ) { +function showToplevel( $namespace = NS_MAIN, $from = '', $to = '', $including = false ) { global $wgOut, $wgContLang; $align = $wgContLang->isRtl() ? 'left' : 'right'; @@ -107,45 +127,54 @@ function showToplevel ( $namespace = NS_MAIN, $including = false ) { $dbr = wfGetDB( DB_SLAVE ); $out = ""; $where = array( 'page_namespace' => $namespace ); + + $from = Title::makeTitleSafe( $namespace, $from ); + $to = Title::makeTitleSafe( $namespace, $to ); + $from = ( $from && $from->isLocal() ) ? $from->getDBKey() : null; + $to = ( $to && $to->isLocal() ) ? $to->getDBKey() : null; + + if( isset($from) ) + $where[] = 'page_title >= '.$dbr->addQuotes( $from ); + if( isset($to) ) + $where[] = 'page_title <= '.$dbr->addQuotes( $to ); global $wgMemc; - $key = wfMemcKey( 'allpages', 'ns', $namespace ); + $key = wfMemcKey( 'allpages', 'ns', $namespace, $from, $to ); $lines = $wgMemc->get( $key ); + + $count = $dbr->estimateRowCount( 'page', '*', $where, __METHOD__ ); + $maxPerSubpage = intval($count/$this->maxLineCount); + $maxPerSubpage = max($maxPerSubpage,$this->maxPerPage); if( !is_array( $lines ) ) { $options = array( 'LIMIT' => 1 ); - if ( ! $dbr->implicitOrderby() ) { - $options['ORDER BY'] = 'page_title'; - } + $options['ORDER BY'] = 'page_title ASC'; $firstTitle = $dbr->selectField( 'page', 'page_title', $where, __METHOD__, $options ); $lastTitle = $firstTitle; - # This array is going to hold the page_titles in order. $lines = array( $firstTitle ); - # If we are going to show n rows, we need n+1 queries to find the relevant titles. $done = false; - for( $i = 0; !$done; ++$i ) { + while( !$done ) { // Fetch the last title of this chunk and the first of the next $chunk = is_null( $lastTitle ) ? '' : 'page_title >= ' . $dbr->addQuotes( $lastTitle ); - $res = $dbr->select( - 'page', /* FROM */ + $chunk = array($chunk); + $res = $dbr->select( 'page', /* FROM */ 'page_title', /* WHAT */ - $where + array($chunk), + array_merge($where,$chunk), __METHOD__, - array ('LIMIT' => 2, 'OFFSET' => $this->maxPerPage - 1, 'ORDER BY' => 'page_title') ); + array ('LIMIT' => 2, 'OFFSET' => $maxPerSubpage - 1, 'ORDER BY' => 'page_title ASC') + ); - if ( $s = $dbr->fetchObject( $res ) ) { + if( $s = $dbr->fetchObject( $res ) ) { array_push( $lines, $s->page_title ); } else { // Final chunk, but ended prematurely. Go back and find the end. $endTitle = $dbr->selectField( 'page', 'MAX(page_title)', - array( - 'page_namespace' => $namespace, - $chunk - ), __METHOD__ ); + array_merge($where,$chunk), + __METHOD__ ); array_push( $lines, $endTitle ); $done = true; } @@ -165,35 +194,39 @@ function showToplevel ( $namespace = NS_MAIN, $including = false ) { // If there are only two or less sections, don't even display them. // Instead, display the first section directly. if( count( $lines ) <= 2 ) { - $this->showChunk( $namespace, '', $including ); + if( !empty($lines) ) { + $this->showChunk( $namespace, $lines[0], $lines[count($lines)-1], $including ); + } else { + $wgOut->addHtml( $this->namespaceForm( $namespace, $from, $to ) ); + } return; } # At this point, $lines should contain an even number of elements. $out .= ""; - while ( count ( $lines ) > 0 ) { - $inpoint = array_shift ( $lines ); - $outpoint = array_shift ( $lines ); - $out .= $this->showline ( $inpoint, $outpoint, $namespace, false ); + while( count ( $lines ) > 0 ) { + $inpoint = array_shift( $lines ); + $outpoint = array_shift( $lines ); + $out .= $this->showline( $inpoint, $outpoint, $namespace ); } $out .= '
'; - $nsForm = $this->namespaceForm( $namespace, '', false ); + $nsForm = $this->namespaceForm( $namespace, $from, $to ); # Is there more? - if ( $including ) { + if( $including ) { $out2 = ''; } else { - $morelinks = ''; - if ( $morelinks != '' ) { + if( isset($from) || isset($to) ) { + global $wgUser; $out2 = ''; $out2 .= '
' . $nsForm; - $out2 .= ''; - $out2 .= $morelinks . '
'; + $out2 .= '' . + $wgUser->getSkin()->makeKnownLink( $wgContLang->specialPage( "Allpages" ), wfMsgHtml ( 'allpages' ) ); + $out2 .= "
"; } else { $out2 = $nsForm . '
'; } } - $wgOut->addHtml( $out2 . $out ); } @@ -207,14 +240,17 @@ function showline( $inpoint, $outpoint, $namespace = NS_MAIN ) { $align = $wgContLang->isRtl() ? 'left' : 'right'; $inpointf = htmlspecialchars( str_replace( '_', ' ', $inpoint ) ); $outpointf = htmlspecialchars( str_replace( '_', ' ', $outpoint ) ); - $queryparams = ($namespace ? "namespace=$namespace" : ''); - $special = SpecialPage::getTitleFor( $this->name, $inpoint ); - $link = $special->escapeLocalUrl( $queryparams ); - - $out = wfMsgHtml( - 'alphaindexline', - "$inpointf", - "$outpointf" + // Don't let the length runaway + $inpointf = $wgContLang->truncate( $inpointf, $this->maxPageLength, '...' ); + $outpointf = $wgContLang->truncate( $outpointf, $this->maxPageLength, '...' ); + + $queryparams = $namespace ? "namespace=$namespace&" : ''; + $special = SpecialPage::getTitleFor( $this->name ); + $link = $special->escapeLocalUrl( $queryparams . 'from=' . urlencode($inpoint) . '&to=' . urlencode($outpoint) ); + + $out = wfMsgHtml( 'alphaindexline', + "$inpointf", + "$outpointf" ); return ''.$out.''; } @@ -222,19 +258,21 @@ function showline( $inpoint, $outpoint, $namespace = NS_MAIN ) { /** * @param integer $namespace (Default NS_MAIN) * @param string $from list all pages from this name (default FALSE) + * @param string $to list all pages to this name (default FALSE) */ -function showChunk( $namespace = NS_MAIN, $from, $including = false ) { +function showChunk( $namespace = NS_MAIN, $from = false, $to = false, $including = false ) { global $wgOut, $wgUser, $wgContLang; $sk = $wgUser->getSkin(); $fromList = $this->getNamespaceKeyAndText($namespace, $from); + $toList = $this->getNamespaceKeyAndText( $namespace, $to ); $namespaces = $wgContLang->getNamespaces(); $align = $wgContLang->isRtl() ? 'left' : 'right'; $n = 0; - if ( !$fromList ) { + if ( !$fromList || !$toList ) { $out = wfMsgWikiHtml( 'allpagesbadtitle' ); } elseif ( !in_array( $namespace, array_keys( $namespaces ) ) ) { // Show errormessage and reset to NS_MAIN @@ -242,14 +280,20 @@ function showChunk( $namespace = NS_MAIN, $from, $including = false ) { $namespace = NS_MAIN; } else { list( $namespace, $fromKey, $from ) = $fromList; + list( $namespace, $toKey, $to ) = $toList; $dbr = wfGetDB( DB_SLAVE ); + $conds = array( + 'page_namespace' => $namespace, + 'page_title >= ' . $dbr->addQuotes( $fromKey ) + ); + if( $toKey !== "" ) { + $conds[] = 'page_title <= ' . $dbr->addQuotes( $toKey ); + } + $res = $dbr->select( 'page', array( 'page_namespace', 'page_title', 'page_is_redirect' ), - array( - 'page_namespace' => $namespace, - 'page_title >= ' . $dbr->addQuotes( $fromKey ) - ), + $conds, __METHOD__, array( 'ORDER BY' => 'page_title', @@ -316,7 +360,8 @@ function showChunk( $namespace = NS_MAIN, $from, $including = false ) { if ( ! $dbr->implicitOrderby() ) { $options['ORDER BY'] = 'page_title'; } - $reallyFirstPage_title = $dbr->selectField( 'page', 'page_title', array( 'page_namespace' => $namespace ), __METHOD__, $options ); + $reallyFirstPage_title = $dbr->selectField( 'page', 'page_title', + array( 'page_namespace' => $namespace ), __METHOD__, $options ); # Show the previous link if it s not the current requested chunk if( $from != $reallyFirstPage_title ) { $prevTitle = Title::makeTitle( $namespace, $reallyFirstPage_title ); @@ -326,7 +371,7 @@ function showChunk( $namespace = NS_MAIN, $from, $including = false ) { } } - $nsForm = $this->namespaceForm( $namespace, $from ); + $nsForm = $this->namespaceForm( $namespace, $from, $to ); $out2 = ''; $out2 .= '
' . $nsForm; $out2 .= '' . diff --git a/languages/messages/MessagesEn.php b/languages/messages/MessagesEn.php index 78c1d6c77e..ee91e3679f 100644 --- a/languages/messages/MessagesEn.php +++ b/languages/messages/MessagesEn.php @@ -2004,6 +2004,7 @@ You can narrow down the view by selecting a log type, the user name, or the affe 'nextpage' => 'Next page ($1)', 'prevpage' => 'Previous page ($1)', 'allpagesfrom' => 'Display pages starting at:', +'allpagesto' => 'Display pages ending at:', 'allarticles' => 'All pages', 'allinnamespace' => 'All pages ($1 namespace)', 'allnotinnamespace' => 'All pages (not in $1 namespace)', diff --git a/maintenance/language/messages.inc b/maintenance/language/messages.inc index 1597e7d14d..3da6225845 100644 --- a/maintenance/language/messages.inc +++ b/maintenance/language/messages.inc @@ -1285,6 +1285,7 @@ $wgMessageStructure = array( 'nextpage', 'prevpage', 'allpagesfrom', + 'allpagesto', 'allarticles', 'allinnamespace', 'allnotinnamespace',