Skip to content

Commit a367fb0

Browse files
committed
Moved update case before delete case
1 parent ba6a4f5 commit a367fb0

File tree

1 file changed

+13
-12
lines changed

1 file changed

+13
-12
lines changed

source/app/blueprints/rest/v2/cases/__init__.py

Lines changed: 13 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -148,30 +148,31 @@ def case_routes_get(identifier):
148148
return response_api_success(CaseSchemaForAPIV2().dump(case))
149149

150150

151-
@cases_blueprint.delete('/<int:identifier>')
151+
@cases_blueprint.put('/<int:identifier>')
152152
@ac_api_requires(Permissions.standard_user)
153-
def case_routes_delete(identifier):
154-
"""
155-
Delete a case by ID
156-
"""
157-
153+
def rest_v2_cases_update(identifier):
158154
if not ac_fast_check_current_user_has_case_access(identifier, [CaseAccessLevel.full_access]):
159155
return ac_api_return_access_denied(caseid=identifier)
160156

161157
try:
162-
cases_delete(identifier)
163-
return response_api_deleted()
158+
case, _ = cases_update(identifier, request.get_json())
159+
return response_api_success(CaseSchemaForAPIV2().dump(case))
164160
except BusinessProcessingError as e:
165161
return response_api_error(e.get_message())
166162

167-
@cases_blueprint.put('/<int:identifier>')
163+
164+
@cases_blueprint.delete('/<int:identifier>')
168165
@ac_api_requires(Permissions.standard_user)
169-
def rest_v2_cases_update(identifier):
166+
def case_routes_delete(identifier):
167+
"""
168+
Delete a case by ID
169+
"""
170+
170171
if not ac_fast_check_current_user_has_case_access(identifier, [CaseAccessLevel.full_access]):
171172
return ac_api_return_access_denied(caseid=identifier)
172173

173174
try:
174-
case, _ = cases_update(identifier, request.get_json())
175-
return response_api_success(CaseSchemaForAPIV2().dump(case))
175+
cases_delete(identifier)
176+
return response_api_deleted()
176177
except BusinessProcessingError as e:
177178
return response_api_error(e.get_message())

0 commit comments

Comments
 (0)