From a0457e79aca2d34ae350ff3978c435cbe009c7c5 Mon Sep 17 00:00:00 2001
From: vmassol <vmassol@13f79535-47bb-0310-9956-ffa450edef68>
Date: Mon, 28 Jun 2004 21:04:04 +0000
Subject: [PATCH] Added first cut of a checkstyle RSS feed. There's lot to
 improve but I'd like to gather some feedback before working more on this
 feature. It relies on the fact that RSS readers cache feed data previously
 read thus keeping an history of the feed content over time.

git-svn-id: https://svn.apache.org/repos/asf/maven/maven-1/plugins/trunk@115584 13f79535-47bb-0310-9956-ffa450edef68
---
 checkstyle/plugin.jelly                       |  15 ++-
 checkstyle/project.xml                        |   7 +-
 .../src/plugin-resources/checkstyle.jsl       |   3 +-
 .../src/plugin-resources/checkstyle2rss.jsl   | 110 ++++++++++++++++++
 .../src/plugin-resources/images/rss.png       | Bin 0 -> 360 bytes
 checkstyle/xdocs/changes.xml                  |   5 +-
 6 files changed, 136 insertions(+), 4 deletions(-)
 create mode 100644 checkstyle/src/plugin-resources/checkstyle2rss.jsl
 create mode 100644 checkstyle/src/plugin-resources/images/rss.png

diff --git a/checkstyle/plugin.jelly b/checkstyle/plugin.jelly
index a25c0a8c..d517d66c 100644
--- a/checkstyle/plugin.jelly
+++ b/checkstyle/plugin.jelly
@@ -1,4 +1,5 @@
 <?xml version="1.0"?>
+
 <!-- 
 /*
  * Copyright 2001-2004 The Apache Software Foundation.
@@ -17,7 +18,6 @@
  */
  -->
 
-
 <!--
   =============================================================================
     Checkstyle plugin for Maven.
@@ -195,6 +195,19 @@
       outputMode="xml"
       prettyPrint="true"/>
 
+    <!-- Generate a RSS feed of the checktyle errors -->
+    <doc:jsl
+      input="${maven.build.dir}/checkstyle-raw-report.xml"
+      output="checkstyle.rss"
+      stylesheet="${plugin.resources}/checkstyle2rss.jsl"
+      encoding="${maven.docs.outputencoding}"
+      outputMode="xml"
+      prettyPrint="true"/>
+
+    <!-- Copy the RSS XML logo -->
+    <mkdir dir="${maven.docs.dest}"/>
+    <copy todir="${maven.docs.dest}/images" file="${plugin.resources}/images/rss.png"/>
+
   </goal>
 
   <!--
diff --git a/checkstyle/project.xml b/checkstyle/project.xml
index 19954726..fa91cd60 100644
--- a/checkstyle/project.xml
+++ b/checkstyle/project.xml
@@ -23,7 +23,7 @@
   <pomVersion>3</pomVersion>
   <id>maven-checkstyle-plugin</id>
   <name>Maven Checkstyle Plugin</name>
-  <currentVersion>2.4.2-SNAPSHOT</currentVersion>
+  <currentVersion>2.5-SNAPSHOT</currentVersion>
   <description/>
   <shortDescription>Produce Checkstyle report</shortDescription>
   <url>http://maven.apache.org/reference/plugins/checkstyle/</url>
@@ -65,6 +65,11 @@
       <name>2.4.1</name>
       <tag>MAVEN_CHECKSTYLE_2_4_1</tag>
     </version>
+    <version>
+      <id>2.5</id>
+      <name>2.5</name>
+      <tag>HEAD</tag>
+    </version>
   </versions>
   <developers>
     <developer>
diff --git a/checkstyle/src/plugin-resources/checkstyle.jsl b/checkstyle/src/plugin-resources/checkstyle.jsl
index a46409e2..a5120909 100644
--- a/checkstyle/src/plugin-resources/checkstyle.jsl
+++ b/checkstyle/src/plugin-resources/checkstyle.jsl
@@ -46,7 +46,8 @@
         <section name="Checkstyle Results">
           <p>
             The following document contains the results of
-            <a href="http://checkstyle.sourceforge.net/">Checkstyle</a>.
+            <a href="http://checkstyle.sourceforge.net/">Checkstyle</a>
+            <a href="checkstyle.rss"><img src="images/rss.png"/></a>.
           </p>
         </section>
 
diff --git a/checkstyle/src/plugin-resources/checkstyle2rss.jsl b/checkstyle/src/plugin-resources/checkstyle2rss.jsl
new file mode 100644
index 00000000..8d80ea3a
--- /dev/null
+++ b/checkstyle/src/plugin-resources/checkstyle2rss.jsl
@@ -0,0 +1,110 @@
+<?xml version="1.0" encoding="ISO-8859-1"?>
+
+<!--
+ * ========================================================================
+ * 
+ * Copyright 2004 The Apache Software Foundation.
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ * 
+ *   http://www.apache.org/licenses/LICENSE-2.0
+ * 
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ * 
+ * ========================================================================
+-->
+
+<jsl:stylesheet
+  select="$doc"
+  xmlns:j="jelly:core"
+  xmlns:jsl="jelly:jsl"
+  xmlns:util="jelly:util"
+  xmlns:x="jelly:xml"
+  xmlns:doc="doc"
+  xmlns="dummy" trim="true">
+
+  <!-- This needs to be instantiated here to be available in the template matches -->
+  <j:useBean var="mavenTool" class="org.apache.maven.util.MavenTool"/>
+  <j:useBean var="htmlescape" class="org.apache.velocity.anakia.Escape"/>
+  <j:useBean var="fileutil" class="org.apache.velocity.texen.util.FileUtil"/>
+  <j:useBean var="pathtool" class="org.apache.maven.util.DVSLPathTool"/>
+
+  <jsl:template match="checkstyle">
+
+    <rss version="0.91">
+      <channel>
+        <title>Checkstyle report</title> 
+        <link>${pom.url}</link> 
+        <description>Checkstyle report</description>
+        <language>en-us</language>
+        <copyright>Copyright 2004 The Apache Software Foundation</copyright>
+        <item>
+
+          <j:set var="fileCount"><x:expr select="count(file)"/></j:set>
+          <j:set var="infoCount"><x:expr select="count(file/error[@severity='info'])"/></j:set>
+          <j:set var="warningCount"><x:expr select="count(file/error[@severity='warning'])"/></j:set>
+          <j:set var="errorCount"><x:expr select="count(file/error[@severity='error'])"/></j:set>        
+
+          <title>File: ${fileCount}, Errors: ${errorCount}, Warnings: ${warningCount}, Infos: ${infoCount}</title>
+          
+          <!-- If we don't go through a tmp variable Jelly inserts a space between ${pom.url} 
+               and '/checkstyle-report.html'. Go figure... -->
+          <j:set var="link" value="${pom.url}/checkstyle-report.html"/>
+          
+          <link>${link}</link> 
+          <description>
+            <p>Click <a href="${pom.url}/checkstyle-report.html">here</a> for the full Checkstyle report.</p>
+            
+          <table summary="Files" border="1">
+            <thead>
+              <tr>
+                <th>Files</th>
+                <th style="width:30px">I</th>
+                <th style="width:30px">W</th>
+                <th style="width:30px">E</th>
+              </tr>
+            </thead>
+            <tbody>
+              <j:set var="fullSrcDir" value="${pom.build.sourceDirectory}"/>
+              <j:set var="srcDir" value="${fileutil.file(fullSrcDir).getCanonicalPath()}"/>
+              <j:set var="srcDirLength" value="${srcDir.length() + 1}"/>
+              <x:set var="files" select="file" sort="@name"/>
+              <!-- x:forEach is busted -->
+              <j:forEach var="file" items="${files}">
+                <!-- Type coercion doesn't work worth a fuck in jexl. -->
+                <j:set var="name" value="${file.attribute('name').getValue()}"/>
+                <j:set var="name" value="${name.substring(mavenTool.toInteger(srcDirLength.toString()))}"/>
+                <util:replace var="name" value="${name}" oldChar="\\" newChar="/"/>
+                <!--- +1 is for the trailing slash above -->
+                <j:set var="infoCount"><x:expr select="count($file/error[@severity='info'])"/></j:set>
+                <j:set var="warningCount"><x:expr select="count($file/error[@severity='warning'])"/></j:set>
+                <j:set var="errorCount"><x:expr select="count($file/error[@severity='error'])"/></j:set>
+                
+                <j:if test="${errorCount + warningCount + infoCount != 0}">
+                  <tr>
+                    <td>
+                      <a href="${link}#${name}">${name}</a>
+                    </td>
+                    <td><doc:formatAsNumber string="${infoCount}" pattern="0"/></td>
+                    <td><doc:formatAsNumber string="${warningCount}" pattern="0"/></td>
+                    <td><doc:formatAsNumber string="${errorCount}" pattern="0"/></td>
+                  </tr>
+                </j:if>
+              </j:forEach>
+            </tbody>
+          </table>
+            
+          </description> 
+        </item>
+      </channel>
+    </rss>
+
+  </jsl:template>
+
+</jsl:stylesheet>
diff --git a/checkstyle/src/plugin-resources/images/rss.png b/checkstyle/src/plugin-resources/images/rss.png
new file mode 100644
index 0000000000000000000000000000000000000000..f0796ac886201eb43b7a15113e2deb1b32fa9a09
GIT binary patch
literal 360
zcmV-u0hj)XP)<h;3K|Lk000e1NJLTq0012T000jN0{{R3FH<zF0002JP)t-s|NsA5
zgt8Pzisb9`*5T}pjg2Q(lu=PpGh&>1pTfh($;sL0vc}u{{{FPIw4<Y<^7i`p`uw@g
z;qv(YgQCcxxzk@?Uh?wt-QC?gXrM}SsCJmYV~Muv>gub**+Xunj;hMg-Rk@N{o&!^
z{r&#_{{B{et%#+@{r~@)vdh1}zle=^0GR*(m;g35HkdZcHkixX<?qkX&=Nj{v97I-
zu+NmP%;@m;`uh5wy4BRw)YsV9X^^`AW&mbpW_Z=8>Hq)$8FWQhbW?9;ba!ELWdKKV
zVQ^?^V?%Flc4cyOWpXuLNbUdt06|GaK~#90Vz9#p80;8u1CXpOE<hDo1_(x?Q1miD
zWEemQQ`VLNrUs8*>?Whx%fLW2AK{k88_xIu00030{{sMfLr(Xb|5zme0000<MNUMn
GLSTXjIl1Tn

literal 0
HcmV?d00001

diff --git a/checkstyle/xdocs/changes.xml b/checkstyle/xdocs/changes.xml
index 64c39485..a9e9ef84 100644
--- a/checkstyle/xdocs/changes.xml
+++ b/checkstyle/xdocs/changes.xml
@@ -25,7 +25,10 @@
     <author email="evenisse@ifrance.com">Emmanuel Venisse</author>
   </properties>
   <body> 
-    <release version="2.4.2-SNAPSHOT" date="In CVS">
+    <release version="2.5-SNAPSHOT" date="In CVS">
+      <action dev="vmassol" type="add">
+        Added generation of a Checkstyle RSS feed.
+      </action>
       <action dev="aheritier" type="fix" issue="MPCHECKSTYLE-21">Fix : Build failed if target directory and License file didn't exist.</action>
     </release>  
     <release version="2.4.1" date="2004-05-18">