Passage par flags plutôt que par valeurs

cotisations/templates/cotisations/aff_cotisations.html

@@ -87,7 +87,7 @@ with this program; if not, write to the Free Software Foundation, Inc.,
                         </li>
                         {% acl_end %}
                         <li>
-                            {% history_button facture text='History' html_class=''%}
+                            {% history_button facture text=True html_class=False%}
                         </li>
                     </ul>
                 </div>

logs/templatetags/logs_extra.py

@@ -36,8 +36,16 @@ def classname(obj):
 
 
 @register.inclusion_tag('buttons/history.html')
-def history_button(instance, text=None, html_class=None):
-    """Creates the correct history button for an instance."""
+def history_button(instance, text=False, html_class=True):
+    """Creates the correct history button for an instance.
+
+    Args:
+        instance: The instance of which you want to get history buttons.
+        text: Flag stating if a 'History' text should be displayed.
+        html_class: Flag stating if the link should have the html classes
+            allowing it to be displayed as a button.
+
+    """
     return {
         'application': instance._meta.app_label,
         'name': instance._meta.model_name,

templates/buttons/history.html

@@ -22,7 +22,7 @@ with this program; if not, write to the Free Software Foundation, Inc.,
 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
 {% endcomment %}
 {% load i18n %}
-<a class="{{ class |default_if_none:"btn btn-info btn-sm"}}" role="button" title="Historique" href="{% url 'logs:history' application name id %}">
-	<i class="fa fa-history"></i> {% if text %}{% trans text %}{% endif %}
+<a {% if class%}class="btn btn-info btn-sm"{% endif %} role="button" title="Historique" href="{% url 'logs:history' application name id %}">
+    <i class="fa fa-history"></i> {% if text %}{% trans 'History' %}{% endif %}
 </a>
 

users/templates/users/profil.html

@@ -71,7 +71,7 @@ with this program; if not, write to the Free Software Foundation, Inc.,
                     Gérer les groupes
                 </a>
                 {% acl_end %}
-                {% history_button users text="History" %}
+                {% history_button users text=True %}
                 </ul>
             </div>
             <div class="panel-body">