Legoktm has uploaded a new change for review.
https://gerrit.wikimedia.org/r/191196
Change subject: skin: Use Title::isKnown() to check page existence in the
personal urls
......................................................................
skin: Use Title::isKnown() to check page existence in the personal urls
This code was added in r3563, which predates both Title::exists() (r8594)
and Title::isKnown() (r44520). Since we're just trying to determine
what color to make these links, we should be using isKnown().
Bug: T85550
Change-Id: I47c8af8485857cc0f3190b4ada1578ab672580af
---
M includes/skins/Skin.php
1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/core
refs/changes/96/191196/1
diff --git a/includes/skins/Skin.php b/includes/skins/Skin.php
index 6cc139b..3d5dc3f 100644
--- a/includes/skins/Skin.php
+++ b/includes/skins/Skin.php
@@ -1166,7 +1166,7 @@
return array(
'href' => $title->getLocalURL( $urlaction ),
- 'exists' => $title->getArticleID() != 0,
+ 'exists' => $title->isKnown(),
);
}
--
To view, visit https://gerrit.wikimedia.org/r/191196
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I47c8af8485857cc0f3190b4ada1578ab672580af
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/core
Gerrit-Branch: master
Gerrit-Owner: Legoktm <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
