Uploaded image for project: 'Maven Changes Plugin'
  1. Maven Changes Plugin
  2. MCHANGES-186

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

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Closed
    • Major
    • Resolution: Fixed
    • 2.2
    • 2.3
    • changes.xml
    • None

    Description

      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

      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.

      This part of the patch should not have been committed. The documentation for the issueLinkTemplatePerSystem parameter states:

      %URL%: this is computed by getting the <issueManagement>/<url> value from the POM, and removing the last '/' and everything that comes after it.

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

        <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>
      

      Attachments

        Activity

          People

            dennisl@apache.org Dennis Lundberg
            dennisl@apache.org Dennis Lundberg
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: