| Index: trunk/extensions/CodeReview/CodeReview.i18n.php |
| — | — | @@ -103,8 +103,8 @@ |
| 104 | 104 | |
| 105 | 105 | 'codereview-reply-link' => 'reply', |
| 106 | 106 | |
| 107 | | - 'codereview-tests-succeeded' => 'succeeded tests', |
| 108 | | - 'codereview-tests-failed' => 'failed tests:', |
| | 107 | + 'codereview-tests-succeeded2' => '$1 succeeded {{PLURAL:$2|test|tests}}', |
| | 108 | + 'codereview-tests-failed2' => '$1 succeeded {{PLURAL:$2|test|tests}}, $3 failed {{PLURAL:$4|test|tests}}:', |
| 109 | 109 | 'codereview-tests-running' => 'Test cases are running...', |
| 110 | 110 | 'codereview-tests-aborted' => 'Test run aborted.', |
| 111 | 111 | |
| Index: trunk/extensions/CodeReview/ui/CodeRevisionView.php |
| — | — | @@ -316,12 +316,19 @@ |
| 317 | 317 | $total = $run->countTotal; |
| 318 | 318 | $success = $run->countSuccess; |
| 319 | 319 | $failed = $total - $success; |
| | 320 | + $success_tests = "<span class='mw-codereview-success'>$success</span>"; |
| 320 | 321 | if( $failed ) { |
| 321 | | - $html .= "<p><span class='mw-codereview-success'>$success</span> ". |
| 322 | | - wfMsgHtml('codereview-tests-succeeded').", " . |
| 323 | | - "<span class='mw-codereview-fail'>$failed</span> ". |
| 324 | | - wfMsgHtml('codereview-tests-failed')."</p>"; |
| 325 | | - |
| | 322 | + $failed_tests = "<span class='mw-codereview-fail'>$failed</span>"; |
| | 323 | + $html .= "<p>" . |
| | 324 | + wfMsgExt( |
| | 325 | + 'codereview-tests-failed2', |
| | 326 | + 'parseinline', |
| | 327 | + $success_tests, |
| | 328 | + $success, |
| | 329 | + $failed_tests, |
| | 330 | + $failed |
| | 331 | + ) . "</p>"; |
| | 332 | + |
| 326 | 333 | $tests = $run->getResults( false ); |
| 327 | 334 | $html .= "<ul>\n"; |
| 328 | 335 | foreach( $tests as $test ) { |
| — | — | @@ -329,9 +336,13 @@ |
| 330 | 337 | } |
| 331 | 338 | $html .= "</ul>\n"; |
| 332 | 339 | } else { |
| 333 | | - $html .= "<p><span class='mw-codereview-success'>$success</span> ". |
| 334 | | - wfMsgHtml('codereview-tests-succeeded')."</p>"; |
| 335 | | - |
| | 340 | + $html .= "<p>" . |
| | 341 | + wfMsgExt( |
| | 342 | + 'codereview-tests-succeeded2', |
| | 343 | + 'parseinline', |
| | 344 | + $success_tests, |
| | 345 | + $success |
| | 346 | + ) . "</p>"; |
| 336 | 347 | } |
| 337 | 348 | } elseif( $run->status == "running" ) { |
| 338 | 349 | $html .= wfMsgExt('codereview-tests-running','parse'); |