X-Git-Url: https://git.heureux-cyclage.org/?a=blobdiff_plain;f=includes%2Fspecials%2FSpecialWantedcategories.php;h=7ddafae40e682170f1fe3df694625ba93d5c57b0;hb=b51076a84446d157bed511246450e70d26e0f945;hp=0035bfabfbd83a29f4b247b636927fe575caf31c;hpb=5a63f7084342e1716267f14e448f8513ff0614fd;p=lhc%2Fweb%2Fwiklou.git

diff --git a/includes/specials/SpecialWantedcategories.php b/includes/specials/SpecialWantedcategories.php
index 0035bfabfb..7ddafae40e 100644
--- a/includes/specials/SpecialWantedcategories.php
+++ b/includes/specials/SpecialWantedcategories.php
@@ -29,28 +29,62 @@
  * @ingroup SpecialPage
  */
 class WantedCategoriesPage extends WantedQueryPage {
+	private $currentCategoryCounts;
 
 	function __construct( $name = 'Wantedcategories' ) {
 		parent::__construct( $name );
 	}
 
 	function getQueryInfo() {
-		return array (
-			'tables' => array ( 'categorylinks', 'page' ),
-			'fields' => array ( 'namespace' => NS_CATEGORY,
-					'title' => 'cl_to',
-					'value' => 'COUNT(*)' ),
-			'conds' => array ( 'page_title IS NULL' ),
-			'options' => array ( 'GROUP BY' => 'cl_to' ),
-			'join_conds' => array ( 'page' => array ( 'LEFT JOIN',
-				array ( 'page_title = cl_to',
+		return array(
+			'tables' => array( 'categorylinks', 'page' ),
+			'fields' => array(
+				'namespace' => NS_CATEGORY,
+				'title' => 'cl_to',
+				'value' => 'COUNT(*)'
+			),
+			'conds' => array( 'page_title IS NULL' ),
+			'options' => array( 'GROUP BY' => 'cl_to' ),
+			'join_conds' => array( 'page' => array( 'LEFT JOIN',
+				array( 'page_title = cl_to',
 					'page_namespace' => NS_CATEGORY ) ) )
 		);
 	}
 
+	function preprocessResults( $db, $res ) {
+		parent::preprocessResults( $db, $res );
+
+		$this->currentCategoryCounts = array();
+
+		if ( !$res->numRows() || !$this->isCached() ) {
+			return;
+		}
+
+		// Fetch (hopefully) up-to-date numbers of pages in each category.
+		// This should be fast enough as we limit the list to a reasonable length.
+
+		$allCategories = array();
+		foreach ( $res as $row ) {
+			$allCategories[] = $row->title;
+		}
+
+		$categoryRes = $db->select(
+			'category',
+			array( 'cat_title', 'cat_pages' ),
+			array( 'cat_title' => $allCategories ),
+			__METHOD__
+		);
+		foreach ( $categoryRes as $row ) {
+			$this->currentCategoryCounts[$row->cat_title] = intval( $row->cat_pages );
+		}
+
+		// Back to start for display
+		$res->seek( 0 );
+	}
+
 	/**
-	 * @param $skin Skin
-	 * @param $result
+	 * @param Skin $skin
+	 * @param object $result Result row
 	 * @return string
 	 */
 	function formatResult( $skin, $result ) {
@@ -59,17 +93,38 @@ class WantedCategoriesPage extends WantedQueryPage {
 		$nt = Title::makeTitle( $result->namespace, $result->title );
 		$text = htmlspecialchars( $wgContLang->convert( $nt->getText() ) );
 
-		$plink = $this->isCached() ?
-			Linker::link( $nt, $text ) :
-			Linker::link(
+		if ( !$this->isCached() ) {
+			// We can assume the freshest data
+			$plink = Linker::link(
 				$nt,
 				$text,
 				array(),
 				array(),
 				array( 'broken' )
 			);
+			$nlinks = $this->msg( 'nmembers' )->numParams( $result->value )->escaped();
+		} else {
+			$plink = Linker::link( $nt, $text );
+
+			$currentValue = isset( $this->currentCategoryCounts[$result->title] )
+				? $this->currentCategoryCounts[$result->title]
+				: 0;
+			$cachedValue = intval( $result->value ); // T76910
+
+			// If the category has been created or emptied since the list was refreshed, strike it
+			if ( $nt->isKnown() || $currentValue === 0 ) {
+				$plink = "<del>$plink</del>";
+			}
+
+			// Show the current number of category entries if it changed
+			if ( $currentValue !== $cachedValue ) {
+				$nlinks = $this->msg( 'nmemberschanged' )
+					->numParams( $cachedValue, $currentValue )->escaped();
+			} else {
+				$nlinks = $this->msg( 'nmembers' )->numParams( $cachedValue )->escaped();
+			}
+		}
 
-		$nlinks = $this->msg( 'nmembers' )->numParams( $result->value )->escaped();
 		return $this->getLanguage()->specialList( $plink, $nlinks );
 	}