Skip to content
Snippets Groups Projects
Commit 04043cce authored by Simon Künzel's avatar Simon Künzel
Browse files

Fix missing error when changing deletion status of object to already set value

parent f9e6d08c
No related branches found
No related tags found
No related merge requests found
...@@ -541,7 +541,7 @@ class ApiObjectClass: ...@@ -541,7 +541,7 @@ class ApiObjectClass:
session.flush() session.flush()
return obj.id return obj.id
def set_deletion(self, def modify_deletion_state(self,
session: SessionDb, session: SessionDb,
modifying_user_id: int, modifying_user_id: int,
object_id: int, object_id: int,
...@@ -563,6 +563,8 @@ class ApiObjectClass: ...@@ -563,6 +563,8 @@ class ApiObjectClass:
obj = session.scalar(query) obj = session.scalar(query)
if obj is None: if obj is None:
raise ApiClientException(ERROR_UNKNOWN_OBJECT) raise ApiClientException(ERROR_UNKNOWN_OBJECT)
if obj.deleted == new_deleted:
raise ApiClientException(ERROR_OBJECT_ERROR("Object is already deleted" if new_deleted else "Object is not deleted"))
obj.deleted = new_deleted obj.deleted = new_deleted
from videoag_common.objects import ChangelogDeletionChangeEntry from videoag_common.objects import ChangelogDeletionChangeEntry
session.add(ChangelogDeletionChangeEntry( session.add(ChangelogDeletionChangeEntry(
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment