# HG changeset patch
# User Marcin Kuzminski <marcin@rhodecode.com>
# Date 2017-11-16 14:59:11
# Node ID 8956a9814105394668498462172e14fd732d6b70
# Parent  ca2046af2866074ac68b4833efcd5d8759b2b0b2

simplevcs: fixed logic of extraction of base_path

diff --git a/rhodecode/lib/middleware/simplevcs.py b/rhodecode/lib/middleware/simplevcs.py
--- a/rhodecode/lib/middleware/simplevcs.py
+++ b/rhodecode/lib/middleware/simplevcs.py
@@ -122,7 +122,11 @@ class SimpleVCS(object):
 
     @property
     def base_path(self):
-        return self.repo_vcs_config.get(*VcsSettingsModel.PATH_SETTING)
+        settings_path = self.repo_vcs_config.get(*VcsSettingsModel.PATH_SETTING)
+        if not settings_path:
+            # try, maybe we passed in explicitly as config option
+            settings_path = self.config.get('base_path')
+        return settings_path
 
     def set_repo_names(self, environ):
         """
diff --git a/rhodecode/lib/middleware/vcs.py b/rhodecode/lib/middleware/vcs.py
--- a/rhodecode/lib/middleware/vcs.py
+++ b/rhodecode/lib/middleware/vcs.py
@@ -181,7 +181,11 @@ class VCSMiddleware(object):
         """
         returns serialized VcsSettings
         """
-        return VcsSettingsModel(repo=repo_name).get_ui_settings_as_config_obj()
+        try:
+            return VcsSettingsModel(
+                repo=repo_name).get_ui_settings_as_config_obj()
+        except Exception:
+            pass
 
     def wrap_in_gzip_if_enabled(self, app, config):
         if self.use_gzip:
@@ -209,6 +213,14 @@ class VCSMiddleware(object):
             # Set acl, url and vcs repo names.
             vcs_handler.set_repo_names(environ)
 
+            # register repo config back to the handler
+            vcs_conf = self.vcs_config(vcs_handler.acl_repo_name)
+            # maybe damaged/non existent settings. We still want to
+            # pass that point to validate on is_valid_and_existing_repo
+            # and return proper HTTP Code back to client
+            if vcs_conf:
+                vcs_handler.repo_vcs_config = vcs_conf
+
             # check for type, presence in database and on filesystem
             if not vcs_handler.is_valid_and_existing_repo(
                     vcs_handler.acl_repo_name,
@@ -218,10 +230,6 @@ class VCSMiddleware(object):
 
             environ['REPO_NAME'] = vcs_handler.url_repo_name
 
-            # register repo config back to the handler
-            vcs_handler.repo_vcs_config = self.vcs_config(
-                vcs_handler.acl_repo_name)
-
             # Wrap handler in middlewares if they are enabled.
             vcs_handler = self.wrap_in_gzip_if_enabled(
                 vcs_handler, self.config)