X-Git-Url: https://git.heureux-cyclage.org/?p=lhc%2Fweb%2Fwiklou.git;a=blobdiff_plain;f=includes%2Fspecials%2FSpecialDoubleRedirects.php;h=d73ac19875c0879733b5ef993f984a658bd0144b;hp=59351dcba52a05fd231f7f47ab4016d4a435802a;hb=6c9a2923fe1ee3a65cb027be5e781772f2b12fbd;hpb=baa9036839396fb3b3a12bc06f52291b083a226d

diff --git a/includes/specials/SpecialDoubleRedirects.php b/includes/specials/SpecialDoubleRedirects.php
index 59351dcba5..d73ac19875 100644
--- a/includes/specials/SpecialDoubleRedirects.php
+++ b/includes/specials/SpecialDoubleRedirects.php
@@ -22,6 +22,7 @@
  */
 
 use Wikimedia\Rdbms\ResultWrapper;
+use Wikimedia\Rdbms\IDatabase;
 
 /**
  * A special page listing redirects to redirecting page.
@@ -65,14 +66,15 @@ class DoubleRedirectsPage extends QueryPage {
 				'title' => 'pa.page_title',
 				'value' => 'pa.page_title',
 
-				'nsb' => 'pb.page_namespace',
-				'tb' => 'pb.page_title',
+				'b_namespace' => 'pb.page_namespace',
+				'b_title' => 'pb.page_title',
 
 				// Select fields from redirect instead of page. Because there may
 				// not actually be a page table row for this target (e.g. for interwiki redirects)
-				'nsc' => 'rb.rd_namespace',
-				'tc' => 'rb.rd_title',
-				'iwc' => 'rb.rd_interwiki',
+				'c_namespace' => 'rb.rd_namespace',
+				'c_title' => 'rb.rd_title',
+				'c_fragment' => 'rb.rd_fragment',
+				'c_interwiki' => 'rb.rd_interwiki',
 			],
 			'conds' => [
 				'ra.rd_from = pa.page_id',
@@ -115,45 +117,41 @@ class DoubleRedirectsPage extends QueryPage {
 	 * @return string
 	 */
 	function formatResult( $skin, $result ) {
-		$titleA = Title::makeTitle( $result->namespace, $result->title );
-
-		// If only titleA is in the query, it means this came from
-		// querycache (which only saves 3 columns).
+		// If no Title B or C is in the query, it means this came from
+		// querycache (which only saves the 3 columns for title A).
 		// That does save the bulk of the query cost, but now we need to
 		// get a little more detail about each individual entry quickly
 		// using the filter of reallyGetQueryInfo.
-		if ( $result && !isset( $result->nsb ) ) {
-			$dbr = wfGetDB( DB_REPLICA );
-			$qi = $this->reallyGetQueryInfo(
-				$result->namespace,
-				$result->title
-			);
-			$res = $dbr->select(
-				$qi['tables'],
-				$qi['fields'],
-				$qi['conds'],
-				__METHOD__
-			);
-
-			if ( $res ) {
-				$result = $dbr->fetchObject( $res );
+		$deep = false;
+		if ( $result ) {
+			if ( isset( $result->b_namespace ) ) {
+				$deep = $result;
+			} else {
+				$dbr = wfGetDB( DB_REPLICA );
+				$qi = $this->reallyGetQueryInfo(
+					$result->namespace,
+					$result->title
+				);
+				$res = $dbr->select(
+					$qi['tables'],
+					$qi['fields'],
+					$qi['conds'],
+					__METHOD__
+				);
+
+				if ( $res ) {
+					$deep = $dbr->fetchObject( $res ) ?: false;
+				}
 			}
 		}
+
+		$titleA = Title::makeTitle( $result->namespace, $result->title );
+
 		$linkRenderer = $this->getLinkRenderer();
-		if ( !$result ) {
+		if ( !$deep ) {
 			return '<del>' . $linkRenderer->makeLink( $titleA, null, [], [ 'redirect' => 'no' ] ) . '</del>';
 		}
 
-		$titleB = Title::makeTitle( $result->nsb, $result->tb );
-		$titleC = Title::makeTitle( $result->nsc, $result->tc, '', $result->iwc );
-
-		$linkA = $linkRenderer->makeKnownLink(
-			$titleA,
-			null,
-			[],
-			[ 'redirect' => 'no' ]
-		);
-
 		// if the page is editable, add an edit link
 		if (
 			// check user permissions
@@ -171,6 +169,14 @@ class DoubleRedirectsPage extends QueryPage {
 			$edit = '';
 		}
 
+		$linkA = $linkRenderer->makeKnownLink(
+			$titleA,
+			null,
+			[],
+			[ 'redirect' => 'no' ]
+		);
+
+		$titleB = Title::makeTitle( $deep->b_namespace, $deep->b_title );
 		$linkB = $linkRenderer->makeKnownLink(
 			$titleB,
 			null,
@@ -178,7 +184,13 @@ class DoubleRedirectsPage extends QueryPage {
 			[ 'redirect' => 'no' ]
 		);
 
-		$linkC = $linkRenderer->makeKnownLink( $titleC );
+		$titleC = Title::makeTitle(
+			$deep->c_namespace,
+			$deep->c_title,
+			$deep->c_fragment,
+			$deep->c_interwiki
+		);
+		$linkC = $linkRenderer->makeKnownLink( $titleC, $titleC->getFullText() );
 
 		$lang = $this->getLanguage();
 		$arr = $lang->getArrow() . $lang->getDirMark();
@@ -200,13 +212,13 @@ class DoubleRedirectsPage extends QueryPage {
 		$batch = new LinkBatch;
 		foreach ( $res as $row ) {
 			$batch->add( $row->namespace, $row->title );
-			if ( isset( $row->nsb ) ) {
+			if ( isset( $row->b_namespace ) ) {
 				// lazy loaded when using cached results
-				$batch->add( $row->nsb, $row->tb );
+				$batch->add( $row->b_namespace, $row->b_title );
 			}
-			if ( isset( $row->iwc ) && !$row->iwc ) {
+			if ( isset( $row->c_interwiki ) && !$row->c_interwiki ) {
 				// lazy loaded when using cached result, not added when interwiki link
-				$batch->add( $row->nsc, $row->tc );
+				$batch->add( $row->c_namespace, $row->c_title );
 			}
 		}
 		$batch->execute();