diff --git a/kallithea/controllers/summary.py b/kallithea/controllers/summary.py --- a/kallithea/controllers/summary.py +++ b/kallithea/controllers/summary.py @@ -119,10 +119,12 @@ class SummaryController(BaseRepoControll username = safe_str(request.authuser.username) _def_clone_uri = _def_clone_uri_by_id = c.clone_uri_tmpl - if '{repo}' in _def_clone_uri: - _def_clone_uri_by_id = _def_clone_uri.replace('{repo}', '_{repoid}') - elif '{repoid}' in _def_clone_uri: - _def_clone_uri_by_id = _def_clone_uri.replace('_{repoid}', '{repo}') + if '{repo}' in _def_clone_uri_by_id: + _def_clone_uri_by_id = _def_clone_uri_by_id.replace('{repo}', '_{repoid}') + elif '_{repoid}' in _def_clone_uri: + _def_clone_uri = _def_clone_uri.replace('_{repoid}', '{repo}') + else: + log.error("Configured clone_uri_tmpl %r has no '{repo}' or '_{repoid}' and cannot toggle to use repo id URLs", c.clone_uri_tmpl) c.clone_repo_url = c.db_repo.clone_url(user=username, uri_tmpl=_def_clone_uri)