From 8c9488229cf33a457381bbb849cebce3509b6991 Mon Sep 17 00:00:00 2001 From: richard-dds Date: Tue, 16 Oct 2018 11:16:53 -0400 Subject: [PATCH] Factor out _raise --- atst/routes/requests/financial_verification.py | 16 ++++++++-------- 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/atst/routes/requests/financial_verification.py b/atst/routes/requests/financial_verification.py index c1fe43bc..3fbacf6f 100644 --- a/atst/routes/requests/financial_verification.py +++ b/atst/routes/requests/financial_verification.py @@ -44,6 +44,10 @@ class FinancialVerificationBase(object): def _apply_task_order_number_error(self, field): field.errors += ("Task Order number not found",) + def _raise(self, form): + form.reset() + raise FormValidationError(form) + class GetFinancialVerificationForm(FinancialVerificationBase): def __init__(self, user, request, is_extended=False): @@ -98,8 +102,7 @@ class UpdateFinancialVerification(FinancialVerificationBase): self.request.id, form.data ) else: - form.reset() - raise FormValidationError(form) + self._raise(form) if should_submit: updated_request = Requests.submit_financial_verification(updated_request) @@ -107,8 +110,7 @@ class UpdateFinancialVerification(FinancialVerificationBase): workspace = Requests.approve_and_create_workspace(updated_request) submitted = True else: - form.reset() - raise FormValidationError(form) + self._raise(form) if submitted: return { @@ -142,8 +144,7 @@ class SaveFinancialVerificationDraft(FinancialVerificationBase): valid = True if not form.validate_draft(): - form.reset() - raise FormValidationError(form) + self._raise(form) if form.pe_id.data and not self.pe_validator.validate( self.request, form.pe_id.data @@ -164,8 +165,7 @@ class SaveFinancialVerificationDraft(FinancialVerificationBase): if valid: return {"request": updated_request} else: - form.reset() - raise FormValidationError(form) + self._raise(form) @requests_bp.route("/requests/verify/", methods=["GET"])