From 23548e9cd25aa4cf71c1b08429177668c79df407 Mon Sep 17 00:00:00 2001
From: Valentin Deniaud <vdeniaud@entrouvert.com>
Date: Tue, 27 Jul 2021 10:41:16 +0200
Subject: [PATCH] manager: remove row link in user roles table (#55839)

---
 src/authentic2/manager/tables.py                              | 4 ++--
 .../manager/templates/authentic2/manager/user_ou_roles.html   | 2 +-
 tests/test_manager.py                                         | 1 -
 3 files changed, 3 insertions(+), 4 deletions(-)

diff --git a/src/authentic2/manager/tables.py b/src/authentic2/manager/tables.py
index c0f3f346..930d751d 100644
--- a/src/authentic2/manager/tables.py
+++ b/src/authentic2/manager/tables.py
@@ -174,7 +174,7 @@ class OuUserRolesTable(tables.Table):
 
     class Meta:
         model = get_role_model()
-        attrs = {'class': 'main', 'id': 'role-table'}
+        attrs = {'class': 'main plaintable', 'id': 'role-table'}
         fields = ('name', 'ou')
         empty_text = _('None')
         order_by = ('name',)
@@ -237,6 +237,6 @@ class UserAuthorizationsTable(tables.Table):
 
     class Meta:
         model = OIDCAuthorization
-        attrs = {'class': 'main', 'id': 'user-authorizations-table'}
+        attrs = {'class': 'main plaintable', 'id': 'user-authorizations-table'}
         fields = ('client', 'created', 'expired')
         empty_text = _('This user has not granted profile data access to any service yet.')
diff --git a/src/authentic2/manager/templates/authentic2/manager/user_ou_roles.html b/src/authentic2/manager/templates/authentic2/manager/user_ou_roles.html
index 66634fdf..5f217379 100644
--- a/src/authentic2/manager/templates/authentic2/manager/user_ou_roles.html
+++ b/src/authentic2/manager/templates/authentic2/manager/user_ou_roles.html
@@ -40,7 +40,7 @@
 {% endblock %}
 
 {% block main %}
- {% with row_link=1 url_name="a2-manager-role-members" %}
+ {% with row_link=0 %}
    {% render_table table "authentic2/manager/table.html" %}
  {% endwith %}
 {% endblock %}
diff --git a/tests/test_manager.py b/tests/test_manager.py
index e702606f..d282d6fd 100644
--- a/tests/test_manager.py
+++ b/tests/test_manager.py
@@ -377,7 +377,6 @@ def test_manager_one_ou(app, superuser, admin, simple_role, settings):
         q = response.pyquery.remove_namespaces()
         assert len(q('table tbody tr')) == 1
         assert q('table tbody tr').text() == 'simple role'
-        assert q('table tbody tr').attr('data-url') == '/manage/roles/%s/' % simple_role.pk
 
         form.set('search-internals', True)
         response = form.submit()
-- 
2.20.1