| Index: trunk/extensions/CodeReview/CodeReview.php |
| — | — | @@ -228,6 +228,8 @@ |
| 229 | 229 | * Any base paths matching regular expressions in these arrays will have their |
| 230 | 230 | * default status set to deferred instead of new. Helpful if you've got a part |
| 231 | 231 | * of the repository you don't care about. |
| | 232 | + * |
| | 233 | + * $wgCodeReviewDeferredPaths = array( 'RepoName' => array( '/path/to/use', '/another/path/to/use' ) ) |
| 232 | 234 | */ |
| 233 | 235 | $wgCodeReviewDeferredPaths = array(); |
| 234 | 236 | |
| Index: trunk/extensions/CodeReview/ui/CodeRepoStatsView.php |
| — | — | @@ -30,7 +30,7 @@ |
| 31 | 31 | foreach ( CodeRevision::getPossibleStates() as $state ) { |
| 32 | 32 | $count = isset( $stats->states[$state] ) ? $stats->states[$state] : 0; |
| 33 | 33 | $count = htmlspecialchars( $wgLang->formatNum( $count ) ); |
| 34 | | - $link = $this->skin->link( |
| | 34 | + $link = Linker::link( |
| 35 | 35 | SpecialPage::getTitleFor( 'Code', $repoName . '/status/' . $state ), |
| 36 | 36 | htmlspecialchars( $this->statusDesc( $state ) ) |
| 37 | 37 | ); |
| — | — | @@ -49,21 +49,26 @@ |
| 50 | 50 | } |
| 51 | 51 | |
| 52 | 52 | if ( !empty( $stats->fixmesPerPath ) ) { |
| 53 | | - $wgOut->wrapWikiMsg( "<h3 id=\"stats-fixme-path\">$1</h3>", 'code-stats-fixme-breakdown-path' ); |
| 54 | | - |
| 55 | | - foreach ( $stats->fixmesPerPath as $path => $fixmes ) { |
| 56 | | - $wgOut->wrapWikiMsg( "<h4 id=\"stats-fixme-path\">$1</h4>", array( 'code-stats-fixme-path', $path ) ); |
| 57 | | - $this->writeAuthorTable( 'fixme', $fixmes, array( 'path' => $path ) ); |
| 58 | | - } |
| | 53 | + $this->writeStatusPathTable( 'fixme', $stats->fixmesPerPath ); |
| 59 | 54 | } |
| 60 | 55 | |
| 61 | 56 | if ( !empty( $stats->newPerPath ) ) { |
| 62 | | - $wgOut->wrapWikiMsg( "<h3 id=\"stats-new-path\">$1</h3>", 'code-stats-new-breakdown-path' ); |
| | 57 | + $this->writeStatusPathTable( 'new', $stats->newPerPath ); |
| | 58 | + } |
| | 59 | + } |
| 63 | 60 | |
| 64 | | - foreach ( $stats->newPerPath as $path => $news ) { |
| 65 | | - $wgOut->wrapWikiMsg( "<h4 id=\"stats-new-path\">$1</h4>", array( 'code-stats-new-path', $path ) ); |
| 66 | | - $this->writeAuthorTable( 'fixme', $news, array( 'path' => $path ) ); |
| 67 | | - } |
| | 61 | + /** |
| | 62 | + * @param $status string |
| | 63 | + * @param $array array |
| | 64 | + */ |
| | 65 | + function writeStatusPathTable( $status, $array ) { |
| | 66 | + global $wgOut; |
| | 67 | + |
| | 68 | + $wgOut->wrapWikiMsg( "<h3 id=\"stats-$status-path\">$1</h3>", "code-stats-$status-breakdown-path" ); |
| | 69 | + |
| | 70 | + foreach ( $array as $path => $news ) { |
| | 71 | + $wgOut->wrapWikiMsg( "<h4 id=\"stats-$status-path\">$1</h4>", array( "code-stats-$status-path", $path ) ); |
| | 72 | + $this->writeAuthorTable( $status, $news, array( 'path' => $path ) ); |
| 68 | 73 | } |
| 69 | 74 | } |
| 70 | 75 | |
| — | — | @@ -93,7 +98,7 @@ |
| 94 | 99 | |
| 95 | 100 | foreach ( $array as $user => $count ) { |
| 96 | 101 | $count = htmlspecialchars( $wgLang->formatNum( $count ) ); |
| 97 | | - $link = $this->skin->link( |
| | 102 | + $link = Linker::link( |
| 98 | 103 | $title, |
| 99 | 104 | htmlspecialchars( $user ), |
| 100 | 105 | array(), |