Convert to wagtail 2

CHANGELOG.rst

 CHANGELOG
 =========
 
+2.0.0
+-----
+
+- Dropped python2
+- Updated wagtail dependency to 2.0
+
 0.4.0
 -----
 

README.rst

@@ -8,7 +8,7 @@ html5 compliant codec using ffmpeg.
 Requirements
 ------------
 
--  Wagtail > 1.7
+-  Wagtail > 2.0
 -  `ffmpeg <https://ffmpeg.org/>`__
 
 Installing

setup.py

@@ -18,8 +18,8 @@ setup(
     url='https://github.com/takeflight/wagtailvideos',
 
     install_requires=[
-        'wagtail>=1.7',
-        'Django>=1.9',
+        'wagtail>=2.0',
+        'Django>=2.0',
         'django-enumchoicefield==0.6.0',
     ],
     extras_require={

tests/app/models.py

 from __future__ import unicode_literals
 
 from django.db import models
-from wagtail.wagtailcore.models import Page
+from wagtail.core.models import Page
 
 from wagtailvideos.edit_handlers import VideoChooserPanel
 

tests/app/settings.py

@@ -9,12 +9,12 @@ INSTALLED_APPS = [
     'taggit',
     'modelcluster',
 
-    'wagtail.wagtailcore',
-    'wagtail.wagtailadmin',
-    'wagtail.wagtailusers',
-    'wagtail.wagtailsites',
-    'wagtail.wagtailsnippets',
-    'wagtail.wagtailimages',
+    'wagtail.core',
+    'wagtail.admin',
+    'wagtail.users',
+    'wagtail.sites',
+    'wagtail.snippets',
+    'wagtail.images',
 
     'django.contrib.admin',
     'django.contrib.auth',
@@ -49,7 +49,7 @@ MIDDLEWARE_CLASSES = [
     'django.contrib.messages.middleware.MessageMiddleware',
     'django.middleware.clickjacking.XFrameOptionsMiddleware',
 
-    'wagtail.wagtailcore.middleware.SiteMiddleware',
+    'wagtail.core.middleware.SiteMiddleware',
 ]
 
 TEMPLATES = [

tests/app/urls.py

@@ -5,8 +5,8 @@ import re
 from django.conf import settings
 from django.conf.urls import include, url
 from django.views.static import serve
-from wagtail.wagtailadmin import urls as wagtailadmin_urls
-from wagtail.wagtailcore import urls as wagtail_urls
+from wagtail.admin import urls as wagtailadmin_urls
+from wagtail.core import urls as wagtail_urls
 
 urlpatterns = [
     url(r'^admin/', include(wagtailadmin_urls)),

tests/storage.py

@@ -145,33 +145,6 @@ class RemoteStorage(FileSystemStorage):
             url = url.lstrip('/')
         return urljoin(self.base_url, url)
 
-    def accessed_time(self, name):
-        warnings.warn(
-            'FileSystemStorage.accessed_time() is deprecated in favor of '
-            'get_accessed_time().',
-            RemovedInDjango20Warning,
-            stacklevel=2,
-        )
-        return datetime.fromtimestamp(os.path.getatime(self.path(name)))
-
-    def created_time(self, name):
-        warnings.warn(
-            'FileSystemStorage.created_time() is deprecated in favor of '
-            'get_created_time().',
-            RemovedInDjango20Warning,
-            stacklevel=2,
-        )
-        return datetime.fromtimestamp(os.path.getctime(self.path(name)))
-
-    def modified_time(self, name):
-        warnings.warn(
-            'FileSystemStorage.modified_time() is deprecated in favor of '
-            'get_modified_time().',
-            RemovedInDjango20Warning,
-            stacklevel=2,
-        )
-        return datetime.fromtimestamp(os.path.getmtime(self.path(name)))
-
     def _datetime_from_timestamp(self, ts):
         """
         If timezone support is enabled, make an aware datetime object in UTC;

tests/test_admin_views.py

@@ -5,12 +5,12 @@ import json
 from django.contrib.auth import get_user_model
 from django.contrib.auth.models import Group, Permission
 from django.core.files.uploadedfile import SimpleUploadedFile
-from django.core.urlresolvers import reverse
+from django.urls import reverse
 from django.template.defaultfilters import filesizeformat
 from django.test import TestCase, override_settings
 from mock import patch
 from wagtail.tests.utils import WagtailTestUtils
-from wagtail.wagtailcore.models import Collection, GroupCollectionPermission
+from wagtail.core.models import Collection, GroupCollectionPermission
 
 from tests.utils import create_test_video_file
 from wagtailvideos.models import Video

tox.ini

@@ -2,8 +2,7 @@
 skip_missing_interpreters = True
 
 envlist =
-	py{27,34,35}-dj{110}-wt{19}
-	py{27,34,35}-dj{110,111}-wt{110}
+	py{34,35,36}-dj{20}-wt{20}
 	# Enforce good style
 	flake8,isort
 
@@ -16,14 +15,8 @@ pip_pre = True
 
 deps =
 	{[base]deps}
-	dj18: Django~=1.8.0
-	dj19: Django~=1.9.0
-	dj110: Django~=1.10.0
-	dj111: Django~=1.11.0
-	wt17: wagtail~=1.7.0
-	wt18: wagtail~=1.8.0
-	wt19: wagtail~=1.9.0
-	wt110: wagtail~=1.10.0
+	dj20: Django~=2.0.0
+	wt20: wagtail~=2.0.0
 
 [testenv:flake8]
 deps = flake8

wagtailvideos/apps.py

-from __future__ import absolute_import, print_function, unicode_literals
-
 from django.apps import AppConfig
 from django.core.checks import Warning, register
 

wagtailvideos/edit_handlers.py

-from __future__ import absolute_import, print_function, unicode_literals
-
-from wagtail.wagtailadmin.edit_handlers import BaseChooserPanel
+from wagtail.admin.edit_handlers import BaseChooserPanel
 
 from .widgets import AdminVideoChooser
 
 
-class BaseVideoChooserPanel(BaseChooserPanel):
+class VideoChooserPanel(BaseChooserPanel):
+    model = None
+    field_name = None
+    _target_model = None
+
     object_type_name = "video"
 
     @classmethod
     def widget_overrides(cls):
         return {cls.field_name: AdminVideoChooser}
-
-
-class VideoChooserPanel(object):
-    def __init__(self, field_name):
-        self.field_name = field_name
-
-    def bind_to_model(self, model):
-        return type(str('_VideoChooserPanel'), (BaseVideoChooserPanel,), {
-            'model': model,
-            'field_name': self.field_name,
-        })

wagtailvideos/ffmpeg.py

-from __future__ import absolute_import, print_function, unicode_literals
-
 import datetime
 import logging
 import os

wagtailvideos/fields.py

-from __future__ import absolute_import, print_function, unicode_literals
-
 from django.conf import settings
 from django.core.exceptions import ValidationError
 from django.forms.fields import FileField

wagtailvideos/forms.py

-from __future__ import absolute_import, print_function, unicode_literals
-
 from django import forms
 from django.forms.models import modelform_factory
 from django.utils.translation import ugettext as _
 from enumchoicefield.forms import EnumField
-from wagtail.wagtailadmin import widgets
-from wagtail.wagtailadmin.forms import (
+from wagtail.admin import widgets
+from wagtail.admin.forms import (
     BaseCollectionMemberForm, collection_member_permission_formset_factory)
 
 from wagtailvideos.fields import WagtailVideoField

wagtailvideos/jinja2tags.py

-from __future__ import absolute_import, unicode_literals
-
 from jinja2.ext import Extension
 
 from .models import Video

wagtailvideos/migrations/0001_initial.py

@@ -4,8 +4,8 @@ from __future__ import absolute_import, print_function, unicode_literals
 
 import django.db.models.deletion
 import taggit.managers
-import wagtail.wagtailcore.models
-import wagtail.wagtailsearch.index
+import wagtail.core.models
+import wagtail.search.index
 from django.conf import settings
 from django.db import migrations, models
 
@@ -31,7 +31,7 @@ class Migration(migrations.Migration):
                 ('file', models.ImageField(height_field='height', upload_to=wagtailvideos.models.get_upload_to, verbose_name='file', width_field='width')),
                 ('created_at', models.DateTimeField(auto_now_add=True, db_index=True, verbose_name='created at')),
                 ('file_size', models.PositiveIntegerField(editable=False, null=True)),
-                ('collection', models.ForeignKey(default=wagtail.wagtailcore.models.get_root_collection_id, on_delete=django.db.models.deletion.CASCADE, related_name='+', to='wagtailcore.Collection', verbose_name='collection')),
+                ('collection', models.ForeignKey(default=wagtail.core.models.get_root_collection_id, on_delete=django.db.models.deletion.CASCADE, related_name='+', to='wagtailcore.Collection', verbose_name='collection')),
                 ('tags', taggit.managers.TaggableManager(blank=True, help_text=None, through='taggit.TaggedItem', to='taggit.Tag', verbose_name='tags')),
                 ('uploaded_by_user', models.ForeignKey(blank=True, editable=False, null=True, on_delete=django.db.models.deletion.SET_NULL, to=settings.AUTH_USER_MODEL, verbose_name='uploaded by user')),
             ],

wagtailvideos/models.py

-from __future__ import absolute_import, print_function, unicode_literals
-
 import logging
 import mimetypes
 import os
@@ -14,7 +12,7 @@ from django.conf import settings
 from django.core.exceptions import SuspiciousFileOperation
 from django.core.files.base import ContentFile
 from django.core.files.temp import NamedTemporaryFile
-from django.core.urlresolvers import reverse
+from django.urls import reverse
 from django.db import models
 from django.db.models.signals import post_save, pre_delete
 from django.dispatch.dispatcher import receiver
@@ -24,10 +22,10 @@ from django.utils.text import mark_safe
 from django.utils.translation import ugettext_lazy as _
 from enumchoicefield import ChoiceEnum, EnumChoiceField
 from taggit.managers import TaggableManager
-from wagtail.wagtailadmin.utils import get_object_usage
-from wagtail.wagtailcore.models import CollectionMember
-from wagtail.wagtailsearch import index
-from wagtail.wagtailsearch.queryset import SearchableQuerySetMixin
+from wagtail.admin.utils import get_object_usage
+from wagtail.core.models import CollectionMember
+from wagtail.search import index
+from wagtail.search.queryset import SearchableQuerySetMixin
 
 from wagtailvideos import ffmpeg
 
@@ -367,7 +365,7 @@ class AbstractVideoTranscode(models.Model):
 
 
 class VideoTranscode(AbstractVideoTranscode):
-    video = models.ForeignKey(Video, related_name='transcodes')
+    video = models.ForeignKey(Video, related_name='transcodes', on_delete=models.CASCADE)
 
     class Meta:
         unique_together = (

wagtailvideos/permissions.py

-from __future__ import absolute_import, print_function, unicode_literals
-
-from wagtail.wagtailcore.permission_policies.collections import \
+from wagtail.core.permission_policies.collections import \
     CollectionOwnershipPermissionPolicy
 
 from wagtailvideos.models import Video

wagtailvideos/templatetags/wagtailvideos_tags.py

-from __future__ import absolute_import, print_function, unicode_literals
-
 from django import template
 
 register = template.Library()

wagtailvideos/urls.py

-from __future__ import absolute_import, print_function, unicode_literals
-
 from django.conf.urls import url
 
 from wagtailvideos.views import chooser, multiple, videos