[JENKINS-30774] Repository validation has been simplified

jenkinsci · Nov 2, 2015 · 240108e · 240108e
1 parent 8a76821
commit 240108e
Showing 1 changed file with 9 additions and 16 deletions.
diff --git a/src/main/java/hudson/scm/SubversionSCM.java b/src/main/java/hudson/scm/SubversionSCM.java
@@ -3075,22 +3075,22 @@ public FormValidation doCheckCredentialsId(StaplerRequest req, @AncestorInPath I
              * Validate the value for a remote (repository) location.
              */
             public FormValidation checkCredentialsId(StaplerRequest req, @Nonnull Item context, String remote, String value) {
-                // if check remote is reporting an issue then we don't need to
                 String url = Util.fixEmptyAndTrim(remote);
                 if (url == null) {
                     return FormValidation.ok();
                 }
 
-                if (descriptor().isValidateRemoteUpToVar()) {
-                    url = (url.indexOf('$') != -1) ? url.substring(0, url.indexOf('$')) : url;
-                } else {
-                    url = new EnvVars(EnvVars.masterEnvVars).expand(url);
-                }
-
+                // TODO: It would be interesting to include global variables.
+                url = new EnvVars(EnvVars.masterEnvVars).expand(url);
                 if (!URL_PATTERN.matcher(url).matches()) {
                     return FormValidation.ok();
                 }
 
+                // Is the URL repository parameterized?
+                if (url.indexOf('$') != -1) {
+                    return FormValidation.warning("This repository has a parameterized URL, validation skipped.");
+                }
+
                 try {
                     String urlWithoutRevision = SvnHelper.getUrlWithoutRevision(url);
 
@@ -3147,15 +3147,8 @@ public FormValidation checkCredentialsId(StaplerRequest req, @Nonnull Item conte
                         }
                     }
                 } catch (SVNException e) {
-                    LOGGER.log(Level.INFO, "Failed to access subversion repository: " + url, e);
-                    String message = Messages.SubversionSCM_doCheckRemote_exceptionMsg1(
-                            Util.escape(url), Util.escape(e.getErrorMessage().getFullMessage()),
-                            "javascript:document.getElementById('svnerror').style.display='block';"
-                                    + "document.getElementById('svnerrorlink').style.display='none';"
-                                    + "return false;")
-                            + "<br/><pre id=\"svnerror\" style=\"display:none\">"
-                            + Functions.printThrowable(e) + "</pre>";
-                    return FormValidation.errorWithMarkup(message);
+                    LOGGER.log(Level.WARNING, "Unable to access to subversion repository. " + e.getMessage());
+                    return FormValidation.warning("Unable to access to subversion repository");
                 }
             }