Anomie has uploaded a new change for review.
https://gerrit.wikimedia.org/r/203840
Change subject: Message::inLanguage() shouldn't unstub StubUserLang
......................................................................
Message::inLanguage() shouldn't unstub StubUserLang
When a string is passed to Message::inLanguage(), it first checks
whether the message's current language's code is equal to the string, to
avoid a call to Language::factory(). But if the message's current
language is an instance of StubUserLang, it's probably less expensive to
just call Language::factory() than it is to unstub.
This also avoids a possible recursion warning from T56193, particularly
if inLanguage() is being used intentionally in an attempt to avoid that
warning.
Change-Id: Ia09adec05cfbb09c09e07c6be1e2d613435664d9
---
M includes/Message.php
1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/core
refs/changes/40/203840/1
diff --git a/includes/Message.php b/includes/Message.php
index 134af0e..4935e33 100644
--- a/includes/Message.php
+++ b/includes/Message.php
@@ -597,7 +597,7 @@
if ( $lang instanceof Language || $lang instanceof StubUserLang
) {
$this->language = $lang;
} elseif ( is_string( $lang ) ) {
- if ( $this->language->getCode() != $lang ) {
+ if ( !$this->language instanceof Language ||
$this->language->getCode() != $lang ) {
$this->language = Language::factory( $lang );
}
} else {
--
To view, visit https://gerrit.wikimedia.org/r/203840
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ia09adec05cfbb09c09e07c6be1e2d613435664d9
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/core
Gerrit-Branch: master
Gerrit-Owner: Anomie <bjor...@wikimedia.org>
_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
