jenkins-bot has submitted this change and it was merged.

Change subject: Replace deprecated Linker::linkKnown with makeKnownLink
......................................................................


Replace deprecated Linker::linkKnown with makeKnownLink

Bug: T121145
Change-Id: If8c06723efb3bcb85860bfc8ee67584f7a92a1b9
---
M frontend/specialpages/actions/ConfirmAccount_body.php
1 file changed, 16 insertions(+), 13 deletions(-)

Approvals:
  Aaron Schulz: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/frontend/specialpages/actions/ConfirmAccount_body.php 
b/frontend/specialpages/actions/ConfirmAccount_body.php
index 7baf302..bb2765e 100644
--- a/frontend/specialpages/actions/ConfirmAccount_body.php
+++ b/frontend/specialpages/actions/ConfirmAccount_body.php
@@ -118,16 +118,17 @@
        protected function addQueueSubtitleLinks() {
                $titleObj = $this->getFullTitle();
 
+               $linkRenderer = $this->getLinkRenderer();
+
                # Show other sub-queue links. Grey out the current one.
                # When viewing a request, show them all.
                if ( $this->acrID || $this->showStale || $this->showRejects || 
$this->showHeld ) {
-                       $listLink = Linker::linkKnown(
+                       $listLink = $linkRenderer->makeKnownLink(
                                $titleObj,
-                               $this->msg( 'confirmaccount-showopen' 
)->escaped() );
+                               $this->msg( 'confirmaccount-showopen' ) );
                } else {
                        $listLink = $this->msg( 'confirmaccount-showopen' 
)->escaped();
                }
-               $linkRenderer = $this->getLinkRenderer();
                if ( $this->acrID || !$this->showHeld ) {
                        $listLink = $this->getLanguage()->pipeList( [
                                $listLink,
@@ -179,9 +180,9 @@
 
                # Say what queue we are in...
                if ( $this->queueType != -1 ) {
-                       $viewall = Linker::linkKnown(
+                       $viewall = $linkRenderer->makeKnownLink(
                                $this->getPageTitle(),
-                               $this->msg( 'confirmaccount-all' )->escaped()
+                               $this->msg( 'confirmaccount-all' )
                        );
 
                        // Give grep a chance to find the usages: 
confirmaccount-type-0, confirmaccount-type-1
@@ -206,37 +207,39 @@
                        $titleObj = SpecialPage::getTitleFor( 
'ConfirmAccounts', $params[0] );
                        $counts = ConfirmAccount::getOpenRequestCount( $i );
 
+                       $linkRenderer = $this->getLinkRenderer();
+
                        $open = '<b>';
-                       $open .= Linker::linkKnown(
+                       $open .= $linkRenderer->makeKnownLink(
                                $titleObj,
                                $this->msg( 'confirmaccount-q-open' 
)->escaped(),
                                [],
-                               'wpShowHeld=0'
+                               [ 'wpShowHeld' => 0 ]
                        );
                        $open .= '</b> [' . $counts['open'] . ']';
 
-                       $held = Linker::linkKnown(
+                       $held = $linkRenderer->makeKnownLink(
                                $titleObj,
                                $this->msg( 'confirmaccount-q-held' 
)->escaped(),
                                [],
-                               'wpShowHeld=1'
+                               [ 'wpShowHeld' => 1 ]
                        );
                        $held .= ' [' . $counts['held'] . ']';
 
-                       $rejects = Linker::linkKnown(
+                       $rejects = $linkRenderer->makeKnownLink(
                                $titleObj,
                                $this->msg( 'confirmaccount-q-rej' )->escaped(),
                                [],
-                               'wpShowRejects=1'
+                               [ 'wpShowRejects' => 1 ]
                        );
                        $rejects .= ' [' . $counts['rejected'] . ']';
 
                        $stale = '<i>';
-                       $stale .= Linker::linkKnown(
+                       $stale .= $linkRenderer->makeKnownLink(
                                $titleObj,
                                $this->msg( 'confirmaccount-q-stale' 
)->escaped(),
                                [],
-                               'wpShowStale=1'
+                               [ 'wpShowStale' => 1 ]
                        );
                        $stale .= '</i>';
 

-- 
To view, visit https://gerrit.wikimedia.org/r/305936
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: If8c06723efb3bcb85860bfc8ee67584f7a92a1b9
Gerrit-PatchSet: 8
Gerrit-Project: mediawiki/extensions/ConfirmAccount
Gerrit-Branch: master
Gerrit-Owner: Paladox <[email protected]>
Gerrit-Reviewer: Aaron Schulz <[email protected]>
Gerrit-Reviewer: Addshore <[email protected]>
Gerrit-Reviewer: Hashar <[email protected]>
Gerrit-Reviewer: JanZerebecki <[email protected]>
Gerrit-Reviewer: Jforrester <[email protected]>
Gerrit-Reviewer: Legoktm <[email protected]>
Gerrit-Reviewer: Paladox <[email protected]>
Gerrit-Reviewer: Umherirrender <[email protected]>
Gerrit-Reviewer: jenkins-bot <>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to