Skip to main content
summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
-rw-r--r--org.eclipse.mylyn.trac.tests/src/org/eclipse/mylyn/trac/tests/ui/TracHyperlinkUtilTest.java3
-rw-r--r--org.eclipse.mylyn.trac.ui/src/org/eclipse/mylyn/internal/trac/ui/TracHyperlinkUtil.java3
2 files changed, 5 insertions, 1 deletions
diff --git a/org.eclipse.mylyn.trac.tests/src/org/eclipse/mylyn/trac/tests/ui/TracHyperlinkUtilTest.java b/org.eclipse.mylyn.trac.tests/src/org/eclipse/mylyn/trac/tests/ui/TracHyperlinkUtilTest.java
index 87c0db555..39fe79a62 100644
--- a/org.eclipse.mylyn.trac.tests/src/org/eclipse/mylyn/trac/tests/ui/TracHyperlinkUtilTest.java
+++ b/org.eclipse.mylyn.trac.tests/src/org/eclipse/mylyn/trac/tests/ui/TracHyperlinkUtilTest.java
@@ -101,6 +101,9 @@ public class TracHyperlinkUtilTest extends TestCase {
links = findTracHyperlinks(repository, "{{abc}}", -1, 0);
assertNull(links);
+
+ links = findTracHyperlinks(repository, "{{{123}}}", -1, 0);
+ assertNull(links);
}
public void testFindHyperlinksChangeset() {
diff --git a/org.eclipse.mylyn.trac.ui/src/org/eclipse/mylyn/internal/trac/ui/TracHyperlinkUtil.java b/org.eclipse.mylyn.trac.ui/src/org/eclipse/mylyn/internal/trac/ui/TracHyperlinkUtil.java
index 57568e65e..bfb6a4416 100644
--- a/org.eclipse.mylyn.trac.ui/src/org/eclipse/mylyn/internal/trac/ui/TracHyperlinkUtil.java
+++ b/org.eclipse.mylyn.trac.ui/src/org/eclipse/mylyn/internal/trac/ui/TracHyperlinkUtil.java
@@ -33,6 +33,7 @@ import org.eclipse.mylyn.tasks.ui.TaskHyperlink;
* Utility class for detecting Trac hyperlinks.
*
* @author Steffen Pingel
+ * @author David Green partial fix hyperlink detection on {{{ see bug 265682
*/
public class TracHyperlinkUtil {
@@ -42,7 +43,7 @@ public class TracHyperlinkUtil {
static Pattern reportPattern1 = createPattern("report:(\\d+)"); //$NON-NLS-1$
- static Pattern reportPattern2 = createPattern("\\{(\\d+)\\}"); //$NON-NLS-1$
+ static Pattern reportPattern2 = Pattern.compile("(?<!!|\\{\\{)\\{(\\d+)\\}"); //$NON-NLS-1$
static Pattern changesetPattern1 = createPattern("(r|changeset:)(\\d+)(/\\w+)?"); //$NON-NLS-1$

Back to the top