Fixed url issue where user could not access history for concrete instance of historical abstract base class

Author: Ross Mechanic

simple_history/admin.py

@@ -162,7 +162,7 @@ def history_form_view(self, request, object_id, version_id):
             'is_popup': False,
             'media': mark_safe(self.media + admin_form.media),
             'errors': helpers.AdminErrorList(form, formsets),
-            'app_label': model._meta.app_label,
+            'app_label': original_opts.app_label,
             'original_opts': original_opts,
             'changelist_url': reverse('%s:%s_%s_changelist' % url_triplet),
             'change_url': reverse('%s:%s_%s_change' % url_triplet,

simple_history/tests/admin.py

@@ -3,7 +3,7 @@
 from django.contrib import admin
 
 from simple_history.admin import SimpleHistoryAdmin
-from .models import Poll, Choice, Person, Book, Document, Paper, Employee
+from .models import Poll, Choice, Person, Book, Document, Paper, Employee, TrackedConcreteBaseExternal
 
 
 class PersonAdmin(SimpleHistoryAdmin):
@@ -22,3 +22,4 @@ class ChoiceAdmin(SimpleHistoryAdmin):
 admin.site.register(Document, SimpleHistoryAdmin)
 admin.site.register(Paper, SimpleHistoryAdmin)
 admin.site.register(Employee, SimpleHistoryAdmin)
+admin.site.register(TrackedConcreteBaseExternal, SimpleHistoryAdmin)

simple_history/tests/external/models/model2.py

@@ -10,3 +10,11 @@ class ExternalModel2(models.Model):
 
     class Meta:
         app_label = 'external'
+
+
+class TrackedAbstractBaseExternal(models.Model):
+    history = HistoricalRecords(inherit=True)
+
+    class Meta:
+        abstract = True
+        app_label = 'external'

simple_history/tests/models.py

@@ -6,7 +6,7 @@
 from simple_history import register
 
 from .custom_user.models import CustomUser as User
-
+from .external.models.model2 import TrackedAbstractBaseExternal
 
 class Poll(models.Model):
     question = models.CharField(max_length=200)
@@ -341,6 +341,13 @@ class UntrackedConcreteBase(models.Model):
     pass
 
 
+class TrackedConcreteBaseExternal(TrackedAbstractBaseExternal):
+    name = models.CharField(max_length=50)
+
+    class Meta:
+        app_label = 'tests'
+
+
 class TrackedWithAbstractBase(TrackedAbstractBaseA):
     pass
 

simple_history/tests/tests/test_admin.py

@@ -13,7 +13,7 @@
 
 from simple_history.models import HistoricalRecords
 from simple_history.admin import SimpleHistoryAdmin, get_complete_version
-from ..models import Book, Person, Poll, State, Employee, Choice
+from ..models import Book, Person, Poll, State, Employee, Choice, TrackedConcreteBaseExternal
 
 try:
     from django.contrib.admin.utils import quote
@@ -551,3 +551,58 @@ def test_history_form_view_getting_history_with_setting_off(self):
         }
         mock_render.assert_called_once_with(
             request, admin.object_history_form_template, context, **extra_kwargs)
+
+    def test_history_form_view_getting_history_abstract_external(self):
+        request = RequestFactory().post('/')
+        request.session = 'session'
+        request._messages = FallbackStorage(request)
+        request.user = self.user
+        request.POST = {'_change_history': True}
+
+        obj = TrackedConcreteBaseExternal.objects.create(name='test')
+        obj.name = "new_test"
+        obj.save()
+        history = obj.history.all()[0]
+
+        admin_site = AdminSite()
+        admin = SimpleHistoryAdmin(TrackedConcreteBaseExternal, admin_site)
+
+        with patch('simple_history.admin.render') as mock_render:
+            with patch('simple_history.admin.SIMPLE_HISTORY_EDIT', True):
+                admin.history_form_view(request, obj.id, history.pk)
+
+        context = {
+            # Verify this is set for history object not poll object
+            'original': history.instance,
+            'change_history': True,
+
+            'title': 'Revert %s' % force_text(history.instance),
+            'adminform': ANY,
+            'object_id': obj.id,
+            'is_popup': False,
+            'media': ANY,
+            'errors': ANY,
+            'app_label': 'tests',
+            'original_opts': ANY,
+            'changelist_url': '/admin/tests/trackedconcretebaseexternal/',
+            'change_url': ANY,
+            'history_url': '/admin/tests/trackedconcretebaseexternal/{pk}/history/'.format(
+                pk=obj.pk),
+            'add': False,
+            'change': True,
+            'has_add_permission': admin.has_add_permission(request),
+            'has_change_permission': admin.has_change_permission(
+                request, obj),
+            'has_delete_permission': admin.has_delete_permission(
+                request, obj),
+            'has_file_field': True,
+            'has_absolute_url': False,
+            'form_url': '',
+            'opts': ANY,
+            'content_type_id': ANY,
+            'save_as': admin.save_as,
+            'save_on_top': admin.save_on_top,
+            'root_path': getattr(admin_site, 'root_path', None),
+        }
+        mock_render.assert_called_once_with(
+            request, admin.object_history_form_template, context, **extra_kwargs)