From 5f0feaf606eb3d985598e198bf765aface30a820 Mon Sep 17 00:00:00 2001
From: aheritier <aheritier@13f79535-47bb-0310-9956-ffa450edef68>
Date: Thu, 19 Oct 2006 15:38:31 +0000
Subject: [PATCH] Add links to issues for the next-releases report.

git-svn-id: https://svn.apache.org/repos/asf/maven/maven-1/plugins/trunk@465663 13f79535-47bb-0310-9956-ffa450edef68
---
 multichanges/plugin.jelly                           |  6 ++++++
 multichanges/project.xml                            |  8 ++++++++
 multichanges/src/plugin-resources/next-releases.jsl | 13 ++++++++-----
 3 files changed, 22 insertions(+), 5 deletions(-)

diff --git a/multichanges/plugin.jelly b/multichanges/plugin.jelly
index fe955e78..aec0894e 100644
--- a/multichanges/plugin.jelly
+++ b/multichanges/plugin.jelly
@@ -93,6 +93,12 @@
             <x:attribute name="url">
               ${reactorProject.url}
             </x:attribute>
+            <x:attribute name="issueTrackingUrl">
+              ${reactorProject.issueTrackingUrl}
+            </x:attribute>
+            <x:attribute name="issueTrackingTemplate">
+              ${reactorProject.context.getVariable('maven.changes.issue.template')}
+            </x:attribute>
             <util:file var="changesAsFile" 
                 name="${reactorProject.context.getVariable('maven.docs.src')}/changes.xml"/>
             <j:choose>
diff --git a/multichanges/project.xml b/multichanges/project.xml
index 445dcd4b..cd09e73b 100644
--- a/multichanges/project.xml
+++ b/multichanges/project.xml
@@ -65,6 +65,14 @@
     </developer>
   </developers>
   <dependencies>
+    <dependency>
+      <groupId>maven</groupId>
+      <artifactId>maven-changes-plugin</artifactId>
+      <version>1.7-SNAPSHOT</version>
+      <properties>
+        <comment>To allow us to use org.apache.maven.changes.IssueFinder.</comment>
+      </properties>
+    </dependency>
     <dependency>
       <groupId>commons-logging</groupId>
       <artifactId>commons-logging</artifactId>
diff --git a/multichanges/src/plugin-resources/next-releases.jsl b/multichanges/src/plugin-resources/next-releases.jsl
index 1da3503a..7b4fa25c 100644
--- a/multichanges/src/plugin-resources/next-releases.jsl
+++ b/multichanges/src/plugin-resources/next-releases.jsl
@@ -77,6 +77,8 @@
       <maven:get var="maven.multiproject.navigation" plugin="maven-multiproject-plugin" property="maven.multiproject.navigation"/>
       <j:set var="navType" value="${maven.multiproject.navigation}"/>            
       <j:set var="artifactId"><x:expr select="@artifactId"/></j:set>
+      <j:set var="issueTrackingUrl"><x:expr select="@issueTrackingUrl"/></j:set>
+      <j:set var="issueTrackingTemplate"><x:expr select="@issueTrackingTemplate"/></j:set>
       <j:choose>
         <j:when test="${navType eq 'aggregate'}">
           <j:set var="baseUrl" value="${maven.multiproject.aggregateDir}${artifactId}"/>
@@ -102,7 +104,9 @@
       <j:set var="issue"><x:expr select="@issue"/></j:set>
       <j:set var="dueto"><x:expr select="@due-to"/></j:set>
       <j:set var="duetoemail"><x:expr select="@due-to-email"/></j:set>
-       <!-- FIXME: Same hack as above -->
+      <!-- FIXME: Big hack here. Jelly looses the reference to the 
+           current node so we have to explicitely remember it for 
+           Jelly! -->
       <x:set var="currentAction" select="."/>
       <tr>
         <td><img src="images/${type}.gif" alt="${type}" title="${type}"/></td>
@@ -114,15 +118,14 @@
               <jsl:applyTemplates trim="false"/>
             </x:if>
 
-            <j:set var="template" value="${maven.changes.issue.template}"/>
-            <j:set var="trackerURL" value="${pom.issueTrackingUrl}"/>
+            <j:useBean var="finder" class="org.apache.maven.changes.IssueFinder"/>
             <x:if select="$currentAction/fixes">
               <j:set var="fixesTotal"><x:expr select="count(./fixes)"/></j:set>
               <j:set var="fixesCount" value="1"/>
               Fixes 
               <x:forEach var="item" select="./fixes">
                 <j:set var="issue"><x:expr select="@issue"/></j:set>
-                ${issue}
+                <a href="${finder.getIssueURL(issueTrackingUrl,issue,issueTrackingTemplate)}">${issue}</a>
                 <j:choose>
                   <j:when test="${fixesCount == fixesTotal}">.</j:when>
                   <j:when test="${fixesCount == fixesTotal-1}"> and </j:when>
@@ -133,7 +136,7 @@
             </x:if>
             <x:if select="not($currentAction/fixes)">
               <j:if test="${!empty(issue)}">
-                Fixes ${issue}.
+                Fixes <a href="${finder.getIssueURL(issueTrackingUrl,issue,issueTrackingTemplate)}">${issue}</a>.
               </j:if>
             </x:if>