Issue links for JIRA are broken when using the %URL% token
----------------------------------------------------------

                 Key: MCHANGES-186
                 URL: http://jira.codehaus.org/browse/MCHANGES-186
             Project: Maven 2.x Changes Plugin
          Issue Type: Bug
          Components: changes-report
    Affects Versions: 2.2
            Reporter: Dennis Lundberg


The fix for MCHANGES-166 included a small change that broke this. See 
http://jira.codehaus.org/browse/MCHANGES-166?focusedCommentId=184918&page=com.atlassian.jira.plugin.system.issuetabpanels%3Acomment-tabpanel#action_184918

{quote}
The private method parseIssueLink(...) should remove only the trailing slash 
from the URL if one exists. This prevents cases where we miss to give a 
trailing slash in the issueManagement.url.
{quote}

This part of the patch should not have been committed. The documentation for 
the issueLinkTemplatePerSystem parameter states:
{quote}
%URL%: this is computed by getting the <issueManagement>/<url> value from the 
POM, and removing the last '/' and everything that comes after it.
{quote}

After the patch this is no longer true. This breaks issue links to JIRA if you 
have a configuration like this:

{code:xml}
  <issueManagement>
    <system>JIRA</system>
    <url>http://jira.mycompany.com/browse/PRODUCT</url>
  </issueManagement>

  <reporting>
    <plugins>
      <plugin>
        <groupId>org.apache.maven.plugins</groupId>
        <artifactId>maven-changes-plugin</artifactId>
        <version>2.2</version>
        <configuration>
          <issueLinkTemplatePerSystem>
            <default>%URL%/%ISSUE%</default>
          </issueLinkTemplatePerSystem>
        </configuration>
      </plugin>
    </plugins>
  </reporting>
{code}

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: 
http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to