jenkins-bot has submitted this change and it was merged.
Change subject: Check in both userspace and usertalkspace if the user exists
......................................................................
Check in both userspace and usertalkspace if the user exists
Change-Id: Iefe6e0447ae1c13c3ad89f4c8b20f135d5f615bd
---
M MassMessageJob.php
1 file changed, 2 insertions(+), 2 deletions(-)
Approvals:
Legoktm: Looks good to me, approved
jenkins-bot: Verified
diff --git a/MassMessageJob.php b/MassMessageJob.php
index 8ca9140..676903d 100644
--- a/MassMessageJob.php
+++ b/MassMessageJob.php
@@ -104,9 +104,9 @@
return true; // Oh well.
}
- // If we're sending to a User talk: page, make sure the user
exists.
+ // If we're sending to a User:/User talk: page, make sure the
user exists.
// Redirects are automatically followed in getLocalTargets
- if ( $title->getNamespace() == NS_USER_TALK ) {
+ if ( $title->getNamespace() == NS_USER ||
$title->getNamespace() == NS_USER_TALK ) {
$user = User::newFromName( $title->getBaseText() );
if ( !$user->getId() ) { // Does not exist
return true; // Should we log anything here?
--
To view, visit https://gerrit.wikimedia.org/r/79756
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Iefe6e0447ae1c13c3ad89f4c8b20f135d5f615bd
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/MassMessage
Gerrit-Branch: master
Gerrit-Owner: Legoktm <legoktm.wikipe...@gmail.com>
Gerrit-Reviewer: Legoktm <legoktm.wikipe...@gmail.com>
Gerrit-Reviewer: jenkins-bot
_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
