[JENKINS-49586] - Update Parent POM and resolve reported issues

jenkinsci · Feb 16, 2018 · baad820 · baad820
1 parent 0c8fbfa
commit baad820
Show file tree

Hide file tree

Showing 7 changed files with 38 additions and 9 deletions.
diff --git a/pom.xml b/pom.xml
@@ -3,7 +3,7 @@
   <parent>
     <groupId>org.jenkins-ci.plugins</groupId>
     <artifactId>plugin</artifactId>
-    <version>1.532</version>
+    <version>3.4</version>
   </parent>
 
   <artifactId>jdepend</artifactId>
@@ -13,8 +13,15 @@
   <name>Jenkins JDepend Plugin</name>
   <description>A Jenkins plugin that uses JDepend to generate metrics.</description>
   <inceptionYear>2009</inceptionYear>
-  <url>http://wiki.jenkins-ci.org/display/JENKINS/JDepend+Plugin</url>
-
+  <url>https://wiki.jenkins.io/display/JENKINS/JDepend+Plugin</url>
+
+  <properties>
+    <jenkins.version>1.625.3</jenkins.version>
+    <java.level>7</java.level>
+    <!-- TODO: Delete once FindBugs issues are fixed (4 issues) -->
+    <findbugs.failOnError>false</findbugs.failOnError>
+  </properties>
+
   <licenses>
       <license>
           <name>BSD</name>
@@ -56,6 +63,13 @@
           <groupId>org.codehaus.mojo</groupId>
           <artifactId>jdepend-maven-plugin</artifactId>
           <version>2.0-beta-2</version>
+          <exclusions>
+            <exclusion>
+              <!-- Newer version comes from Jenkins Core's Acegi Security. No comments... -->
+              <groupId>oro</groupId>
+              <artifactId>oro</artifactId>
+            </exclusion>
+          </exclusions>
       </dependency>
       <dependency>
           <groupId>jdepend</groupId>
@@ -66,6 +80,18 @@
         <groupId>org.apache.maven.doxia</groupId>
         <artifactId>doxia-core</artifactId>
         <version>1.1.1</version>
+        <exclusions>
+          <exclusion>
+            <!-- Newer version comes from Jenkins Core. -->
+            <groupId>commons-lang</groupId>
+            <artifactId>commons-lang</artifactId>
+          </exclusion>
+        </exclusions>
+      </dependency>
+      <dependency> <!-- Upper bounds conflict between Docix Core and JTH -->
+        <groupId>xerces</groupId>
+        <artifactId>xercesImpl</artifactId>
+        <version>2.11.0</version>
       </dependency>
       <dependency>
         <groupId>org.apache.maven.doxia</groupId>
@@ -77,14 +103,14 @@
     <repositories>
         <repository>
             <id>repo.jenkins-ci.org</id>
-            <url>http://repo.jenkins-ci.org/public/</url>
+            <url>https://repo.jenkins-ci.org/public/</url>
         </repository>
     </repositories>
 
     <pluginRepositories>
         <pluginRepository>
             <id>repo.jenkins-ci.org</id>
-            <url>http://repo.jenkins-ci.org/public/</url>
+            <url>https://repo.jenkins-ci.org/public/</url>
         </pluginRepository>
     </pluginRepositories>
 

diff --git a/src/main/java/hudson/plugins/jdepend/JDependBuildAction.java b/src/main/java/hudson/plugins/jdepend/JDependBuildAction.java
@@ -61,7 +61,7 @@ public String getUrlName() {
 
 	/**
 	 * Get the HTML string of the JDepend report.
-	 * This report is HTML tidied, and had the <html><body> tags
+	 * This report is HTML tidied, and had the {@code <html><body>} tags
 	 * and such cruft removed.
 	 * 
 	 * @return JDepend HTML report

diff --git a/src/main/java/hudson/plugins/jdepend/JDependParser.java b/src/main/java/hudson/plugins/jdepend/JDependParser.java
@@ -39,7 +39,6 @@ public JDependParser(File xmlFile) throws ParserConfigurationException,
      * 
      * This would work better if it cached the result.
      * 
-     * @param packages
      * @return The total number of classes parsed.
      */
 	protected String getTotalClasses()

diff --git a/src/main/resources/hudson/plugins/jdepend/JDependBuildAction/index.jelly b/src/main/resources/hudson/plugins/jdepend/JDependBuildAction/index.jelly
@@ -1,3 +1,4 @@
+<?jelly escape-by-default='true'?>
 <j:jelly xmlns:j="jelly:core" xmlns:st="jelly:stapler" xmlns:d="jelly:define" xmlns:l="/lib/layout" xmlns:t="/lib/hudson" xmlns:f="/lib/form" xmlns:i="jelly:fmt" xmlns:local="local">
 
     <l:layout>
@@ -28,7 +29,7 @@
         <st:include it="${it.build}" page="sidepanel.jelly" />
 
         <l:main-panel>
-            ${it.getJDependHtml()}
+            <j:out value="${it.getJDependHtml()}"/>
         </l:main-panel>
     </l:layout>
 </j:jelly>
diff --git a/src/main/resources/hudson/plugins/jdepend/JDependProjectAction/index.jelly b/src/main/resources/hudson/plugins/jdepend/JDependProjectAction/index.jelly
@@ -1,9 +1,10 @@
+<?jelly escape-by-default='true'?>
 <j:jelly xmlns:j="jelly:core" xmlns:st="jelly:stapler" xmlns:d="jelly:define" xmlns:l="/lib/layout" xmlns:t="/lib/hudson" xmlns:f="/lib/form" xmlns:i="jelly:fmt" xmlns:local="local">
     <l:layout>
         <st:include it="${it.project}" page="sidepanel.jelly" />
 
         <l:main-panel>
-            ${it.getJDependHtml()}
+            <j:out value="${it.getJDependHtml()}"/>
         </l:main-panel>
     </l:layout>
 </j:jelly>
diff --git a/src/main/resources/hudson/plugins/jdepend/JDependRecorder/config.jelly b/src/main/resources/hudson/plugins/jdepend/JDependRecorder/config.jelly
@@ -1,3 +1,4 @@
+<?jelly escape-by-default='true'?>
 <j:jelly xmlns:j="jelly:core" xmlns:st="jelly:stapler" xmlns:d="jelly:define" xmlns:l="/lib/layout" xmlns:t="/lib/hudson" xmlns:f="/lib/form">
   <!--
     This jelly script is used for per-project configuration.

diff --git a/src/main/resources/index.jelly b/src/main/resources/index.jelly
@@ -1,3 +1,4 @@
+<?jelly escape-by-default='true'?>
 <div>
   This is a plugin that runs JDepend reports on builds.
 </div>