| Index: trunk/extensions/CodeReview/CodeReleaseNotes.php |
| — | — | @@ -133,7 +133,7 @@ |
| 134 | 134 | |
| 135 | 135 | // Quick relevance tests (these *should* be over-inclusive a little if anything) |
| 136 | 136 | private function isRelevant( $summary ) { |
| 137 | | - # Fixed a bug? Mentioned a config var? Mentioned a rev? |
| | 137 | + # Fixed a bug? Mentioned a config var? |
| 138 | 138 | if( preg_match( '/\b(bug #?(\d+)|\$[we]g[0-9a-z]{3,50})\b/i', $summary ) ) |
| 139 | 139 | return true; |
| 140 | 140 | # Sanity check: summary cannot be *too* short to be useful |
| — | — | @@ -148,9 +148,6 @@ |
| 149 | 149 | if( preg_match( '/\b(wiki|HTML\d|CSS\d|UTF-?8|(Apache|PHP|CGI|Java|Perl|Python|\w+SQL) ?\d?\.?\d?)\b/i', $summary ) ) |
| 150 | 150 | return true; |
| 151 | 151 | # Longish summary :) |
| 152 | | - if( $words > 35 ) |
| 153 | | - return true; |
| 154 | | - # Otherwise false |
| 155 | | - return false; |
| | 152 | + return ( $words > 35 ); |
| 156 | 153 | } |
| 157 | 154 | } |