mirror of
https://github.com/cisagov/manage.get.gov.git
synced 2025-08-12 12:39:43 +02:00
cleanup
This commit is contained in:
parent
23e99c054d
commit
bd018e14ab
1 changed files with 0 additions and 9 deletions
|
@ -5,12 +5,6 @@ from registrar.models.utility.portfolio_helper import UserPortfolioPermissionCho
|
||||||
from .utility.time_stamped_model import TimeStampedModel
|
from .utility.time_stamped_model import TimeStampedModel
|
||||||
from django.contrib.postgres.fields import ArrayField
|
from django.contrib.postgres.fields import ArrayField
|
||||||
|
|
||||||
# ---Logger
|
|
||||||
import logging
|
|
||||||
from venv import logger
|
|
||||||
from registrar.management.commands.utility.terminal_helper import TerminalColors, TerminalHelper
|
|
||||||
logger = logging.getLogger(__name__)
|
|
||||||
|
|
||||||
|
|
||||||
class UserPortfolioPermission(TimeStampedModel):
|
class UserPortfolioPermission(TimeStampedModel):
|
||||||
"""This is a linking table that connects a user with a role on a portfolio."""
|
"""This is a linking table that connects a user with a role on a portfolio."""
|
||||||
|
@ -115,11 +109,8 @@ class UserPortfolioPermission(TimeStampedModel):
|
||||||
|
|
||||||
# Check if a user is set without accessing the related object.
|
# Check if a user is set without accessing the related object.
|
||||||
has_user = bool(self.user_id)
|
has_user = bool(self.user_id)
|
||||||
TerminalHelper.colorful_logger(logger.info, TerminalColors.OKCYAN, f"***CLEANING***")
|
|
||||||
if has_user:
|
if has_user:
|
||||||
existing_permission_pks = UserPortfolioPermission.objects.filter(user=self.user).values_list("pk", flat=True)
|
existing_permission_pks = UserPortfolioPermission.objects.filter(user=self.user).values_list("pk", flat=True)
|
||||||
TerminalHelper.colorful_logger(logger.info, TerminalColors.OKCYAN, f"existing_permission_pks: {existing_permission_pks}")
|
|
||||||
TerminalHelper.colorful_logger(logger.info, TerminalColors.OKCYAN, f"self pk: {self.pk}")
|
|
||||||
if not flag_is_active_for_user(self.user, "multiple_portfolios") and existing_permission_pks.exists() and not self.pk in existing_permission_pks:
|
if not flag_is_active_for_user(self.user, "multiple_portfolios") and existing_permission_pks.exists() and not self.pk in existing_permission_pks:
|
||||||
raise ValidationError(
|
raise ValidationError(
|
||||||
"This user is already assigned to a portfolio. "
|
"This user is already assigned to a portfolio. "
|
||||||
|
|
Loading…
Add table
Add a link
Reference in a new issue