Delete functions should not throw an error, if the ressource does not exists. -> Consider it as deleted if does not exist

This commit is contained in:
Patrick Jentsch 2020-04-21 10:27:42 +02:00
parent e83c762551
commit 29f1be5c80

View File

@ -5,16 +5,20 @@ def delete_corpus_(app, corpus_id):
with app.app_context(): with app.app_context():
corpus = Corpus.query.get(corpus_id) corpus = Corpus.query.get(corpus_id)
if corpus is None: if corpus is None:
raise Exception('Corpus {} not found!'.format(corpus_id)) # raise Exception('Corpus {} not found!'.format(corpus_id))
corpus.delete() pass
else:
corpus.delete()
def delete_corpus_file_(app, corpus_file_id): def delete_corpus_file_(app, corpus_file_id):
with app.app_context(): with app.app_context():
corpus_file = CorpusFile.query.get(corpus_file_id) corpus_file = CorpusFile.query.get(corpus_file_id)
if corpus_file is None: if corpus_file is None:
raise Exception('Corpus file {} not found!'.format(corpus_file_id)) # raise Exception('Corpus file {} not found!'.format(corpus_file_id))
corpus_file.delete() pass
else:
corpus_file.delete()
def edit_corpus_file_(app, corpus_file_id): def edit_corpus_file_(app, corpus_file_id):
@ -22,4 +26,5 @@ def edit_corpus_file_(app, corpus_file_id):
corpus_file = CorpusFile.query.get(corpus_file_id) corpus_file = CorpusFile.query.get(corpus_file_id)
if corpus_file is None: if corpus_file is None:
raise Exception('Corpus file {} not found!'.format(corpus_file_id)) raise Exception('Corpus file {} not found!'.format(corpus_file_id))
corpus_file.insert_metadata() else:
corpus_file.insert_metadata()