Enable/ disable revert using only a settings attribute (#632)

* show revert text / buttons based on value of revert_disabled.

* add tests for revert disabled functionality

* add SIMPLE_HISTORY_REVERT_DISABLED as attr to control visibility of Revert button.

* capture --tag from args

* correct variable name to revert_disabled

* update docs

* CHANGES.rst

* remove unused test tags

* fix minor formatting issues in templates

* fix, only show text if revert is NOT disabled

* use random file name in tests (uuid)

Author: erikvw

.gitignore

@@ -16,3 +16,4 @@ htmlcov/
 MANIFEST
 test_files/
 venv/
+.DS_Store

CHANGES.rst

@@ -9,6 +9,8 @@ Unreleased
 - import model `ContentType` in `SimpleHistoryAdmin` using `django_apps.get_model`
   to avoid possible `AppRegistryNotReady` exception (gh-630)
 - Fix `utils.update_change_reason` when user specifies excluded_fields
+- settings.SIMPLE_HISTORY_REVERT_DISABLED if True removes the Revert
+  button from the history form for all historical models (gh-632))
 
 2.8.0 (2019-12-02)
 ------------------

docs/admin.rst

@@ -68,3 +68,16 @@ admin class
 
 
 .. image:: screens/5_history_list_display.png
+
+Disabling the option to revert an object
+~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
+
+By default, an object can be reverted to its previous version. To disable this option update your settings with the following:
+
+.. code-block:: python
+
+    SIMPLE_HISTORY_REVERT_DISABLED=True
+
+When ``SIMPLE_HISTORY_REVERT_DISABLED`` is set to ``True``, the revert button is removed from the form.
+
+.. image:: screens/10_revert_disabled.png

docs/screens/10_revert_disabled.png

@@ -73,12 +73,14 @@ def __getitem__(self, item):
 
 
 def main():
-
     if not settings.configured:
         settings.configure(**DEFAULT_SETTINGS)
     django.setup()
-    failures = DiscoverRunner(failfast=False).run_tests(["simple_history.tests"])
-    failures |= DiscoverRunner(failfast=False).run_tests(
+    tags = [t.split("=")[1] for t in sys.argv if t.startswith("--tag")]
+    failures = DiscoverRunner(failfast=False, tags=tags).run_tests(
+        ["simple_history.tests"]
+    )
+    failures |= DiscoverRunner(failfast=False, tags=tags).run_tests(
         ["simple_history.registry_tests"]
     )
     sys.exit(failures)

runtests.py

@@ -88,7 +88,7 @@ def history_view(self, request, object_id, extra_context=None):
             content_type.model,
         )
         context = {
-            "title": _("Change history: %s") % force_str(obj),
+            "title": self.history_view_title(obj),
             "action_list": action_list,
             "module_name": capfirst(force_str(opts.verbose_name_plural)),
             "object": obj,
@@ -97,6 +97,7 @@ def history_view(self, request, object_id, extra_context=None):
             "opts": opts,
             "admin_user_view": admin_user_view,
             "history_list_display": history_list_display,
+            "revert_disabled": self.revert_disabled,
         }
         context.update(self.admin_site.each_context(request))
         context.update(extra_context or {})
@@ -105,6 +106,12 @@ def history_view(self, request, object_id, extra_context=None):
             request, self.object_history_template, context, **extra_kwargs
         )
 
+    def history_view_title(self, obj):
+        if self.revert_disabled and not SIMPLE_HISTORY_EDIT:
+            return _("View history: %s") % force_str(obj)
+        else:
+            return _("Change history: %s") % force_str(obj)
+
     def response_change(self, request, obj):
         if "_change_history" in request.POST and SIMPLE_HISTORY_EDIT:
             verbose_name = obj._meta.verbose_name
@@ -175,7 +182,7 @@ def history_form_view(self, request, object_id, version_id, extra_context=None):
         model_name = original_opts.model_name
         url_triplet = self.admin_site.name, original_opts.app_label, model_name
         context = {
-            "title": _("Revert %s") % force_str(obj),
+            "title": self.history_form_view_title(obj),
             "adminform": admin_form,
             "object_id": object_id,
             "original": obj,
@@ -188,6 +195,7 @@ def history_form_view(self, request, object_id, version_id, extra_context=None):
             "change_url": reverse("%s:%s_%s_change" % url_triplet, args=(obj.pk,)),
             "history_url": reverse("%s:%s_%s_history" % url_triplet, args=(obj.pk,)),
             "change_history": change_history,
+            "revert_disabled": self.revert_disabled,
             # Context variables copied from render_change_form
             "add": False,
             "change": True,
@@ -212,6 +220,12 @@ def history_form_view(self, request, object_id, version_id, extra_context=None):
             request, self.object_history_form_template, context, **extra_kwargs
         )
 
+    def history_form_view_title(self, obj):
+        if self.revert_disabled:
+            return _("View %s") % force_str(obj)
+        else:
+            return _("Revert %s") % force_str(obj)
+
     def render_history_view(self, request, template, context, **kwargs):
         """Catch call to render, to allow overriding."""
         return render(request, template, context, **kwargs)
@@ -226,3 +240,7 @@ def content_type_model_cls(self):
         """Returns the ContentType model class.
         """
         return django_apps.get_model("contenttypes.contenttype")
+
+    @property
+    def revert_disabled(self):
+        return getattr(settings, "SIMPLE_HISTORY_REVERT_DISABLED", False)

simple_history/admin.py

@@ -7,9 +7,8 @@
 
 {% block content %}
   <div id="content-main">
-
-    <p>{% blocktrans %}Choose a date from the list below to revert to a previous version of this object.{% endblocktrans %}</p>
-
+    {% if not revert_disabled %}<p>
+      {% blocktrans %}Choose a date from the list below to revert to a previous version of this object.{% endblocktrans %}</p>{% endif %}
     <div class="module">
       {% if action_list %}
         {% display_list %}
@@ -19,4 +18,3 @@
     </div>
   </div>
 {% endblock %}
-

simple_history/templates/simple_history/object_history.html

@@ -9,7 +9,7 @@
     <a href="{{changelist_url}}">{{opts.verbose_name_plural|capfirst}}</a> &rsaquo;
     <a href="{{change_url}}">{{original|truncatewords:"18"}}</a> &rsaquo;
     <a href="../">{% trans "History" %}</a> &rsaquo;
-    {% blocktrans with original_opts.verbose_name as verbose_name %}Revert {{verbose_name}}{% endblocktrans %}
+    {% if revert_disabled %}{% blocktrans with original_opts.verbose_name as verbose_name %}View {{verbose_name}}{% endblocktrans %}{% else %}{% blocktrans with original_opts.verbose_name as verbose_name %}Revert {{verbose_name}}{% endblocktrans %}{% endif %}
   </div>
 {% endblock %}
 
@@ -18,5 +18,5 @@
 {% endblock %}
 
 {% block form_top %}
-<p>{% blocktrans %}Press the 'Revert' button below to revert to this version of the object.{% endblocktrans %} {% if change_history %}{% blocktrans %}Or press the 'Change History' button to edit the history.{% endblocktrans %}{% endif %}</p>
+<p>{% if not revert_disabled %}{% blocktrans %}Press the 'Revert' button below to revert to this version of the object. {% endblocktrans %}{% endif %}{% if change_history %}{% blocktrans %}Press the 'Change History' button below to edit the history.{% endblocktrans %}{% endif %}</p>
 {% endblock %}

simple_history/templates/simple_history/object_history_form.html

@@ -1,5 +1,8 @@
 {% load i18n %}
 <div class="submit-row">
-<input type="submit" value="{% trans 'Revert' %}" class="default" name="_save" {{ onclick_attrib }}/>
-{% if change_history %}<input type="submit" value="{% trans 'Change History' %}" class="default" name="_change_history" {{ onclick_attrib }}/>{% endif %}
+  {% if not revert_disabled %}
+    <input type="submit" value="{% trans 'Revert' %}" class="default" name="_save" {{ onclick_attrib }}/>{% endif %}
+  {% if change_history %}
+    <input type="submit" value="{% trans 'Change History' %}" class="default" name="_change_history" {{ onclick_attrib }}/>{% endif %}
+  <a href="{{ history_url }}" class="closelink">{% trans 'Close' %}</a>
 </div>

simple_history/templates/simple_history/submit_line.html

@@ -13,6 +13,7 @@
     FileModel,
     Paper,
     Person,
+    Planet,
     Poll,
 )
 
@@ -33,6 +34,13 @@ def test_method(self, obj):
     history_list_display = ["title", "test_method"]
 
 
+class PlanetAdmin(SimpleHistoryAdmin):
+    def test_method(self, obj):
+        return "test_method_value"
+
+    history_list_display = ["title", "test_method"]
+
+
 admin.site.register(Poll, SimpleHistoryAdmin)
 admin.site.register(Choice, ChoiceAdmin)
 admin.site.register(Person, PersonAdmin)
@@ -43,3 +51,4 @@ def test_method(self, obj):
 admin.site.register(ConcreteExternal, SimpleHistoryAdmin)
 admin.site.register(ExternalModelWithCustomUserIdField, SimpleHistoryAdmin)
 admin.site.register(FileModel, FileModelAdmin)
+admin.site.register(Planet, PlanetAdmin)