filter by READY, edge case handling

src/registrar/templates/django/admin/domain_change_list.html

@@ -3,13 +3,6 @@
 {% block object-tools %}
 
 <ul class="object-tools">
-    {% if has_add_permission %}
-    <li>
-    <a href="{% url 'admin:registrar_domain_add' %}" class="addlink">
-        Add Domain
-    </a>
-    </li>
-    {% endif %}
     <li>
         <a href="{% url 'admin:registrar_domain_export_data_type' %}" class="button">Export all domain metadata</a>
     </li>
@@ -19,5 +12,12 @@
     <li>
         <a href="{% url 'admin:registrar_domain_export_data_federal' %}" class="button">Export current-federal.csv</a>
     </li>
+    {% if has_add_permission %}
+    <li>
+    <a href="{% url 'admin:registrar_domain_add' %}" class="addlink">
+        Add Domain
+    </a>
+    </li>
+    {% endif %}
 </ul>
 {% endblock %}

src/registrar/utility/csv_export.py

@@ -1,4 +1,5 @@
 import csv
+from registrar.models.domain import Domain
 from registrar.models.domain_information import DomainInformation
 from registrar.models.public_contact import PublicContact
 
@@ -25,16 +26,28 @@ def export_domains_to_writer(writer, columns, sort_fields, filter_condition):
             "State": domainInfo.state_territory,
             "AO": domainInfo.authorizing_official.first_name
             + " "
-            + domainInfo.authorizing_official.last_name,
+            + domainInfo.authorizing_official.last_name
-            "AO email": domainInfo.authorizing_official.email,
+            if domainInfo.authorizing_official
+            else " ",
+            "AO email": domainInfo.authorizing_official.email
+            if domainInfo.authorizing_official
+            else " ",
             "Submitter": domainInfo.submitter.first_name
             + " "
-            + domainInfo.submitter.last_name,
+            + domainInfo.submitter.last_name
-            "Submitter title": domainInfo.submitter.title,
+            if domainInfo.submitter
-            "Submitter email": domainInfo.submitter.email,
+            else " ",
-            "Submitter phone": domainInfo.submitter.phone,
+            "Submitter title": domainInfo.submitter.title
+            if domainInfo.submitter
+            else " ",
+            "Submitter email": domainInfo.submitter.email
+            if domainInfo.submitter
+            else " ",
+            "Submitter phone": domainInfo.submitter.phone
+            if domainInfo.submitter
+            else " ",
             "Security Contact Email": security_contacts[0].email
-            if security_contacts.exists()
+            if security_contacts
             else " ",
             "Status": domainInfo.domain.state,
         }
@@ -62,7 +75,7 @@ def export_data_type_to_csv(csv_file):
         # 'Expiration Date'
     ]
     sort_fields = ["domain__name"]
-    filter_condition = {}
+    filter_condition = {"domain__state": Domain.State.READY}
     export_domains_to_writer(writer, columns, sort_fields, filter_condition)
 
 
@@ -79,7 +92,7 @@ def export_data_full_to_csv(csv_file):
         "Security Contact Email",
     ]
     sort_fields = ["domain__name", "federal_agency", "organization_type"]
-    filter_condition = {}
+    filter_condition = {"domain__state": Domain.State.READY}
     export_domains_to_writer(writer, columns, sort_fields, filter_condition)
 
 
@@ -96,5 +109,5 @@ def export_data_federal_to_csv(csv_file):
         "Security Contact Email",
     ]
     sort_fields = ["domain__name", "federal_agency", "organization_type"]
-    filter_condition = {"organization_type__icontains": "federal"}
+    filter_condition = {"organization_type__icontains": "federal", "domain__state": Domain.State.READY}
     export_domains_to_writer(writer, columns, sort_fields, filter_condition)