From be3707a4e2f1214500710780e49000d9d9c19ebd Mon Sep 17 00:00:00 2001
From: Valentin Deniaud <vdeniaud@entrouvert.com>
Date: Mon, 6 May 2019 17:12:50 +0200
Subject: [PATCH 7/8] manager: use could_{action} instead of can_ in templates
 (#33515)

---
 .../templates/authentic2/manager/ou_detail.html    |  4 ++--
 .../manager/templates/authentic2/manager/ous.html  |  2 +-
 .../templates/authentic2/manager/role_members.html | 14 +++++++-------
 .../authentic2/manager/role_members_table.html     |  2 +-
 .../authentic2/manager/role_permissions.html       |  4 ++--
 .../authentic2/manager/role_permissions_table.html |  2 +-
 .../templates/authentic2/manager/roles.html        |  2 +-
 .../authentic2/manager/service_roles_table.html    |  2 +-
 .../templates/authentic2/manager/user_detail.html  |  4 ++--
 .../templates/authentic2/manager/user_roles.html   |  2 +-
 .../authentic2/manager/user_roles_table.html       |  2 +-
 11 files changed, 20 insertions(+), 20 deletions(-)

diff --git a/src/authentic2/manager/templates/authentic2/manager/ou_detail.html b/src/authentic2/manager/templates/authentic2/manager/ou_detail.html
index 9f22ac9e..3a115283 100644
--- a/src/authentic2/manager/templates/authentic2/manager/ou_detail.html
+++ b/src/authentic2/manager/templates/authentic2/manager/ou_detail.html
@@ -17,13 +17,13 @@
 {% block appbar %}
   {{ block.super }}
   <span class="actions">
-  {% if view.can_delete %}
+  {% if view.could_delete %}
     <a rel="popup" href="{% url "a2-manager-ou-delete" pk=object.pk %}">{% trans "Delete" %}</a>
   {% else %}
     <a class="disabled" title="{% trans "You do not have the rights to delete this organizational unit." %}" href="#">{% trans "Delete" %}</a>
   {% endif %}
 
-  {% if view.can_change %}
+  {% if view.could_change %}
     <a href="{% url "a2-manager-ou-edit" pk=object.pk %}">{% trans "Edit" %}</a>
   {% else %}
     <a class="disabled" title="{% trans "You do not have the rights to edit this organizational unit." %}" href="#">{% trans "Edit" %}</a>
diff --git a/src/authentic2/manager/templates/authentic2/manager/ous.html b/src/authentic2/manager/templates/authentic2/manager/ous.html
index e6da200c..73013353 100644
--- a/src/authentic2/manager/templates/authentic2/manager/ous.html
+++ b/src/authentic2/manager/templates/authentic2/manager/ous.html
@@ -10,7 +10,7 @@
 {% block appbar %}
   {{ block.super }}
   <span class="actions">
-  {% if view.can_add %}
+  {% if view.could_add %}
     <a href="{% url "a2-manager-ou-add" %}" rel="popup">{% trans "Add organizational unit" %}</a>
   {% endif %}
   </span>
diff --git a/src/authentic2/manager/templates/authentic2/manager/role_members.html b/src/authentic2/manager/templates/authentic2/manager/role_members.html
index de5ca308..36f30741 100644
--- a/src/authentic2/manager/templates/authentic2/manager/role_members.html
+++ b/src/authentic2/manager/templates/authentic2/manager/role_members.html
@@ -19,15 +19,15 @@
 {% block appbar %}
   {{ block.super }}
   <span class="actions">
-  {% if not object.is_internal and view.can_delete %}
+  {% if not object.is_internal and view.could_delete %}
     <a rel="popup" href="{% url "a2-manager-role-delete" pk=object.pk %}">{% trans "Delete" %}</a>
   {% else %}
     <a class="disabled" title="{% trans "This role is technical, you cannot delete it." %}" href="#">{% trans "Delete" %}</a>
   {% endif %}
-  {% if view.can_change and not object.is_internal %}
+  {% if view.could_change and not object.is_internal %}
     <a href="{% url "a2-manager-role-edit" pk=object.pk %}">{% trans "Edit" %}</a>
   {% else %}
-    {% if not view.can_change %}
+    {% if not view.could_change %}
       <a class="disabled" title="{% trans "Permission denied" %}" href="#">{% trans "Edit" %}</a>
     {% else %}
       <a class="disabled" title="{% trans "This role is technical, you cannot edit it." %}" href="#">{% trans "Edit" %}</a>
@@ -52,7 +52,7 @@
 
  {% include "authentic2/manager/export_include.html" with export_view_name="a2-manager-role-members-export" %}
 
- {% if view.can_change %}
+ {% if view.could_change %}
    <form method="post" class="manager-m2m-add-form">
            {% csrf_token %}
            {{ form }}
@@ -71,7 +71,7 @@
        <a title="{% trans "Indirect child role" %}" class="disabled role-remove icon-minus-sign"></a>
      {% endif %}
    {% endfor %}
-  {% if view.can_change %}
+  {% if view.could_change %}
     <a rel="popup" href="{% url "a2-manager-role-add-admin-user" pk=object.pk %}" class="role-add icon-add-sign"></a>
   {% else %}
     <a title="{% trans "Permission denied" %}" class="disabled role-add icon-add-sign"></a>
@@ -87,7 +87,7 @@
        <a title="{% trans "Indirect admin role" %}" class="disabled role-remove icon-minus-sign"></a>
      {% endif %}
    {% endfor %}
-  {% if view.can_change %}
+  {% if view.could_change %}
     <a rel="popup" href="{% url "a2-manager-role-add-admin-role" pk=object.pk %}" class="role-add icon-add-sign"></a>
   {% else %}
     <a title="{% trans "Permission denied" %}" class="disabled role-add icon-add-sign"></a>
@@ -103,7 +103,7 @@
        <a title="{% trans "Indirect child role" %}" class="disabled role-remove icon-minus-sign"></a>
      {% endif %}
    {% endfor %}
-  {% if view.can_change %}
+  {% if view.could_change %}
     <a rel="popup" href="{% url "a2-manager-role-add-child" pk=object.pk %}" class="role-add icon-add-sign"></a>
   {% else %}
     <a title="{% trans "Permission denied" %}" class="disabled role-add icon-add-sign"></a>
diff --git a/src/authentic2/manager/templates/authentic2/manager/role_members_table.html b/src/authentic2/manager/templates/authentic2/manager/role_members_table.html
index cffdaa1e..8fc98c7d 100644
--- a/src/authentic2/manager/templates/authentic2/manager/role_members_table.html
+++ b/src/authentic2/manager/templates/authentic2/manager/role_members_table.html
@@ -7,6 +7,6 @@
     <th></th>
   {% endblock %}
   {% block table.tbody.last.column %}
-  <td>{% if table.context.view.can_change and row.record.direct %}<a class="icon-remove-sign js-remove-object" data-confirm="{% blocktrans with username=row.record.get_full_name role=table.context.object %}Do you really want to remove user &quot;{{ username }}&quot; from role &quot;{{ role }}&quot;&nbsp;?{% endblocktrans %}" href="#" data-pk-arg="user"></a>{% endif %}</td>
+  <td>{% if table.context.view.could_change and row.record.direct %}<a class="icon-remove-sign js-remove-object" data-confirm="{% blocktrans with username=row.record.get_full_name role=table.context.object %}Do you really want to remove user &quot;{{ username }}&quot; from role &quot;{{ role }}&quot;&nbsp;?{% endblocktrans %}" href="#" data-pk-arg="user"></a>{% endif %}</td>
   {% endblock %}
 {% endif %}
diff --git a/src/authentic2/manager/templates/authentic2/manager/role_permissions.html b/src/authentic2/manager/templates/authentic2/manager/role_permissions.html
index 95708862..d3562f6e 100644
--- a/src/authentic2/manager/templates/authentic2/manager/role_permissions.html
+++ b/src/authentic2/manager/templates/authentic2/manager/role_permissions.html
@@ -10,10 +10,10 @@
 {% block appbar %}
   {{ block.super }}
   <span class="actions">
-  {% if view.can_delete %}
+  {% if view.could_delete %}
     <a rel="popup" href="{% url "a2-manager-role-delete" pk=object.pk %}">{% trans "Delete" %}</a>
   {% endif %}
-  {% if view.can_change and not object.is_internal %}
+  {% if view.could_change and not object.is_internal %}
     <a href="{% url "a2-manager-role-edit" pk=object.pk %}">{% trans "Edit" %}</a>
   {% endif %}
   <a href="{% url "a2-manager-role-members" pk=object.pk %}">{% trans "Members" %}</a>
diff --git a/src/authentic2/manager/templates/authentic2/manager/role_permissions_table.html b/src/authentic2/manager/templates/authentic2/manager/role_permissions_table.html
index 8675ed9e..88a42a3a 100644
--- a/src/authentic2/manager/templates/authentic2/manager/role_permissions_table.html
+++ b/src/authentic2/manager/templates/authentic2/manager/role_permissions_table.html
@@ -7,6 +7,6 @@
     <th></th>
   {% endblock %}
   {% block table.tbody.last.column %}
-    <td>{% if table.context.view.can_change %}<a class="icon-remove-sign js-remove-object" href="#" data-pk-arg="permission"></a>{% endif %}</td>
+    <td>{% if table.context.view.could_change %}<a class="icon-remove-sign js-remove-object" href="#" data-pk-arg="permission"></a>{% endif %}</td>
   {% endblock %}
 {% endif %}
diff --git a/src/authentic2/manager/templates/authentic2/manager/roles.html b/src/authentic2/manager/templates/authentic2/manager/roles.html
index 57f01e6f..c228a5a3 100644
--- a/src/authentic2/manager/templates/authentic2/manager/roles.html
+++ b/src/authentic2/manager/templates/authentic2/manager/roles.html
@@ -6,7 +6,7 @@
 {% block appbar %}
   {{ block.super }}
   <span class="actions">
-  {% if view.can_add %}
+  {% if view.could_add %}
     <a href="{% url "a2-manager-role-add" %}" rel="popup">{% trans "Add role" %}</a>
   {% else %}
     <a href="#" class="disabled" rel="popup">{% trans "Add role" %}</a>
diff --git a/src/authentic2/manager/templates/authentic2/manager/service_roles_table.html b/src/authentic2/manager/templates/authentic2/manager/service_roles_table.html
index 6b440f58..b3d2f6f1 100644
--- a/src/authentic2/manager/templates/authentic2/manager/service_roles_table.html
+++ b/src/authentic2/manager/templates/authentic2/manager/service_roles_table.html
@@ -7,6 +7,6 @@
     <th></th>
   {% endblock %}
   {% block table.tbody.last.column %}
-    <td>{% if table.context.view.can_change %}<a class="icon-remove-sign js-remove-object" data-confirm="{% blocktrans with name=row.record.name slug=table.context.object %}Do you really want to remove role &quot;{{ name }}&quot; from service &quot;{{ slug }}&quot;&nbsp;?{% endblocktrans %}" href="#" data-pk-arg="role"></a>{% endif %}</td>
+    <td>{% if table.context.view.could_change %}<a class="icon-remove-sign js-remove-object" data-confirm="{% blocktrans with name=row.record.name slug=table.context.object %}Do you really want to remove role &quot;{{ name }}&quot; from service &quot;{{ slug }}&quot;&nbsp;?{% endblocktrans %}" href="#" data-pk-arg="role"></a>{% endif %}</td>
   {% endblock %}
 {% endif %}
diff --git a/src/authentic2/manager/templates/authentic2/manager/user_detail.html b/src/authentic2/manager/templates/authentic2/manager/user_detail.html
index 9a60b94a..661ce730 100644
--- a/src/authentic2/manager/templates/authentic2/manager/user_detail.html
+++ b/src/authentic2/manager/templates/authentic2/manager/user_detail.html
@@ -6,13 +6,13 @@
 {% block appbar %}
   {{ block.super }}
   <span class="actions">
-  {% if view.can_delete %}
+  {% if view.could_delete %}
     <a rel="popup" href="{% url "a2-manager-user-delete" pk=object.pk %}">{% trans "Delete" %}</a>
   {% else %}
     <a class="disabled" title="{% trans "You do not have the rights to delete this user." %}" href="#">{% trans "Delete" %}</a>
   {% endif %}
 
-  {% if view.can_change %}
+  {% if view.could_change %}
     <a href="{% url "a2-manager-user-edit" pk=object.pk %}">{% trans "Edit" %}</a>
   {% else %}
     <a class="disabled" title="{% trans "You do not have the rights to edit this user." %}" href="#">{% trans "Edit" %}</a>
diff --git a/src/authentic2/manager/templates/authentic2/manager/user_roles.html b/src/authentic2/manager/templates/authentic2/manager/user_roles.html
index fa99f710..7320a76a 100644
--- a/src/authentic2/manager/templates/authentic2/manager/user_roles.html
+++ b/src/authentic2/manager/templates/authentic2/manager/user_roles.html
@@ -33,7 +33,7 @@
    {% render_table table "authentic2/manager/user_roles_table.html" %}
  {% endwith %}
 
- {% if view.can_change %}
+ {% if view.could_change %}
    <form method="post" class="manager-m2m-add-form">
            {% csrf_token %}
            {{ form }}
diff --git a/src/authentic2/manager/templates/authentic2/manager/user_roles_table.html b/src/authentic2/manager/templates/authentic2/manager/user_roles_table.html
index d173a54f..eebb9dbb 100644
--- a/src/authentic2/manager/templates/authentic2/manager/user_roles_table.html
+++ b/src/authentic2/manager/templates/authentic2/manager/user_roles_table.html
@@ -7,6 +7,6 @@
     <th></th>
   {% endblock %}
   {% block table.tbody.last.column %}
-  <td>{% if table.context.view.can_change and row.record.member %}<a class="icon-remove-sign js-remove-object" data-confirm="{% blocktrans with name=row.record.name username=table.context.object.get_full_name %}Do you really want to remove role &quot;{{ name }}&quot; from user &quot;{{ username }}&quot;&nbsp;?{% endblocktrans %}" href="#" data-pk-arg="role"></a>{% endif %}</td>
+  <td>{% if table.context.view.could_change and row.record.member %}<a class="icon-remove-sign js-remove-object" data-confirm="{% blocktrans with name=row.record.name username=table.context.object.get_full_name %}Do you really want to remove role &quot;{{ name }}&quot; from user &quot;{{ username }}&quot;&nbsp;?{% endblocktrans %}" href="#" data-pk-arg="role"></a>{% endif %}</td>
   {% endblock %}
 {% endif %}
-- 
2.20.1