(refs #171)Fix link target of pull request number.

This commit is contained in:
takezoe
2013-10-25 04:28:58 +09:00
parent d1f0d01ae8
commit 1e750f4b9d

View File

@@ -15,10 +15,12 @@ trait LinkConverter { self: RequestCache =>
.replaceAll("&", "&amp;").replaceAll("<", "&lt;").replaceAll(">", "&gt;").replaceAll("\"", "&quot;") .replaceAll("&", "&amp;").replaceAll("<", "&lt;").replaceAll(">", "&gt;").replaceAll("\"", "&quot;")
// convert issue id to link // convert issue id to link
.replaceBy(("(^|\\W)" + issueIdPrefix + "(\\d+)(\\W|$)").r){ m => .replaceBy(("(^|\\W)" + issueIdPrefix + "(\\d+)(\\W|$)").r){ m =>
if(getIssue(repository.owner, repository.name, m.group(2)).isDefined){ getIssue(repository.owner, repository.name, m.group(2)) match {
Some(s"""${m.group(1)}<a href="${context.path}/${repository.owner}/${repository.name}/issues/${m.group(2)}">#${m.group(2)}</a>${m.group(3)}""") case Some(issue) if(issue.isPullRequest)
} else { => Some(s"""${m.group(1)}<a href="${context.path}/${repository.owner}/${repository.name}/pull/${m.group(2)}">#${m.group(2)}</a>${m.group(3)}""")
Some(s"""${m.group(1)}#${m.group(2)}${m.group(3)}""") case Some(_) => Some(s"""${m.group(1)}<a href="${context.path}/${repository.owner}/${repository.name}/issues/${m.group(2)}">#${m.group(2)}</a>${m.group(3)}""")
case None => Some(s"""${m.group(1)}#${m.group(2)}${m.group(3)}""")
} }
} }
// convert @username to link // convert @username to link