[FIXED JENKINS-10556] Auto Install JDK asks for Oracle account, but t…

…he link goes 404
jenkinsci · Aug 2, 2011 · 6f335c3 · ndeloof · Dec 16, 2011 · 6f335c3
1 parent 3e52131
commit 6f335c3
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/changelog.html b/changelog.html
@@ -55,7 +55,9 @@
 <!-- Record your changes in the trunk here. -->
 <div id="trunk" style="display:none"><!--=TRUNK-BEGIN=-->
 <ul class=image>
-  <li class=>
+  <li class=bug>
+    Auto Install JDK asks for Oracle account, but the link goes 404.
+    (<a href="https://issues.jenkins-ci.org/browse/JENKINS-10556">issue 10556</a>)
 </ul>
 </div><!--=TRUNK-END=-->
 

diff --git a/core/src/main/java/hudson/tools/JDKInstaller.java b/core/src/main/java/hudson/tools/JDKInstaller.java
@@ -630,7 +630,7 @@ public List<JDKFamily> getInstallableJDKs() throws IOException {
 
         public FormValidation doCheckAcceptLicense(@QueryParameter boolean value) {
             if (username==null || password==null)
-                return FormValidation.errorWithMarkup(Messages.JDKInstaller_RequireOracleAccount(Stapler.getCurrentRequest().getContextPath()+getDescriptorUrl()+"/enterCredential"));
+                return FormValidation.errorWithMarkup(Messages.JDKInstaller_RequireOracleAccount(Stapler.getCurrentRequest().getContextPath()+'/'+getDescriptorUrl()+"/enterCredential"));
             if (value) {
                 return FormValidation.ok();
             } else {