http://www.mediawiki.org/wiki/Special:Code/MediaWiki/70586
Revision: 70586 Author: reedy Date: 2010-08-06 19:17:45 +0000 (Fri, 06 Aug 2010) Log Message: ----------- If the user can't retrieve email, theres no point getting their lang! Modified Paths: -------------- trunk/extensions/CodeReview/backend/CodeRevision.php Modified: trunk/extensions/CodeReview/backend/CodeRevision.php =================================================================== --- trunk/extensions/CodeReview/backend/CodeRevision.php 2010-08-06 19:07:56 UTC (rev 70585) +++ trunk/extensions/CodeReview/backend/CodeRevision.php 2010-08-06 19:17:45 UTC (rev 70586) @@ -399,14 +399,17 @@ $title = SpecialPage::getTitleFor( 'Code', $this->mRepo->getName() . '/' . $this->mId ); $title->setFragment( "#c{$commentId}" ); $url = $title->getFullUrl(); + foreach ( $users as $userId => $user ) { // No sense in notifying this commenter if ( $wgUser->getId() == $user->getId() ) { continue; } - // Send message in receiver's language - $lang = array( 'language' => $user->getOption( 'language' ) ); + if ( $user->canReceiveEmail() ) { + // Send message in receiver's language + $lang = array( 'language' => $user->getOption( 'language' ) ); + $user->sendMail( wfMsgExt( 'codereview-email-subj', $lang, $this->mRepo->getName(), $this->getIdString() ), wfMsgExt( 'codereview-email-body', $lang, $wgUser->getName(), $url, $this->getIdStringUnique(), $text ) _______________________________________________ MediaWiki-CVS mailing list MediaWiki-CVS@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-cvs