Legoktm has uploaded a new change for review. https://gerrit.wikimedia.org/r/79756
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(-) git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/MassMessage refs/changes/56/79756/1 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: newchange Gerrit-Change-Id: Iefe6e0447ae1c13c3ad89f4c8b20f135d5f615bd Gerrit-PatchSet: 1 Gerrit-Project: mediawiki/extensions/MassMessage Gerrit-Branch: master Gerrit-Owner: Legoktm <legoktm.wikipe...@gmail.com> _______________________________________________ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits