Merge pull request #197 from treyhunner/django19

Add Django 1.9 support

Author: macro1

.editorconfig

@@ -8,6 +8,7 @@ charset = utf-8
 indent_style = space
 end_of_line = lf
 insert_final_newline = true
+max_line_length = 79
 trim_trailing_whitespace = true
 
 [*.{py,rst,ini}]

.travis.yml

@@ -3,7 +3,6 @@ language: python
 sudo: false
 
 python:
-  - 2.6
   - 2.7
   - 3.2
   - 3.3
@@ -13,30 +12,36 @@ env:
   - DJANGO="Django>=1.6,<1.7"
   - DJANGO="Django>=1.7,<1.8"
   - DJANGO="Django>=1.8,<1.9"
+  - DJANGO="Django>=1.9a,<1.10"
 
 install:
-  - pip install -U 'coverage<4' codecov $DJANGO
+  - pip install -U 'coverage<4' codecov
+  - pip install -U --pre $DJANGO
   - python -c 'from __future__ import print_function; import django; print("Django " + django.get_version())'
 
 script: coverage run setup.py test
 
 matrix:
   exclude:
-   - python: 2.6
-     env: DJANGO="Django>=1.6,<1.7"
-   - python: 2.6
-     env: DJANGO="Django>=1.7,<1.8"
-   - python: 2.6
-     env: DJANGO="Django>=1.8,<1.9"
    - python: 3.2
      env: DJANGO="Django>=1.4,<1.5"
    - python: 3.3
      env: DJANGO="Django>=1.4,<1.5"
+   - python: 3.2
+     env: DJANGO="Django>=1.9a,<1.10"
+   - python: 3.3
+     env: DJANGO="Django>=1.9a,<1.10"
 
   include:
+   - python: 2.6
+     env: DJANGO="Django>=1.4,<1.5"
    - python: 3.4
      env: DJANGO="Django>=1.7,<1.8"
    - python: 3.4
      env: DJANGO="Django>=1.8,<1.9"
+   - python: 3.5
+     env: DJANGO="Django>=1.8,<1.9"
+   - python: 3.5
+     env: DJANGO="Django>=1.9a,<1.10"
 
 after_success: codecov

CHANGES.rst

@@ -5,6 +5,7 @@ tip (unreleased)
 ----------------
 - Add ability to list history in admin when the object instance is deleted. (gh-72)
 - Add ability to change history through the admin. (Enabled with the `SIMPLE_HISTORY_EDIT` setting.)
+- Add Django 1.9 support.
 
 1.6.3 (2015-07-30)
 ------------------

README.rst

@@ -20,7 +20,7 @@ django-simple-history
 
 django-simple-history stores Django model state on every create/update/delete.
 
-This app requires Django 1.4.2 or greater and Python 2.6 or greater.
+This app requires Django 1.4.2, 1.6, or greater and Python 2.6 or greater.
 
 Getting Help
 ------------

docs/conf.py

@@ -95,7 +95,7 @@
 
 # The theme to use for HTML and HTML Help pages.  See the documentation for
 # a list of builtin themes.
-html_theme = 'default'
+# html_theme = 'default'
 
 # Theme options are theme-specific and customize the look and feel of a theme
 # further.  For a list of options available for each theme, see the

docs/usage.rst

@@ -43,7 +43,7 @@ settings:
     ]
 
 If you do not want to use the middleware, you can explicitly indicate
-the user making the change as documented in :ref:`recording_user`.
+the user making the change as documented in :doc:`/advanced`.
 
 Models
 ~~~~~~
@@ -121,7 +121,7 @@ An example of admin integration for the ``Poll`` and ``Choice`` models:
     admin.site.register(Poll, SimpleHistoryAdmin)
     admin.site.register(Choice, SimpleHistoryAdmin)
 
-Changing a history-tracked model from the admin interface will automatically record the user who made the change (see :ref:`recording_user`).
+Changing a history-tracked model from the admin interface will automatically record the user who made the change (see :doc:`/advanced`).
 
 
 Querying history

runtests.py

@@ -1,15 +1,15 @@
 #!/usr/bin/env python
-import sys
-from shutil import rmtree
+import logging
 from os.path import abspath, dirname, join
+from shutil import rmtree
+import sys
+import warnings
 
 import django
 from django.conf import settings
 
-
 sys.path.insert(0, abspath(dirname(__file__)))
 
-
 media_root = join(abspath(dirname(__file__)), 'test_files')
 rmtree(media_root, ignore_errors=True)
 
@@ -39,6 +39,10 @@
         'django.contrib.auth.middleware.AuthenticationMiddleware',
         'django.contrib.messages.middleware.MessageMiddleware',
     ],
+    TEMPLATES=[{
+        'BACKEND': 'django.template.backends.django.DjangoTemplates',
+        'APP_DIRS': True,
+    }],
 )
 
 if django.VERSION >= (1, 5):
@@ -62,4 +66,5 @@ def main():
 
 
 if __name__ == "__main__":
+    logging.basicConfig()
     main()

setup.py

@@ -2,8 +2,7 @@
 import simple_history
 
 tests_require = [
-    "Django>=1.4", "webtest==2.0.6", "django-webtest==1.7", "mock==1.0.1"
-]
+    'Django>=1.4', 'WebTest==2.0.18', 'django-webtest==1.7.8', 'mock==1.0.1']
 try:
     from unittest import skipUnless
 except ImportError:  # Python 2.6 compatibility
@@ -21,7 +20,9 @@
     author_email='corey@qr7.com',
     maintainer='Trey Hunner',
     url='https://github.com/treyhunner/django-simple-history',
-    packages=["simple_history", "simple_history.management", "simple_history.management.commands"],
+    packages=[
+        'simple_history', 'simple_history.management',
+        'simple_history.management.commands', 'simple_history.templatetags'],
     classifiers=[
         "Development Status :: 5 - Production/Stable",
         "Framework :: Django",

simple_history/admin.py

@@ -2,8 +2,9 @@
 
 from django import http
 from django.core.exceptions import PermissionDenied
-from django.conf.urls import patterns, url
-from django.contrib.admin import helpers, ModelAdmin
+from django.conf.urls import url
+from django.contrib import admin
+from django.contrib.admin import helpers
 from django.contrib.contenttypes.models import ContentType
 from django.core.urlresolvers import reverse
 from django.shortcuts import get_object_or_404, render
@@ -27,7 +28,7 @@
 SIMPLE_HISTORY_EDIT = getattr(settings, 'SIMPLE_HISTORY_EDIT', False)
 
 
-class SimpleHistoryAdmin(ModelAdmin):
+class SimpleHistoryAdmin(admin.ModelAdmin):
     object_history_template = "simple_history/object_history.html"
     object_history_form_template = "simple_history/object_history_form.html"
 
@@ -40,16 +41,16 @@ def get_urls(self):
             info = opts.app_label, opts.model_name
         except AttributeError:  # Django < 1.7
             info = opts.app_label, opts.module_name
-        history_urls = patterns(
-            "",
+        history_urls = [
             url("^([^/]+)/history/([^/]+)/$",
                 admin_site.admin_view(self.history_form_view),
                 name='%s_%s_simple_history' % info),
-        )
+        ]
         return history_urls + urls
 
     def history_view(self, request, object_id, extra_context=None):
-        "The 'history' admin view for this model."
+        """The 'history' admin view for this model."""
+        request.current_app = self.admin_site.name
         model = self.model
         opts = model._meta
         app_label = opts.app_label
@@ -81,7 +82,7 @@ def history_view(self, request, object_id, extra_context=None):
         }
         context.update(extra_context or {})
         return render(request, template_name=self.object_history_template,
-                      dictionary=context, current_app=self.admin_site.name)
+                      dictionary=context, current_app=request.current_app)
 
     def response_change(self, request, obj):
         if '_change_history' in request.POST and SIMPLE_HISTORY_EDIT:
@@ -101,6 +102,7 @@ def response_change(self, request, obj):
                 request, obj)
 
     def history_form_view(self, request, object_id, version_id):
+        request.current_app = self.admin_site.name
         original_opts = self.model._meta
         model = getattr(
             self.model,
@@ -120,7 +122,7 @@ def history_form_view(self, request, object_id, version_id):
             change_history = False
 
         if '_change_history' in request.POST and SIMPLE_HISTORY_EDIT:
-            obj = obj.history.get(pk=version_id)
+            obj = obj.history.get(pk=version_id).instance
 
         formsets = []
         form_class = self.get_form(request, obj)
@@ -186,7 +188,7 @@ def history_form_view(self, request, object_id, version_id):
             'root_path': getattr(self.admin_site, 'root_path', None),
         }
         return render(request, template_name=self.object_history_form_template,
-                      dictionary=context, current_app=self.admin_site.name)
+                      dictionary=context, current_app=request.current_app)
 
     def save_model(self, request, obj, form, change):
         """Set special model attribute to user for reference after save"""

simple_history/models.py

@@ -16,17 +16,14 @@
 from django.utils.timezone import now
 from django.utils.translation import string_concat
 
-try:
-    import importlib
-except ImportError:
-    from django.utils import importlib
 try:
     from django.apps import apps
 except ImportError:  # Django < 1.7
-    from django.db.models.loading import get_app
-    apps = None
-else:
-    get_app = apps.get_app
+    from django.db.models import get_app
+try:
+    import importlib
+except ImportError:  # Python < 2.7
+    from django.utils import importlib
 try:
     from south.modelsinspector import add_introspection_rules
 except ImportError:  # south not present
@@ -103,14 +100,14 @@ def create_history_model(self, model):
             # registered under different app
             attrs['__module__'] = self.module
         elif app_module != self.module:
-            if apps is None:  # Django < 1.7
-                # has meta options with app_label
-                app = get_app(model._meta.app_label)
-                attrs['__module__'] = app.__name__  # full dotted name
-            else:
+            try:
                 # Abuse an internal API because the app registry is loading.
                 app = apps.app_configs[model._meta.app_label]
-                attrs['__module__'] = app.name      # full dotted name
+            except NameError:  # Django < 1.7
+                models_module = get_app(model._meta.app_label).__name__
+            else:
+                models_module = app.name
+            attrs['__module__'] = models_module
 
         fields = self.copy_fields(model)
         attrs.update(fields)
@@ -130,7 +127,10 @@ def copy_fields(self, model):
         fields = {}
         for field in model._meta.fields:
             field = copy.copy(field)
-            field.rel = copy.copy(field.rel)
+            try:
+                field.remote_field = copy.copy(field.remote_field)
+            except AttributeError:
+                field.rel = copy.copy(field.rel)
             if isinstance(field, OrderWrt):
                 # OrderWrt is a proxy field, switch to a plain IntegerField
                 field.__class__ = models.IntegerField
@@ -146,7 +146,8 @@ def copy_fields(self, model):
                     field_arguments['db_constraint'] = False
                 if getattr(old_field, 'to_fields', []):
                     field_arguments['to_field'] = old_field.to_fields[0]
-                elif django.get_version() < "1.6" and old_field.rel.field_name != 'id':
+                elif (django.get_version() < "1.6" and
+                      old_field.rel.field_name != 'id'):
                     field_arguments['to_field'] = old_field.rel.field_name
                 if getattr(old_field, 'db_column', None):
                     field_arguments['db_column'] = old_field.db_column