refactor request details rendering logic

atst/models/request.py

@@ -204,3 +204,11 @@ class Request(Base):
             review = self.latest_status.review
             if review:
                 return review.comment
+
+    @property
+    def has_financial_data(self):
+        return (
+            self.is_pending_ccpo_approval
+            or self.is_pending_financial_verification_changes
+            or self.is_approved
+        ) and self.task_order

atst/routes/requests/approval.py

@@ -20,9 +20,7 @@ def map_ccpo_authorizing(user):
 
 def render_approval(request, form=None):
     data = request.body
-    pending_final_approval = request.is_pending_ccpo_approval
-    pending_review = request.is_pending_ccpo_acceptance or pending_final_approval
-    if pending_final_approval and request.task_order:
+    if request.has_financial_data:
         data["task_order"] = request.task_order.to_dictionary()
 
     if not form:
@@ -35,8 +33,6 @@ def render_approval(request, form=None):
         reviews=list(reversed(request.reviews)),
         request=request,
         current_status=request.status.value,
-        pending_review=pending_review,
-        financial_review=pending_final_approval,
         f=form or CCPOReviewForm(),
     )
 

atst/routes/requests/requests_form.py

@@ -128,24 +128,18 @@ def requests_submit(request_id=None):
 @requests_bp.route("/requests/details/<string:request_id>", methods=["GET"])
 def view_request_details(request_id=None):
     request = Requests.get(g.current_user, request_id)
-    financial_review = (
-        request.is_pending_ccpo_approval
-        or request.is_approved
-        or request.is_pending_financial_verification_changes
-    )
     requires_fv_action = (
         request.is_pending_financial_verification
         or request.is_pending_financial_verification_changes
     )
 
     data = request.body
-    if financial_review and request.task_order:
+    if request.has_financial_data:
         data["task_order"] = request.task_order.to_dictionary()
 
     return render_template(
         "requests/details.html",
         data=data,
         request=request,
-        financial_review=financial_review,
         requires_fv_action=requires_fv_action,
     )