Index: trunk/extensions/CodeReview/CodeReview.i18n.php |
— | — | @@ -162,8 +162,10 @@ |
163 | 163 | 'codereview-email-body3' => 'User "$1" changed the status of $2. |
164 | 164 | |
165 | 165 | Old Status: $3 |
166 | | -New Status: $4', |
| 166 | +New Status: $4 |
167 | 167 | |
| 168 | +Full URL: $5', |
| 169 | + |
168 | 170 | 'codereview-email-subj4' => '[$1 $2]: New comment added, and revision status changed', |
169 | 171 | 'codereview-email-body4' => 'User "$1" changed the status of $2. |
170 | 172 | |
Index: trunk/extensions/CodeReview/ui/CodeRevisionCommitter.php |
— | — | @@ -117,18 +117,18 @@ |
118 | 118 | $dbw->commit(); |
119 | 119 | |
120 | 120 | if ( $statusChanged || $commentAdded ) { |
| 121 | + $url = $this->mRev->getFullUrl( $commentId ); |
121 | 122 | if ( $statusChanged && $commentAdded ) { |
122 | | - $url = $this->mRev->getFullUrl( $commentId ); |
123 | 123 | $this->mRev->emailNotifyUsersOfChanges( 'codereview-email-subj4', 'codereview-email-body4', |
124 | | - $wgUser->getName(), $this->mRev->getIdStringUnique(), $this->mRev->getOldStatus(), $this->mRev->getStatus(), |
125 | | - $url, $this->text |
| 124 | + $wgUser->getName(), $this->mRev->getIdStringUnique(), $this->mRev->getOldStatus(), |
| 125 | + $this->mRev->getStatus(), $url, $this->text |
126 | 126 | ); |
127 | 127 | } else if ( $statusChanged ) { |
128 | 128 | $this->mRev->emailNotifyUsersOfChanges( 'codereview-email-subj3', 'codereview-email-body3', |
129 | | - $wgUser->getName(), $this->mRev->getIdStringUnique(), $this->mRev->getOldStatus(), $this->mRev->getStatus() |
| 129 | + $wgUser->getName(), $this->mRev->getIdStringUnique(), $this->mRev->getOldStatus(), |
| 130 | + $this->mRev->getStatus(), $url |
130 | 131 | ); |
131 | 132 | } else if ( $commentAdded ) { |
132 | | - $url = $this->mRev->getFullUrl( $commentId ); |
133 | 133 | $this->mRev->emailNotifyUsersOfChanges( 'codereview-email-subj', 'codereview-email-body', |
134 | 134 | $wgUser->getName(), $url, $this->mRev->getIdStringUnique(), $this->text |
135 | 135 | ); |