#2852: property to portfolio related fields - [AD]

src/registrar/admin.py

@@ -1700,20 +1700,44 @@ def queryset(self, request, queryset):
             if self.value() == "0":
                 return queryset.filter(Q(is_election_board=False) | Q(is_election_board=None))
 
+    @admin.display(description=_("Generic Org Type"))
+    def converted_generic_org_type(self, obj):
+        return obj.converted_generic_org_type
+
+    @admin.display(description=_("Organization Name"))
+    def converted_organization_name(self, obj):
+        return obj.converted_organization_name
+
+    @admin.display(description=_("Federal Agency"))
+    def converted_federal_agency(self, obj):
+        return obj.converted_federal_agency
+
+    @admin.display(description=_("Federal Type"))
+    def converted_federal_type(self, obj):
+        return obj.converted_federal_type
+
+    @admin.display(description=_("City"))
+    def converted_city(self, obj):
+        return obj.converted_city
+
+    @admin.display(description=_("State/Territory"))
+    def converted_state_territory(self, obj):
+        return obj.converted_state_territory
+
     # Columns
     list_display = [
         "requested_domain",
         "first_submitted_date",
         "last_submitted_date",
         "last_status_update",
         "status",
-        "generic_org_type",
-        "federal_type",
-        "federal_agency",
-        "organization_name",
         "custom_election_board",
-        "city",
-        "state_territory",
+        "converted_generic_org_type",
+        "converted_organization_name",
+        "converted_federal_agency",
+        "converted_federal_type",
+        "converted_city",
+        "converted_state_territory",
         "investigator",
     ]
 
@@ -1852,7 +1876,7 @@ def status_history(self, obj):
 
     # Read only that we'll leverage for CISA Analysts
     analyst_readonly_fields = [
-        "federal_agency",
+        "converted_federal_agency",
         "creator",
         "about_your_organization",
         "requested_domain",
@@ -1869,15 +1893,16 @@ def status_history(self, obj):
         "suborganization_city",
         "suborganization_state_territory",
     ]
+
     autocomplete_fields = [
         "approved_domain",
         "requested_domain",
         "creator",
-        "senior_official",
         "investigator",
         "portfolio",
         "sub_organization",
     ]
+
     filter_horizontal = ("current_websites", "alternative_domains", "other_contacts")
 
     # Table ordering

src/registrar/models/domain_request.py

@@ -1409,3 +1409,48 @@ def _form_complete(self, request):
         if not is_complete or not self._is_general_form_complete(request):
             return False
         return True
+
+    """The following converted_ property methods get field data from this domain request's portfolio,
+    if there is an associated portfolio. If not, they return data from the domain request model."""
+
+    @property
+    def converted_organization_name(self):
+        if self.portfolio:
+            return self.portfolio.organization_name
+        return self.organization_name
+
+    @property
+    def converted_generic_org_type(self):
+        if self.portfolio:
+            return self.portfolio.organization_type
+        return self.generic_org_type
+
+    @property
+    def converted_federal_agency(self):
+        if self.portfolio:
+            return self.portfolio.federal_agency
+        return self.federal_agency
+
+    @property
+    def converted_federal_type(self):
+        if self.portfolio:
+            return self.portfolio.federal_type
+        return self.federal_type
+
+    @property
+    def converted_city(self):
+        if self.portfolio:
+            return self.portfolio.city
+        return self.city
+
+    @property
+    def converted_state_territory(self):
+        if self.portfolio:
+            return self.portfolio.state_territory
+        return self.state_territory
+
+    @property
+    def converted_senior_official(self):
+        if self.portfolio:
+            return self.portfolio.senior_official
+        return self.senior_official

src/registrar/tests/test_admin_request.py

@@ -578,7 +578,7 @@ def test_short_org_name_in_domain_requests_list(self):
         # of the request
         self.assertContains(response, "Federal", count=52)
         # This may be a bit more robust
-        self.assertContains(response, '<td class="field-generic_org_type">Federal</td>', count=1)
+        self.assertContains(response, '<td class="field-converted_generic_org_type">federal</td>', count=1)
         # Now let's make sure the long description does not exist
         self.assertNotContains(response, "Federal: an agency of the U.S. government")
 
@@ -1696,7 +1696,7 @@ def test_readonly_fields_for_analyst(self):
                 "alternative_domains",
                 "is_election_board",
                 "status_history",
-                "federal_agency",
+                "converted_federal_agency",
                 "creator",
                 "about_your_organization",
                 "requested_domain",

src/registrar/tests/test_models_requests.py

@@ -14,6 +14,7 @@
     DraftDomain,
     FederalAgency,
     AllowedEmail,
+    Portfolio,
 )
 
 import boto3_mocking
@@ -95,6 +96,7 @@ def tearDown(self):
         DomainRequest.objects.all().delete()
         DraftDomain.objects.all().delete()
         Domain.objects.all().delete()
+        Portfolio.objects.all().delete()
         User.objects.all().delete()
         self.mock_client.EMAILS_SENT.clear()
 
@@ -1045,3 +1047,26 @@ def test_has_other_contacts_returns_false(self):
             status=DomainRequest.DomainRequestStatus.STARTED, name="no-others.gov", has_other_contacts=False
         )
         self.assertEquals(domain_request.has_other_contacts(), False)
+
+    @less_console_noise_decorator
+    def test_converted_type(self):
+        """test that new property fields works as expected to pull domain req info such as fed agency,
+        generic org type, and others from portfolio"""
+        fed_agency = FederalAgency.objects.filter(agency="Non-Federal Agency").first()
+        portfolio = Portfolio.objects.create(
+            organization_name="Test Portfolio",
+            creator=self.dummy_user_2,
+            federal_agency=fed_agency,
+            organization_type=DomainRequest.OrganizationChoices.FEDERAL,
+        )
+
+        domain_request = completed_domain_request(name="domainre1.gov", portfolio=portfolio)
+
+        self.assertEqual(portfolio.organization_type, domain_request.converted_generic_org_type)
+        self.assertEqual(portfolio.federal_agency, domain_request.converted_federal_agency)
+
+        domain_request2 = completed_domain_request(
+            name="domainreq2.gov", federal_agency=fed_agency, generic_org_type=DomainRequest.OrganizationChoices.TRIBAL
+        )
+        self.assertEqual(domain_request2.generic_org_type, domain_request2.converted_generic_org_type)
+        self.assertEqual(domain_request2.federal_agency, domain_request2.converted_federal_agency)

src/registrar/tests/test_reports.py

@@ -701,6 +701,7 @@ def test_domain_request_growth(self):
             csv_file.seek(0)
             # Read the content into a variable
             csv_content = csv_file.read()
+
             expected_content = (
                 "Domain request,Domain type,Federal type\n"
                 "city3.gov,Federal,Executive\n"

src/registrar/utility/csv_export.py

@@ -660,17 +660,17 @@ def exporting_dr_data_to_csv(cls, response, request=None):
                     cls.safe_get(getattr(request, "all_alternative_domains", None)),
                     cls.safe_get(getattr(request, "all_other_contacts", None)),
                     cls.safe_get(getattr(request, "all_current_websites", None)),
-                    cls.safe_get(getattr(request, "federal_agency", None)),
-                    cls.safe_get(getattr(request.senior_official, "first_name", None)),
-                    cls.safe_get(getattr(request.senior_official, "last_name", None)),
-                    cls.safe_get(getattr(request.senior_official, "email", None)),
-                    cls.safe_get(getattr(request.senior_official, "title", None)),
+                    cls.safe_get(getattr(request, "converted_federal_agency", None)),
+                    cls.safe_get(getattr(request.converted_senior_official, "first_name", None)),
+                    cls.safe_get(getattr(request.converted_senior_official, "last_name", None)),
+                    cls.safe_get(getattr(request.converted_senior_official, "email", None)),
+                    cls.safe_get(getattr(request.converted_senior_official, "title", None)),
                     cls.safe_get(getattr(request.creator, "first_name", None)),
                     cls.safe_get(getattr(request.creator, "last_name", None)),
                     cls.safe_get(getattr(request.creator, "email", None)),
-                    cls.safe_get(getattr(request, "organization_name", None)),
-                    cls.safe_get(getattr(request, "city", None)),
-                    cls.safe_get(getattr(request, "state_territory", None)),
+                    cls.safe_get(getattr(request, "converted_organization_name", None)),
+                    cls.safe_get(getattr(request, "converted_city", None)),
+                    cls.safe_get(getattr(request, "converted_state_territory", None)),
                     cls.safe_get(getattr(request, "purpose", None)),
                     cls.safe_get(getattr(request, "cisa_representative_email", None)),
                     cls.safe_get(getattr(request, "last_submitted_date", None)),