[FIXED JENKINS-38861] Do not warn of duplicate IDs when updating cred…

…entials
jenkinsci · Oct 10, 2016 · 62a1e11 · 62a1e11
1 parent 7431152
commit 62a1e11
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/...s/com/cloudbees/plugins/credentials/impl/BaseStandardCredentials/id-and-description.jelly b/...s/com/cloudbees/plugins/credentials/impl/BaseStandardCredentials/id-and-description.jelly
@@ -27,10 +27,11 @@
   <!-- When instance != null, we must not have checkUrl, since doCheckId has no way of knowing that this is *supposed* to exist already. -->
   <!-- But there appears to be no way to turn off checkUrl when field is specified. -->
   <!-- So the only apparent workaround is to disable field when instance != null, and code name and value manually. -->
-  <f:entry field="${instance != null ? null : 'id'}" title="${%ID}">
-    <f:textbox name="_.id" value="${instance != null ? instance.id : null}"
+  <f:entry field="${instance != null ? null : 'id'}" title="${%ID}" help="${descriptor.getHelpFile('id')}">
+    <f:textbox name="_.id"
+               value="${instance != null ? instance.id : null}"
                readonly="${instance != null ? 'readonly' : null}"
-               checkUrl="${descriptor.getCheckUrl('id')}"
+               checkUrl="${instance != null ? null : descriptor.getCheckUrl('id')}"
     />
   </f:entry>
   <f:entry title="${%Description}" field="description">