diff --git a/src/registrar/admin.py b/src/registrar/admin.py index 25f2afe34..8085af631 100644 --- a/src/registrar/admin.py +++ b/src/registrar/admin.py @@ -2563,7 +2563,7 @@ class PortfolioAdmin(ListHeaderAdmin): def save_model(self, request, obj, form, change): - if not obj.creator is None: + if obj.creator is not None: # ---- update creator ---- # Set the creator field to the current admin user obj.creator = request.user if request.user.is_authenticated else None diff --git a/src/registrar/models/federal_agency.py b/src/registrar/models/federal_agency.py index 521d5875a..8db415bbd 100644 --- a/src/registrar/models/federal_agency.py +++ b/src/registrar/models/federal_agency.py @@ -234,4 +234,4 @@ class FederalAgency(TimeStampedModel): @classmethod def get_non_federal_agency(cls): """Returns the non-federal agency.""" - return FederalAgency.objects.filter(agency="Non-Federal Agency").first() \ No newline at end of file + return FederalAgency.objects.filter(agency="Non-Federal Agency").first() diff --git a/src/registrar/models/portfolio.py b/src/registrar/models/portfolio.py index 3382de98a..a05422960 100644 --- a/src/registrar/models/portfolio.py +++ b/src/registrar/models/portfolio.py @@ -20,7 +20,7 @@ class Portfolio(TimeStampedModel): # use the short names in Django admin OrganizationChoices = DomainRequest.OrganizationChoices StateTerritoryChoices = DomainRequest.StateTerritoryChoices - + # Stores who created this model. If no creator is specified in DJA, # then the creator will default to the current request user""" creator = models.ForeignKey("registrar.User", on_delete=models.PROTECT, help_text="Associated user", unique=False)