From 9dec0753f28cfd40b3df60214edd0d645fb8d50c Mon Sep 17 00:00:00 2001 From: zandercymatics <141044360+zandercymatics@users.noreply.github.com> Date: Thu, 7 Mar 2024 10:42:30 -0700 Subject: [PATCH] Revert change where only status change is forbidden --- src/registrar/admin.py | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/src/registrar/admin.py b/src/registrar/admin.py index 0d9045920..6dc6cd94d 100644 --- a/src/registrar/admin.py +++ b/src/registrar/admin.py @@ -134,6 +134,9 @@ class DomainApplicationAdminForm(forms.ModelForm): status = cleaned_data.get("status") investigator = cleaned_data.get("investigator") + # Get the old status + initial_status = self.initial.get("status", None) + # We only care about investigator when in these statuses checked_statuses = [ DomainApplication.ApplicationStatus.APPROVED, @@ -144,7 +147,7 @@ class DomainApplicationAdminForm(forms.ModelForm): ] # If a status change occured, check for validity - if status in checked_statuses: + if status != initial_status and status in checked_statuses: # Checks the "investigators" field for validity. # That field must obey certain conditions when an application is approved. # Will call "add_error" if any issues are found.