X-Git-Url: https://git.heureux-cyclage.org/?a=blobdiff_plain;f=includes%2FSiteStats.php;h=6a2d0e2adada89fe64ed1bbf65da995ff814fdc0;hb=42ba9ea516beb06969bd3e579aa506821629ecb8;hp=6ce1aed814022e036ee0e06b50b8e6943f5f222d;hpb=7f3d6713e7191c476adb3d3c30344a0327b853d3;p=lhc%2Fweb%2Fwiklou.git

diff --git a/includes/SiteStats.php b/includes/SiteStats.php
index 6ce1aed814..6a2d0e2ada 100644
--- a/includes/SiteStats.php
+++ b/includes/SiteStats.php
@@ -34,9 +34,6 @@ class SiteStats {
 	/** @var bool */
 	private static $loaded = false;
 
-	/** @var int */
-	private static $jobs;
-
 	/** @var int[] */
 	private static $pageCount = [];
 
@@ -213,24 +210,24 @@ class SiteStats {
 	}
 
 	/**
+	 * Total number of jobs in the job queue.
 	 * @return int
 	 */
 	static function jobs() {
-		if ( !isset( self::$jobs ) ) {
-			try{
-				self::$jobs = array_sum( JobQueueGroup::singleton()->getQueueSizes() );
-			} catch ( JobQueueError $e ) {
-				self::$jobs = 0;
-			}
-			/**
-			 * Zero rows still do single row read for row that doesn't exist,
-			 * but people are annoyed by that
-			 */
-			if ( self::$jobs == 1 ) {
-				self::$jobs = 0;
-			}
-		}
-		return self::$jobs;
+		$cache = MediaWikiServices::getInstance()->getMainWANObjectCache();
+		return $cache->getWithSetCallback(
+			$cache->makeKey( 'SiteStats', 'jobscount' ),
+			$cache::TTL_MINUTE,
+			function ( $oldValue, &$ttl, array &$setOpts ) {
+				try{
+					$jobs = array_sum( JobQueueGroup::singleton()->getQueueSizes() );
+				} catch ( JobQueueError $e ) {
+					$jobs = 0;
+				}
+				return $jobs;
+			},
+			[ 'pcTTL' => $cache::TTL_PROC_LONG ]
+		);
 	}
 
 	/**
@@ -296,7 +293,6 @@ class SiteStatsInit {
 	private $mUsers = null, $mFiles = null;
 
 	/**
-	 * Constructor
 	 * @param bool|IDatabase $database
 	 * - boolean: Whether to use the master DB
 	 * - IDatabase: Database connection to use