| Index: branches/wmf/1.17wmf1/extensions/CodeReview/backend/CodeRevision.php |
| — | — | @@ -437,14 +437,14 @@ |
| 438 | 438 | } |
| 439 | 439 | |
| 440 | 440 | /** |
| 441 | | - * @param $subject |
| 442 | | - * @param $body |
| | 441 | + * @param $subject String |
| | 442 | + * @param $body String |
| 443 | 443 | * @return void |
| 444 | 444 | */ |
| 445 | 445 | public function emailNotifyUsersOfChanges( $subject, $body ) { |
| 446 | 446 | // Give email notices to committer and commenters |
| 447 | 447 | global $wgCodeReviewENotif, $wgEnableEmail, $wgCodeReviewCommentWatcherEmail, |
| 448 | | - $wgCodeReviewCommentWatcherName, $wgUser; |
| | 448 | + $wgCodeReviewCommentWatcherName; |
| 449 | 449 | if ( !$wgCodeReviewENotif || !$wgEnableEmail ) { |
| 450 | 450 | return; |
| 451 | 451 | } |
| — | — | @@ -475,7 +475,7 @@ |
| 476 | 476 | $lang = array( 'language' => $user->getOption( 'language' ) ); |
| 477 | 477 | |
| 478 | 478 | $localSubject = wfMsgExt( $subject, $lang, $this->mRepo->getName(), $this->getIdString() ); |
| 479 | | - $localBody = call_user_func_array( 'wfMsgExt', array_merge( array( $body, $lang, $wgUser->getName() ), $args ) ); |
| | 479 | + $localBody = call_user_func_array( 'wfMsgExt', array_merge( array( $body, $lang ), $args ) ); |
| 480 | 480 | |
| 481 | 481 | $user->sendMail( $localSubject, $localBody ); |
| 482 | 482 | } |