X-Git-Url: https://git.heureux-cyclage.org/?a=blobdiff_plain;f=includes%2FCategory.php;h=77f1212fcfb2287493e7633fc4dcd823a40f0e7a;hb=571a44a4bef0cdecc329636bfaa8f3935380c3f0;hp=f8ac8aeb59d1ccdca3a90e75550a959a09dfaef5;hpb=2252838b4a0ea09d393c26281eae62ee8c5cda98;p=lhc%2Fweb%2Fwiklou.git

diff --git a/includes/Category.php b/includes/Category.php
index f8ac8aeb59..77f1212fcf 100644
--- a/includes/Category.php
+++ b/includes/Category.php
@@ -25,8 +25,6 @@
  * Category objects are immutable, strictly speaking. If you call methods that change the database,
  * like to refresh link counts, the objects will be appropriately reinitialized.
  * Member variables are lazy-initialized.
- *
- * @todo Move some stuff from CategoryPage.php to here, and use that.
  */
 class Category {
 	/** Name of the category, normalized to DB-key form */
@@ -299,10 +297,10 @@ class Category {
 	/**
 	 * Generic accessor
 	 * @param string $key
-	 * @return bool
+	 * @return mixed
 	 */
 	private function getX( $key ) {
-		if ( !$this->initialize( self::LAZY_INIT_ROW ) ) {
+		if ( $this->{$key} === null && !$this->initialize( self::LAZY_INIT_ROW ) ) {
 			return false;
 		}
 		return $this->{$key};
@@ -420,4 +418,45 @@ class Category {
 
 		return true;
 	}
+
+	/**
+	 * Call refreshCounts() if there are no entries in the categorylinks table
+	 * or if the category table has a row that states that there are no entries
+	 *
+	 * Due to lock errors or other failures, the precomputed counts can get out of sync,
+	 * making it hard to know when to delete the category row without checking the
+	 * categorylinks table.
+	 *
+	 * @return bool Whether links were refreshed
+	 * @since 1.32
+	 */
+	public function refreshCountsIfEmpty() {
+		$dbw = wfGetDB( DB_MASTER );
+
+		$hasLink = $dbw->selectField(
+			'categorylinks',
+			'1',
+			[ 'cl_to' => $this->getName() ],
+			__METHOD__
+		);
+		if ( !$hasLink ) {
+			$this->refreshCounts(); // delete any category table entry
+
+			return true;
+		}
+
+		$hasBadRow = $dbw->selectField(
+			'category',
+			'1',
+			[ 'cat_title' => $this->getName(), 'cat_pages <= 0' ],
+			__METHOD__
+		);
+		if ( $hasBadRow ) {
+			$this->refreshCounts(); // clean up this row
+
+			return true;
+		}
+
+		return false;
+	}
 }