mirror of
https://github.com/cisagov/manage.get.gov.git
synced 2025-07-21 10:16:13 +02:00
Fix middleware redirect bug
This commit is contained in:
parent
9ccea3f776
commit
45479e86c7
3 changed files with 14 additions and 4 deletions
|
@ -3006,7 +3006,10 @@ class PortfolioAdmin(ListHeaderAdmin):
|
||||||
|
|
||||||
def get_user_portfolio_permission_admins(self, obj):
|
def get_user_portfolio_permission_admins(self, obj):
|
||||||
"""Returns each admin on UserPortfolioPermission for a given portfolio."""
|
"""Returns each admin on UserPortfolioPermission for a given portfolio."""
|
||||||
|
if obj:
|
||||||
return obj.portfolio_users.filter(portfolio=obj, roles__contains=[UserPortfolioRoleChoices.ORGANIZATION_ADMIN])
|
return obj.portfolio_users.filter(portfolio=obj, roles__contains=[UserPortfolioRoleChoices.ORGANIZATION_ADMIN])
|
||||||
|
else:
|
||||||
|
return []
|
||||||
|
|
||||||
def get_non_admin_users(self, obj):
|
def get_non_admin_users(self, obj):
|
||||||
# Filter UserPortfolioPermission objects related to the portfolio that do NOT have the "Admin" role
|
# Filter UserPortfolioPermission objects related to the portfolio that do NOT have the "Admin" role
|
||||||
|
@ -3021,7 +3024,10 @@ class PortfolioAdmin(ListHeaderAdmin):
|
||||||
|
|
||||||
def get_user_portfolio_permission_non_admins(self, obj):
|
def get_user_portfolio_permission_non_admins(self, obj):
|
||||||
"""Returns each admin on UserPortfolioPermission for a given portfolio."""
|
"""Returns each admin on UserPortfolioPermission for a given portfolio."""
|
||||||
|
if obj:
|
||||||
return obj.portfolio_users.exclude(roles__contains=[UserPortfolioRoleChoices.ORGANIZATION_ADMIN])
|
return obj.portfolio_users.exclude(roles__contains=[UserPortfolioRoleChoices.ORGANIZATION_ADMIN])
|
||||||
|
else:
|
||||||
|
return []
|
||||||
|
|
||||||
def display_admins(self, obj):
|
def display_admins(self, obj):
|
||||||
"""Get joined users who are Admin, unpack and return an HTML block.
|
"""Get joined users who are Admin, unpack and return an HTML block.
|
||||||
|
@ -3042,7 +3048,9 @@ class PortfolioAdmin(ListHeaderAdmin):
|
||||||
address_id = f"portfolio-administrator-{portfolio_admin.pk}"
|
address_id = f"portfolio-administrator-{portfolio_admin.pk}"
|
||||||
if len(admins) > 1:
|
if len(admins) > 1:
|
||||||
admin_details += (
|
admin_details += (
|
||||||
f'<label class="organization-admin-label" for="{address_id}">Organization admin {i+1}</label>'
|
f'<label class="organization-admin-label padding-top-0" for="{address_id}">'
|
||||||
|
f'Organization admin {i+1}'
|
||||||
|
'</label>'
|
||||||
)
|
)
|
||||||
admin_details += f'<address id="{address_id}" class="margin-bottom-2 dja-address-contact-list">'
|
admin_details += f'<address id="{address_id}" class="margin-bottom-2 dja-address-contact-list">'
|
||||||
admin_details += f'<a href="{change_url}">{escape(portfolio_admin.user)}</a><br>'
|
admin_details += f'<a href="{change_url}">{escape(portfolio_admin.user)}</a><br>'
|
||||||
|
|
|
@ -80,7 +80,7 @@ class UserPortfolioPermission(TimeStampedModel):
|
||||||
readable_roles = sorted(
|
readable_roles = sorted(
|
||||||
[UserPortfolioRoleChoices.get_user_portfolio_role_label(role) for role in self.roles]
|
[UserPortfolioRoleChoices.get_user_portfolio_role_label(role) for role in self.roles]
|
||||||
)
|
)
|
||||||
return f"{self.user}' " f"<Roles: {', '.join(readable_roles)}>" if self.roles else ""
|
return f"{self.user}" f" <Roles: {', '.join(readable_roles)}>" if self.roles else ""
|
||||||
|
|
||||||
def get_readable_roles(self):
|
def get_readable_roles(self):
|
||||||
"""Returns a list of labels of each role in self.roles"""
|
"""Returns a list of labels of each role in self.roles"""
|
||||||
|
|
|
@ -49,11 +49,13 @@ class CheckUserProfileMiddleware:
|
||||||
self.setup_page,
|
self.setup_page,
|
||||||
self.logout_page,
|
self.logout_page,
|
||||||
"/admin",
|
"/admin",
|
||||||
|
"/__debug__",
|
||||||
]
|
]
|
||||||
self.other_excluded_pages = [
|
self.other_excluded_pages = [
|
||||||
self.profile_page,
|
self.profile_page,
|
||||||
self.logout_page,
|
self.logout_page,
|
||||||
"/admin",
|
"/admin",
|
||||||
|
"/__debug__",
|
||||||
]
|
]
|
||||||
|
|
||||||
self.excluded_pages = {
|
self.excluded_pages = {
|
||||||
|
|
Loading…
Add table
Add a link
Reference in a new issue