# HG changeset patch
# User Marcin Kuzminski <marcin@rhodecode.com>
# Date 2018-06-05 14:09:55
# Node ID b79a1f785fc99888503ed59a3b5b4108da8e58a3
# Parent  3e9c7773e89151b822515d59008a42d83932ff33

validators: fixed problems with new mercurial/git url validators.

diff --git a/rhodecode/model/validation_schema/validators.py b/rhodecode/model/validation_schema/validators.py
--- a/rhodecode/model/validation_schema/validators.py
+++ b/rhodecode/model/validation_schema/validators.py
@@ -89,7 +89,7 @@ def url_validator(url, repo_type, config
             # initially check if it's at least the proper URL
             # or does it pass basic auth
 
-            MercurialRepository.check_url(url, config)
+            return MercurialRepository.check_url(url, config)
         elif 'svn+http' in url[:8]:  # svn->hg import
             SubversionRepository.check_url(url, config)
         elif 'git+http' in url[:8]:  # git->hg import
@@ -106,7 +106,7 @@ def url_validator(url, repo_type, config
         if 'http' in url[:4]:
             # initially check if it's at least the proper URL
             # or does it pass basic auth
-            GitRepository.check_url(url, config)
+            return GitRepository.check_url(url, config)
         elif 'svn+http' in url[:8]:  # svn->git import
             raise NotImplementedError()
         elif 'hg+http' in url[:8]:  # hg->git import
@@ -121,7 +121,7 @@ def url_validator(url, repo_type, config
         # no validation for SVN yet
         return
 
-    raise InvalidCloneUrl('No repo type specified')
+    raise InvalidCloneUrl('Invalid repo type specified: `{}`'.format(repo_type))
 
 
 class CloneUriValidator(object):