exception-handling: nicer error catching on repository creation.

This commit is contained in:
Marcin Kuzminski 2016-12-05 08:51:52 +01:00
parent 9dbb6e83d0
commit 6bf503349b
2 changed files with 6 additions and 6 deletions

View file

@ -159,8 +159,7 @@ def create_repo(form_data, cur_user):
state=state
)
action_logger(cur_user, 'user_created_repo',
repo_name_full, '', DBS)
action_logger(cur_user, 'user_created_repo', repo_name_full, '', DBS)
DBS.commit()
# now create this repo on Filesystem
@ -179,9 +178,9 @@ def create_repo(form_data, cur_user):
# set new created state
repo.set_state(Repository.STATE_CREATED)
DBS.commit()
except Exception as e:
log.warning('Exception %s occurred when creating repository, '
'doing cleanup...', e)
except Exception:
log.warning('Exception occurred when creating repository, '
'doing cleanup...', exc_info=True)
# rollback things manually !
repo = Repository.get_by_repo_name(repo_name_full)
if repo:

View file

@ -264,7 +264,8 @@ class SettingsModel(BaseModel):
repo = self._get_repo(self.repo)
if not repo:
raise Exception(
'Repository {} cannot be found'.format(self.repo))
'Repository `{}` cannot be found inside the database'.format(
self.repo))
return repo
def _filter_by_repo(self, model, query):