[JENKINS-21818] Merge pull request #1131

jenkinsci · Apr 13, 2014 · 998d102 · 998d102
2 parents 6699644 + a42ebcc
commit 998d102
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 8 deletions.
diff --git a/changelog.html b/changelog.html
@@ -83,6 +83,9 @@
     (<a href="https://issues.jenkins-ci.org/browse/JENKINS-21394">issue 21394</a>)
   <li class=rfe>
     Debian package now creates 'jenkins' group
+  <li class=rfe>
+    Suppress fingerprint link if fingerprint record isn't available.
+    (<a href="https://issues.jenkins-ci.org/browse/JENKINS-21818">issue 21818</a>)
 </ul>
 </div><!--=TRUNK-END=-->
 

diff --git a/core/src/main/resources/hudson/model/DirectoryBrowserSupport/dir.jelly b/core/src/main/resources/hudson/model/DirectoryBrowserSupport/dir.jelly
@@ -75,10 +75,12 @@ THE SOFTWARE.
                     </td>
                     <td>
                       <j:if test="${x.readable}">
-                        <a href="${x.href}/*fingerprint*/">
-                          <img src="${imagesURL}/16x16/fingerprint.png" alt="fingerprint" height="16" width="16" />
-                        </a>
-                        <st:nbsp/>
+                        <j:if test="${app.fingerprintMap.ready}">
+                          <a href="${x.href}/*fingerprint*/">
+                            <img src="${imagesURL}/16x16/fingerprint.png" alt="fingerprint" height="16" width="16" />
+                          </a>
+                          <st:nbsp/>
+                        </j:if>
                         <a href="${x.href}/*view*/">${%view}</a>
                       </j:if>                      
                     </td>

diff --git a/core/src/main/resources/lib/hudson/artifactList.jelly b/core/src/main/resources/lib/hudson/artifactList.jelly
@@ -60,10 +60,12 @@ THE SOFTWARE.
                     ${h.humanReadableByteSize(f.getFileSize())}
                   </td>
                   <td>
-                    <a href="${baseURL}artifact/${f.href}/*fingerprint*/">
-                      <img src="${imagesURL}/16x16/fingerprint.png" alt="[fingerprint]" height="16" width="16"/>
-                    </a>
-                    <st:nbsp/>
+                    <j:if test="${app.fingerprintMap.ready}">
+                      <a href="${baseURL}artifact/${f.href}/*fingerprint*/">
+                        <img src="${imagesURL}/16x16/fingerprint.png" alt="[fingerprint]" height="16" width="16"/>
+                      </a>
+                      <st:nbsp/>
+                    </j:if>
                     <a href="${baseURL}artifact/${f.href}/*view*/">${%view}</a>
                   </td>
                 </tr>