http://www.mediawiki.org/wiki/Special:Code/MediaWiki/82788
Revision: 82788
Author: reedy
Date: 2011-02-25 10:56:46 +0000 (Fri, 25 Feb 2011)
Log Message:
-----------
* (bug 27703) CR should parse 'bug#25848'
Made space optional between bug and it's number, possibly including a hash or
not
Modified Paths:
--------------
trunk/extensions/CodeReview/backend/CodeCommentLinker.php
trunk/extensions/CodeReview/backend/CodeRevision.php
trunk/extensions/CodeReview/ui/CodeReleaseNotes.php
Modified: trunk/extensions/CodeReview/backend/CodeCommentLinker.php
===================================================================
--- trunk/extensions/CodeReview/backend/CodeCommentLinker.php 2011-02-25
10:43:38 UTC (rev 82787)
+++ trunk/extensions/CodeReview/backend/CodeCommentLinker.php 2011-02-25
10:56:46 UTC (rev 82788)
@@ -25,7 +25,7 @@
array( $this, 'generalLink' ), $text );
$text = preg_replace_callback( '/\br(\d+)\b/',
array( $this, 'messageRevLink' ), $text );
- $text = preg_replace_callback( '/\bbug #?(\d+)\b/i',
+ $text = preg_replace_callback( '/\bbug ?#?(\d+)\b/i',
array( $this, 'messageBugLink' ), $text );
return $text;
}
Modified: trunk/extensions/CodeReview/backend/CodeRevision.php
===================================================================
--- trunk/extensions/CodeReview/backend/CodeRevision.php 2011-02-25
10:43:38 UTC (rev 82787)
+++ trunk/extensions/CodeReview/backend/CodeRevision.php 2011-02-25
10:56:46 UTC (rev 82788)
@@ -476,7 +476,7 @@
// Update bug references table...
$affectedBugs = array();
$m = array();
- if ( preg_match_all( '/\bbug #?(\d+)\b/', $this->message, $m )
) {
+ if ( preg_match_all( '/\bbug ?#?(\d+)\b/', $this->message, $m )
) {
$data = array();
foreach ( $m[1] as $bug ) {
$data[] = array(
Modified: trunk/extensions/CodeReview/ui/CodeReleaseNotes.php
===================================================================
--- trunk/extensions/CodeReview/ui/CodeReleaseNotes.php 2011-02-25 10:43:38 UTC
(rev 82787)
+++ trunk/extensions/CodeReview/ui/CodeReleaseNotes.php 2011-02-25 10:56:46 UTC
(rev 82788)
@@ -138,7 +138,7 @@
// Quick relevance tests (these *should* be over-inclusive a little if
anything)
private function isRelevant( $summary, $whole = true ) {
# Fixed a bug? Mentioned a config var?
- if ( preg_match( '/\b(bug #?(\d+)|\$[we]g[0-9a-z]{3,50})\b/i',
$summary ) ) {
+ if ( preg_match( '/\b(bug ?#?(\d+)|\$[we]g[0-9a-z]{3,50})\b/i',
$summary ) ) {
return true;
}
# Sanity check: summary cannot be *too* short to be useful
_______________________________________________
MediaWiki-CVS mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-cvs
