From eb4cd2719ee1606fe495d4df6d584828c0b2910f Mon Sep 17 00:00:00 2001
From: CocoByte <nicolle.leclair@gmail.com>
Date: Mon, 23 Dec 2024 23:56:38 -0700
Subject: [PATCH] reverted a few suggested commits -- fixed issues

---
 src/registrar/admin.py | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/src/registrar/admin.py b/src/registrar/admin.py
index 4456c9cdc..165f12b42 100644
--- a/src/registrar/admin.py
+++ b/src/registrar/admin.py
@@ -1980,8 +1980,10 @@ class DomainRequestAdmin(ListHeaderAdmin, ImportExportModelAdmin):
             )
 
         def queryset(self, request, queryset):
-            filter_for_portfolio = self.value() == "1"
-            return queryset.filter(portfolio__isnull=filter_for_portfolio)
+            if self.value() == "1":
+                return queryset.filter(Q(portfolio__isnull=False))
+            if self.value() == "0":
+                return queryset.filter(Q(portfolio__isnull=True))
 
     # ------ Custom fields ------
     def custom_election_board(self, obj):
@@ -1993,7 +1995,7 @@ class DomainRequestAdmin(ListHeaderAdmin, ImportExportModelAdmin):
     @admin.display(description=_("Requested Domain"))
     def custom_requested_domain(self, obj):
         # Example: Show different icons based on `status`
-        url = reverse("admin:registrar_domainrequest_changelist", args=[obj.id])
+        url = reverse("admin:registrar_domainrequest_changelist") + f"?portfolio={obj.id}"
         text = obj.requested_domain
         if obj.portfolio:
             return format_html('<a href="{}"><img src="/public/admin/img/icon-yes.svg"> {}</a>', url, text)