wagtail 7.2rc1__py3-none-any.whl → 7.3rc1__py3-none-any.whl

wagtail/admin/tests/test_userbar.py

@@ -9,6 +9,7 @@ from django.utils.translation import gettext
 
 from wagtail import hooks
 from wagtail.admin.staticfiles import versioned_static
+from wagtail.admin.ui.components import Component
 from wagtail.admin.userbar import AccessibilityItem, Userbar
 from wagtail.coreutils import get_dummy_request
 from wagtail.models import PAGE_TEMPLATE_VAR, Locale, Page, Site
@@ -761,7 +762,7 @@ class TestUserbarComponent(WagtailTestUtils, TestCase):
         self.assertIsNotNone(button)
         self.assertEqual(
             button.get_text(separator=" | ", strip=True).strip(),
-            "Issues found | Accessibility",
+            "Issues found | Checks",
         )
 
     def test_render_no_request(self):
@@ -790,7 +791,7 @@ class TestUserbarComponent(WagtailTestUtils, TestCase):
         self.assertIsNotNone(button)
         self.assertEqual(
             button.get_text(separator=" | ", strip=True).strip(),
-            "Issues found | Accessibility",
+            "Issues found | Checks",
         )
 
         css_links = soup.select("link[rel='stylesheet']")
@@ -842,7 +843,7 @@ class TestUserbarComponent(WagtailTestUtils, TestCase):
         self.assertIsNotNone(button)
         self.assertEqual(
             button.get_text(separator=" | ", strip=True).strip(),
-            "Issues found | Accessibility",
+            "Issues found | Checks",
         )
 
     def test_render_with_page(self):
@@ -868,7 +869,7 @@ class TestUserbarComponent(WagtailTestUtils, TestCase):
         self.assertIsNotNone(accessibility_button)
         self.assertEqual(
             accessibility_button.get_text(separator=" | ", strip=True).strip(),
-            "Issues found | Accessibility",
+            "Issues found | Checks",
         )
 
     @override_settings(WAGTAILADMIN_BASE_URL=None)
@@ -899,7 +900,7 @@ class TestUserbarComponent(WagtailTestUtils, TestCase):
         self.assertIsNotNone(button)
         self.assertEqual(
             button.get_text(separator=" | ", strip=True).strip(),
-            "Issues found | Accessibility",
+            "Issues found | Checks",
         )
 
         css_links = soup.select("link[rel='stylesheet']")
@@ -923,3 +924,66 @@ class TestUserbarComponent(WagtailTestUtils, TestCase):
                 versioned_static("wagtailadmin/js/userbar.js"),
             ],
         )
+
+    def test_component_item(self):
+        class TestItem(Component):
+            def render_html(self, parent_context):
+                return "<li><a href='#test-item'>Test item</a></li>"
+
+        def hook(request, items, page):
+            items.append(TestItem())
+
+        with hooks.register_temporarily("construct_wagtail_userbar", hook):
+            rendered = Userbar(object=self.homepage).render_html(
+                {"request": self.request, PAGE_TEMPLATE_VAR: self.homepage}
+            )
+            soup = self.get_soup(rendered)
+
+        self.assertIsNotNone(soup.css.select_one("a[href='#test-item']"))
+
+    def test_legacy_render_item(self):
+        class LegacyItem:
+            def render(self, request):
+                return "<li><a href='#legacy-item'>Legacy item</a></li>"
+
+        def hook(request, items, page):
+            items.append(LegacyItem())
+
+        with (
+            self.assertWarnsMessage(
+                RemovedInWagtail80Warning,
+                "Userbar items now use the `render_html(parent_context)` method instead of `render(request)` - "
+                "view the `construct_wagtail_userbar` docs to update LegacyItem",
+            ),
+            hooks.register_temporarily(
+                "construct_wagtail_userbar",
+                hook,
+            ),
+        ):
+            rendered = Userbar(object=self.homepage).render_html(
+                {"request": self.request, PAGE_TEMPLATE_VAR: self.homepage}
+            )
+            soup = self.get_soup(rendered)
+
+        self.assertIsNotNone(soup.css.select_one("a[href='#legacy-item']"))
+
+    def test_component_item_with_media(self):
+        class ItemWithMedia(Component):
+            def render_html(self, parent_context):
+                return ""
+
+            class Media:
+                js = ["custom-item.js"]
+                css = {"all": ["custom-item.css"]}
+
+        def hook(request, items, page):
+            items.append(ItemWithMedia())
+
+        with hooks.register_temporarily("construct_wagtail_userbar", hook):
+            rendered = Userbar(object=self.homepage).render_html(
+                {"request": self.request, PAGE_TEMPLATE_VAR: self.homepage}
+            )
+            soup = self.get_soup(rendered)
+
+        self.assertIsNotNone(soup.css.select_one('script[src$="custom-item.js"]'))
+        self.assertIsNotNone(soup.css.select_one('link[href$="custom-item.css"]'))

wagtail/admin/tests/test_views_generic.py

@@ -9,7 +9,7 @@ from wagtail.test.utils import WagtailTestUtils
 class TestGenericIndexView(WagtailTestUtils, TestCase):
     fixtures = ["test.json"]
 
-    def get(self, params={}):
+    def get(self, params=None):
         return self.client.get(reverse("testapp_generic_index"), params)
 
     def test_non_integer_primary_key(self):
@@ -28,7 +28,7 @@ class TestGenericIndexView(WagtailTestUtils, TestCase):
 class TestGenericIndexViewWithoutModel(WagtailTestUtils, TestCase):
     fixtures = ["test.json"]
 
-    def get(self, params={}):
+    def get(self, params=None):
         return self.client.get(reverse("testapp_generic_index_without_model"), params)
 
     def test_non_integer_primary_key(self):
@@ -41,7 +41,7 @@ class TestGenericIndexViewWithoutModel(WagtailTestUtils, TestCase):
 class TestGenericCreateView(WagtailTestUtils, TestCase):
     fixtures = ["test.json"]
 
-    def get(self, params={}):
+    def get(self, params=None):
         return self.client.get(reverse("testapp_generic_create"), params)
 
     def test_get_create_view(self):
@@ -74,7 +74,7 @@ class TestGenericCreateView(WagtailTestUtils, TestCase):
 class TestGenericEditView(WagtailTestUtils, TestCase):
     fixtures = ["test.json"]
 
-    def get(self, object_pk, params={}):
+    def get(self, object_pk, params=None):
         return self.client.get(
             reverse("testapp_generic_edit", args=(object_pk,)), params
         )
@@ -120,7 +120,7 @@ class TestGenericEditView(WagtailTestUtils, TestCase):
 class TestGenericDeleteView(WagtailTestUtils, TestCase):
     fixtures = ["test.json"]
 
-    def get(self, object_pk, params={}):
+    def get(self, object_pk, params=None):
         return self.client.get(
             reverse("testapp_generic_edit", args=(object_pk,)), params
         )

wagtail/admin/tests/test_workflows.py

@@ -143,7 +143,7 @@ class TestWorkflowsIndexView(AdminTemplateTestUtils, WagtailTestUtils, TestCase)
         ]
         WorkflowPage.objects.bulk_create(workflow_pages)
 
-    def get(self, params={}):
+    def get(self, params=None):
         return self.client.get(reverse("wagtailadmin_workflows:index"), params)
 
     def test_simple(self):
@@ -339,7 +339,7 @@ class TestWorkflowPermissions(WagtailTestUtils, TestCase):
     def setUp(self):
         self.user = self.login()
 
-    def get(self, params={}):
+    def get(self, params=None):
         return self.client.get(reverse(self.url_name), params)
 
     def test_simple(self):
@@ -417,10 +417,10 @@ class TestWorkflowsCreateView(AdminTemplateTestUtils, WagtailTestUtils, TestCase
             FullFeaturedSnippet
         )
 
-    def get(self, params={}):
+    def get(self, params=None):
         return self.client.get(reverse("wagtailadmin_workflows:add"), params)
 
-    def post(self, post_data={}):
+    def post(self, post_data=None):
         return self.client.post(reverse("wagtailadmin_workflows:add"), post_data)
 
     def test_get(self):
@@ -676,12 +676,12 @@ class TestWorkflowsEditView(AdminTemplateTestUtils, WagtailTestUtils, TestCase):
         moderators.user_set.add(self.moderator)
         moderators.permissions.add(Permission.objects.get(codename="change_workflow"))
 
-    def get(self, params={}):
+    def get(self, params=None):
         return self.client.get(
             reverse("wagtailadmin_workflows:edit", args=[self.workflow.id]), params
         )
 
-    def post(self, post_data={}):
+    def post(self, post_data=None):
         return self.client.post(
             reverse("wagtailadmin_workflows:edit", args=[self.workflow.id]), post_data
         )
@@ -1028,7 +1028,7 @@ class TestRemoveWorkflow(WagtailTestUtils, TestCase):
         moderators.user_set.add(self.moderator)
         moderators.permissions.add(Permission.objects.get(codename="change_workflow"))
 
-    def post(self, post_data={}):
+    def post(self, post_data=None):
         return self.client.post(
             reverse(
                 "wagtailadmin_workflows:remove", args=[self.page.id, self.workflow.id]
@@ -1082,7 +1082,7 @@ class TestTaskIndexView(AdminTemplateTestUtils, WagtailTestUtils, TestCase):
         moderators.user_set.add(self.moderator)
         moderators.permissions.add(Permission.objects.get(codename="change_task"))
 
-    def get(self, params={}):
+    def get(self, params=None):
         return self.client.get(reverse("wagtailadmin_workflows:task_index"), params)
 
     def test_simple(self):
@@ -1370,7 +1370,7 @@ class TestCreateTaskView(AdminTemplateTestUtils, WagtailTestUtils, TestCase):
         moderators.user_set.add(self.moderator)
         moderators.permissions.add(Permission.objects.get(codename="add_task"))
 
-    def get(self, url_kwargs=None, params={}):
+    def get(self, url_kwargs=None, params=None):
         url_kwargs = url_kwargs or {}
         url_kwargs.setdefault("app_label", SimpleTask._meta.app_label)
         url_kwargs.setdefault("model_name", SimpleTask._meta.model_name)
@@ -1378,7 +1378,7 @@ class TestCreateTaskView(AdminTemplateTestUtils, WagtailTestUtils, TestCase):
             reverse("wagtailadmin_workflows:add_task", kwargs=url_kwargs), params
         )
 
-    def post(self, post_data={}):
+    def post(self, post_data=None):
         return self.client.post(
             reverse(
                 "wagtailadmin_workflows:add_task",
@@ -1504,12 +1504,12 @@ class TestEditTaskView(AdminTemplateTestUtils, WagtailTestUtils, TestCase):
         moderators.user_set.add(self.moderator)
         moderators.permissions.add(Permission.objects.get(codename="change_task"))
 
-    def get(self, params={}):
+    def get(self, params=None):
         return self.client.get(
             reverse("wagtailadmin_workflows:edit_task", args=[self.task.id]), params
         )
 
-    def post(self, post_data={}):
+    def post(self, post_data=None):
         return self.client.post(
             reverse("wagtailadmin_workflows:edit_task", args=[self.task.id]), post_data
         )
@@ -3434,10 +3434,12 @@ class TestPageWorkflowReportResults(TestPageWorkflowReport):
     header_buttons_parent_selector = (
         '[data-controller="w-teleport"]'
         '[data-w-teleport-target-value="#w-slim-header-buttons"]'
+        '[data-w-teleport-mode-value="innerHTML"]'
     )
     drilldown_selector = (
         '[data-controller="w-teleport"]'
         '[data-w-teleport-target-value="#filters-drilldown"]'
+        '[data-w-teleport-mode-value="innerHTML"]'
     )
     extra_params = "&_w_filter_fragment=true"
 

wagtail/admin/tests/viewsets/test_model_viewset.py

@@ -91,6 +91,19 @@ class TestModelViewSetGroup(WagtailTestUtils, TestCase):
         self.assertNotContains(response, reverse("blockcounts_streammodel:index"))
 
 
+class TestAdminURLs(WagtailTestUtils, TestCase):
+    def setUp(self):
+        self.user = self.login()
+
+    def test_cannot_reverse_mismatched_converter_value(self):
+        with self.assertRaises(NoReverseMatch):
+            reverse("streammodel:edit", kwargs={"pk": "123abc"})
+
+    def test_404_on_mismatched_converter_value(self):
+        response = self.client.get("/admin/streammodel/edit/123abc/")
+        self.assertEqual(response.status_code, 404)
+
+
 class TestTemplateConfiguration(WagtailTestUtils, TestCase):
     def setUp(self):
         self.user = self.login()

wagtail/admin/ui/autosave.py

@@ -0,0 +1,5 @@
+from wagtail.admin.ui.components import Component
+
+
+class AutosaveIndicator(Component):
+    template_name = "wagtailadmin/shared/autosave/indicator.html"

wagtail/admin/ui/editing_sessions.py

@@ -13,12 +13,14 @@ class EditingSessionsModule(Component):
         release_url,
         other_sessions,
         revision_id=None,
+        revision_created_at=None,
     ):
         self.current_session = current_session
         self.ping_url = ping_url
         self.release_url = release_url
         self.sessions_list = EditingSessionsList(current_session, other_sessions)
         self.revision_id = revision_id
+        self.revision_created_at = revision_created_at
 
     def get_context_data(self, parent_context):
         ping_interval = getattr(
@@ -33,6 +35,7 @@ class EditingSessionsModule(Component):
             "ping_interval": ping_interval,
             "sessions_list": self.sessions_list,
             "revision_id": self.revision_id,
+            "revision_created_at": self.revision_created_at,
         }
 
 

wagtail/admin/ui/side_panels.py

@@ -22,13 +22,7 @@ class BaseSidePanel(Component):
             self.panel = panel
 
         def get_context_data(self, parent_context):
-            # Inherit classes from fragments defined in slim_header.html
-            inherit = {
-                "nav_icon_button_classes",
-                "nav_icon_classes",
-                "nav_icon_counter_classes",
-            }
-            context = {key: parent_context.get(key) for key in inherit}
+            context = {}
             context["toggle"] = self
             context["panel"] = self.panel
             context["count"] = 0
@@ -98,16 +92,15 @@ class StatusSidePanel(BaseSidePanel):
                 "wagtailadmin/shared/side_panels/includes/status/locale.html"
             )
 
-        if self.object.pk:
-            if self.locking_enabled:
-                templates.append(
-                    "wagtailadmin/shared/side_panels/includes/status/locked.html"
-                )
+        if self.locking_enabled:
+            templates.append(
+                "wagtailadmin/shared/side_panels/includes/status/locked.html"
+            )
 
-            if self.usage_url:
-                templates.append(
-                    "wagtailadmin/shared/side_panels/includes/status/usage.html"
-                )
+        if self.usage_url is not None:
+            templates.append(
+                "wagtailadmin/shared/side_panels/includes/status/usage.html"
+            )
 
         return templates
 
@@ -211,10 +204,11 @@ class StatusSidePanel(BaseSidePanel):
         }
 
     def get_usage_context(self):
+        usage_count = 0
+        if self.object.pk:
+            usage_count = ReferenceIndex.get_grouped_references_to(self.object).count()
         return {
-            "usage_count": ReferenceIndex.get_grouped_references_to(
-                self.object
-            ).count(),
+            "usage_count": usage_count,
             "usage_url": self.usage_url,
         }
 
@@ -229,9 +223,13 @@ class StatusSidePanel(BaseSidePanel):
         context["history_url"] = self.history_url
         context["status_templates"] = self.get_status_templates(context)
         context["last_updated_info"] = self.last_updated_info
+        context["is_partial"] = parent_context.get("is_partial", False)
+        context["hydrate_create_view"] = parent_context.get(
+            "hydrate_create_view", False
+        )
         context.update(self.get_scheduled_publishing_context(parent_context))
         context.update(self.get_lock_context(parent_context))
-        if self.object.pk and self.usage_url:
+        if self.usage_url is not None:
             context.update(self.get_usage_context())
         return context
 
@@ -240,6 +238,7 @@ class PageStatusSidePanel(StatusSidePanel):
     def __init__(self, *args, **kwargs):
         self.parent_page = kwargs.pop("parent_page", None)
         super().__init__(*args, **kwargs)
+        self.usage_url = ""
         if self.object.pk:
             self.usage_url = reverse("wagtailadmin_pages:usage", args=(self.object.pk,))
 

wagtail/admin/userbar.py

@@ -1,6 +1,6 @@
 from warnings import warn
 
-from django.template.loader import render_to_string
+from django.forms import Media
 from django.utils import translation
 from django.utils.translation import gettext_lazy as _
 
@@ -14,18 +14,18 @@ from wagtail.users.models import UserProfile
 from wagtail.utils.deprecation import RemovedInWagtail80Warning
 
 
-class BaseItem:
-    template = "wagtailadmin/userbar/item_base.html"
+class BaseItem(Component):
+    template_name = "wagtailadmin/userbar/item_base.html"
 
-    def get_context_data(self, request):
-        return {"self": self, "request": request}
-
-    def render(self, request):
-        return render_to_string(self.template, self.get_context_data(request))
+    def get_context_data(self, parent_context):
+        context = super().get_context_data(parent_context)
+        context["self"] = self
+        context["request"] = parent_context.get("request")
+        return context
 
 
 class AdminItem(BaseItem):
-    template = "wagtailadmin/userbar/item_admin.html"
+    template_name = "wagtailadmin/userbar/item_admin.html"
 
 
 class AccessibilityItem(BaseItem):
@@ -37,7 +37,7 @@ class AccessibilityItem(BaseItem):
         """Whether the accessibility checker is being run in the page editor."""
 
     #: The template to use for rendering the item.
-    template = "wagtailadmin/userbar/item_accessibility.html"
+    template_name = "wagtailadmin/userbar/item_accessibility.html"
 
     #: A list of CSS selector(s) to test specific parts of the page.
     #: For more details, see `Axe documentation <https://github.com/dequelabs/axe-core/blob/master/doc/context.md#the-include-property>`__.
@@ -212,21 +212,23 @@ class AccessibilityItem(BaseItem):
             "spec": self.get_axe_spec(request),
         }
 
-    def get_context_data(self, request):
-        return {
-            **super().get_context_data(request),
-            "axe_configuration": self.get_axe_configuration(request),
-        }
+    def get_context_data(self, parent_context):
+        context = super().get_context_data(parent_context)
+        context["axe_configuration"] = self.get_axe_configuration(
+            parent_context.get("request")
+        )
+        return context
 
 
 class AddPageItem(BaseItem):
-    template = "wagtailadmin/userbar/item_page_add.html"
+    template_name = "wagtailadmin/userbar/item_page_add.html"
 
     def __init__(self, page):
         self.page = page
         self.parent_page = page.get_parent()
 
-    def render(self, request):
+    def render_html(self, parent_context):
+        request = parent_context.get("request")
         # Don't render if the page doesn't have an id
         if not self.page.id:
             return ""
@@ -236,17 +238,18 @@ class AddPageItem(BaseItem):
         if not permission_checker.can_add_subpage():
             return ""
 
-        return super().render(request)
+        return super().render_html(parent_context)
 
 
 class ExplorePageItem(BaseItem):
-    template = "wagtailadmin/userbar/item_page_explore.html"
+    template_name = "wagtailadmin/userbar/item_page_explore.html"
 
     def __init__(self, page):
         self.page = page
         self.parent_page = page.get_parent()
 
-    def render(self, request):
+    def render_html(self, parent_context):
+        request = parent_context.get("request")
         # Don't render if the page doesn't have an id
         if not self.page.id:
             return ""
@@ -259,16 +262,17 @@ class ExplorePageItem(BaseItem):
         ):
             return ""
 
-        return super().render(request)
+        return super().render_html(parent_context)
 
 
 class EditPageItem(BaseItem):
-    template = "wagtailadmin/userbar/item_page_edit.html"
+    template_name = "wagtailadmin/userbar/item_page_edit.html"
 
     def __init__(self, page):
         self.page = page
 
-    def render(self, request):
+    def render_html(self, parent_context):
+        request = parent_context.get("request")
         # Don't render if the page doesn't have an id
         if not self.page.id:
             return ""
@@ -286,7 +290,7 @@ class EditPageItem(BaseItem):
         if not permission_checker.can_edit():
             return ""
 
-        return super().render(request)
+        return super().render_html(parent_context)
 
 
 def apply_userbar_hooks(request, items, page):
@@ -359,10 +363,26 @@ class Userbar(Component):
             apply_userbar_hooks(request, items, self.object)
 
             # Render the items
-            rendered_items = [item.render(request) for item in items]
+            rendered_items = []
+            media = Media()
+            for item in items:
+                # Backwards compatibility for legacy items with a render method.
+                if hasattr(item, "render"):
+                    warn(
+                        "Userbar items now use the `render_html(parent_context)` method instead of `render(request)` -"
+                        f" view the `construct_wagtail_userbar` docs to update {item.__class__.__name__}",
+                        RemovedInWagtail80Warning,
+                    )
+
+                    content = item.render(request)
+                else:
+                    content = item.render_html(parent_context)
+                if content:
+                    rendered_items.append(content)
 
-            # Remove any unrendered items
-            rendered_items = [item for item in rendered_items if item]
+                # Always append media even if the item might render nothing.
+                if hasattr(item, "media"):
+                    media += item.media
 
             if request:
                 origin = f"{request.scheme}://{request.get_host()}"
@@ -374,6 +394,7 @@ class Userbar(Component):
                 "request": request,
                 "origin": origin,
                 "items": rendered_items,
+                "media": media,
                 "position": self.position,
                 "page": self.object,
                 "revision_id": revision_id,

wagtail/admin/views/bulk_action/dispatcher.py

@@ -7,8 +7,8 @@ from wagtail.admin.views.bulk_action.registry import bulk_action_registry as reg
 def index(request, app_label, model_name, action):
     try:
         model = apps.get_model(app_label, model_name)
-    except LookupError:
-        raise Http404
+    except LookupError as e:
+        raise Http404 from e
     action_class = registry.get_bulk_action_class(app_label, model_name, action)
     if action_class is not None:
         return action_class(request, model).dispatch(request)

wagtail/admin/views/chooser.py

@@ -243,8 +243,8 @@ class BrowseView(View):
 
         try:
             self.desired_classes = page_models_from_string(page_type_string)
-        except (ValueError, LookupError):
-            raise Http404
+        except (ValueError, LookupError) as e:
+            raise Http404 from e
 
         # Find parent page
         if parent_page_id:
@@ -350,8 +350,8 @@ class BrowseView(View):
         paginator = Paginator(pages, per_page=25)
         try:
             pages = paginator.page(request.GET.get("p", 1))
-        except InvalidPage:
-            raise Http404
+        except InvalidPage as e:
+            raise Http404 from e
 
         # Annotate each page with can_choose/can_descend flags
         for page in pages:
@@ -445,8 +445,8 @@ class SearchView(View):
 
         try:
             desired_classes = page_models_from_string(page_type_string)
-        except (ValueError, LookupError):
-            raise Http404
+        except (ValueError, LookupError) as e:
+            raise Http404 from e
 
         pages = Page.objects.all()
         if self.i18n_enabled:

wagtail/admin/views/editing_sessions.py

@@ -1,3 +1,5 @@
+from datetime import datetime
+
 from django.apps import apps
 from django.contrib.admin.utils import unquote
 from django.contrib.contenttypes.models import ContentType
@@ -18,8 +20,8 @@ from wagtail.models import Page, Revision, RevisionMixin, WorkflowMixin
 def ping(request, app_label, model_name, object_id, session_id):
     try:
         model = apps.get_model(app_label, model_name)
-    except LookupError:
-        raise Http404
+    except LookupError as e:
+        raise Http404 from e
 
     unquoted_object_id = unquote(object_id)
 
@@ -31,9 +33,9 @@ def ping(request, app_label, model_name, object_id, session_id):
     else:
         try:
             permission_policy = model.snippet_viewset.permission_policy
-        except AttributeError:
+        except AttributeError as e:
             # model is neither a Page nor a snippet
-            raise Http404
+            raise Http404 from e
 
         can_edit = permission_policy.user_has_permission_for_instance(
             request.user, "change", obj
@@ -108,11 +110,22 @@ def ping(request, app_label, model_name, object_id, session_id):
         all_revisions = obj.revisions.defer("content")
         try:
             original_revision = all_revisions.get(id=revision_id)
-        except Revision.DoesNotExist:
-            raise Http404
+        except Revision.DoesNotExist as e:
+            raise Http404 from e
 
+        try:
+            # The created_at of the latest revision when the session was created
+            created_at = datetime.fromisoformat(
+                request.POST.get("revision_created_at", "")
+            )
+        except ValueError:
+            # Invalid or missing, assume it's the same as the real created_at
+            created_at = original_revision.created_at
+
+        # The newest revision may have been updated via autosave after the
+        # session was created, so filter by created_at rather than the pk.
         newest_revision = (
-            all_revisions.filter(created_at__gt=original_revision.created_at)
+            all_revisions.filter(created_at__gt=created_at)
             .order_by("-created_at", "-pk")
             .select_related("user")
             .first()

wagtail/admin/views/generic/__init__.py

@@ -10,6 +10,7 @@ from .mixins import (  # noqa: F401
     CreateEditViewOptionalFeaturesMixin,
     HookResponseMixin,
     IndexViewOptionalFeaturesMixin,
+    JsonPostResponseMixin,
     LocaleMixin,
     PanelMixin,
     RevisionsRevertMixin,