Skip to content

Commit 3d7b0b8

Browse files
author
Patrick Gan
committed
Fix linting issues
1 parent b10c7a4 commit 3d7b0b8

File tree

2 files changed

+21
-9
lines changed

2 files changed

+21
-9
lines changed

rca/account_management/views.py

Lines changed: 13 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -5,9 +5,9 @@
55
from django.contrib.auth.models import Group
66
from django.core.exceptions import PermissionDenied
77
from django.core.mail import send_mail
8+
from django.shortcuts import redirect, render
89
from django.template.loader import render_to_string
910
from django.urls import reverse
10-
from django.shortcuts import redirect, render
1111
from django.utils.decorators import method_decorator
1212
from django.utils.translation import gettext as _
1313
from django.views.decorators.csrf import csrf_exempt
@@ -197,23 +197,29 @@ class CustomLogoutView(LogoutView):
197197
"""Check if user is signed in via SSO. If yes, redirect them to a confirmation logout page."""
198198

199199
def dispatch(self, request, *args, **kwargs):
200-
if self.request.session.get('_auth_user_backend', '') == 'social_core.backends.azuread_tenant.AzureADTenantOAuth2':
200+
if (
201+
self.request.session.get("_auth_user_backend", "")
202+
== "social_core.backends.azuread_tenant.AzureADTenantOAuth2"
203+
):
201204
# redirect to `sso_logout_confirmation`
202-
return redirect('sso_logout_confirmation')
203-
205+
return redirect("sso_logout_confirmation")
206+
204207
return super().dispatch(request, *args, **kwargs)
205208

206209

207210
class SSOLogoutConfirmationView(LogoutView):
208211
template_name = "patterns/pages/auth/logout_confirmation.html"
209212

210213
def dispatch(self, request, *args, **kwargs):
211-
if self.request.session.get('_auth_user_backend', '') == 'social_core.backends.azuread_tenant.AzureADTenantOAuth2':
214+
if (
215+
self.request.session.get("_auth_user_backend", "")
216+
== "social_core.backends.azuread_tenant.AzureADTenantOAuth2"
217+
):
212218
# If the request is a POST, log out the user
213-
if request.method == 'POST':
219+
if request.method == "POST":
214220
return super().dispatch(request, *args, **kwargs)
215221

216222
# If the request is GET, render the confirmation page
217223
return render(request, self.template_name)
218-
224+
219225
return super().dispatch(request, *args, **kwargs)

rca/urls.py

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,11 @@
1111
from wagtail.documents import urls as wagtaildocs_urls
1212
from wagtail.utils.urlpatterns import decorate_urlpatterns
1313

14-
from rca.account_management.views import CustomLoginView, CustomLogoutView, SSOLogoutConfirmationView
14+
from rca.account_management.views import (
15+
CustomLoginView,
16+
CustomLogoutView,
17+
SSOLogoutConfirmationView,
18+
)
1519
from rca.search import views as search_views
1620
from rca.utils.cache import get_default_cache_control_decorator
1721
from rca.wagtailapi.api import api_router
@@ -24,7 +28,9 @@
2428
path("admin/login/", CustomLoginView.as_view(), name="wagtailcore_login"),
2529
path("admin/_util/login/", CustomLoginView.as_view(), name="wagtailcore_login"),
2630
path("admin/logout/", CustomLogoutView.as_view(), name="wagtailcore_logout"),
27-
path("logout/", SSOLogoutConfirmationView.as_view(), name="sso_logout_confirmation"),
31+
path(
32+
"logout/", SSOLogoutConfirmationView.as_view(), name="sso_logout_confirmation"
33+
),
2834
path("django-admin/", admin.site.urls),
2935
path("admin/", include(wagtailadmin_urls)),
3036
path("documents2/", include(wagtaildocs_urls)),

0 commit comments

Comments
 (0)