| Index: trunk/extensions/CodeReview/ui/CodeRevisionCommitter.php |
| — | — | @@ -57,22 +57,23 @@ |
| 58 | 58 | } |
| 59 | 59 | $dbw->commit(); |
| 60 | 60 | |
| 61 | | - if ( $statusChanged || $commentAdded ) { |
| 62 | | - if ( $statusChanged && $commentAdded ) { |
| 63 | | - $url = $this->mRev->getFullUrl( $commentId ); |
| 64 | | - $this->mRev->emailNotifyUsersOfChanges( 'codereview-email-subj4', 'codereview-email-body4', |
| 65 | | - $wgUser->getName(), $this->mRev->getIdStringUnique(), $this->mRev->mOldStatus, $this->mRev->mStatus, |
| | 61 | + if ( $statusChanged || $commentAdded ) { |
| | 62 | + if ( $statusChanged && $commentAdded ) { |
| | 63 | + $url = $this->mRev->getFullUrl( $commentId ); |
| | 64 | + $this->mRev->emailNotifyUsersOfChanges( 'codereview-email-subj4', 'codereview-email-body4', |
| | 65 | + $wgUser->getName(), $this->mRev->getIdStringUnique(), $this->mRev->mOldStatus, $this->mRev->mStatus, |
| 66 | 66 | $url, $this->text |
| 67 | | - ); |
| 68 | | - } else if ( $statusChanged ) { |
| | 67 | + ); |
| | 68 | + } else if ( $statusChanged ) { |
| 69 | 69 | $this->mRev->emailNotifyUsersOfChanges( 'codereview-email-subj3', 'codereview-email-body3', |
| 70 | 70 | $wgUser->getName(), $this->mRev->getIdStringUnique(), $this->mRev->mOldStatus, $this->mRev->mStatus |
| 71 | | - ); |
| 72 | | - } else if ( $commentAdded ) { |
| 73 | | - $url = $this->mRev->getFullUrl( $commentId ); |
| 74 | | - $this->mRev->emailNotifyUsersOfChanges( 'codereview-email-subj', 'codereview-email-body', |
| 75 | | - $wgUser->getName(), $url, $this->mRev->getIdStringUnique(), $this->text ); |
| 76 | | - } |
| | 71 | + ); |
| | 72 | + } else if ( $commentAdded ) { |
| | 73 | + $url = $this->mRev->getFullUrl( $commentId ); |
| | 74 | + $this->mRev->emailNotifyUsersOfChanges( 'codereview-email-subj', 'codereview-email-body', |
| | 75 | + $wgUser->getName(), $url, $this->mRev->getIdStringUnique(), $this->text |
| | 76 | + ); |
| | 77 | + } |
| 77 | 78 | } |
| 78 | 79 | |
| 79 | 80 | // Return to rev page |