| Index: branches/wmf/1.16wmf4/extensions/CodeReview/backend/CodeRevision.php |
| — | — | @@ -432,7 +432,9 @@ |
| 433 | 433 | continue; |
| 434 | 434 | } |
| 435 | 435 | |
| 436 | | - if ( $user->canReceiveEmail() ) { |
| | 436 | + // canReceiveEmail() returns false for the fake watcher user, so exempt it |
| | 437 | + // This is ugly |
| | 438 | + if ( $userId == 0 || $user->canReceiveEmail() ) { |
| 437 | 439 | // Send message in receiver's language |
| 438 | 440 | $lang = array( 'language' => $user->getOption( 'language' ) ); |
| 439 | 441 | |
| Property changes on: branches/wmf/1.16wmf4/extensions/CodeReview/ui/CodeRevisionView.php |
| ___________________________________________________________________ |
| Modified: svn:mergeinfo |
| 440 | 442 | Merged /trunk/extensions/CodeReview/ui/CodeRevisionView.php:r77698 |
| Property changes on: branches/wmf/1.16wmf4/extensions/CodeReview |
| ___________________________________________________________________ |
| Modified: svn:mergeinfo |
| 441 | 443 | Merged /trunk/extensions/CodeReview:r77698 |