wagtail 6.4.1__py3-none-any.whl → 7.0rc1__py3-none-any.whl

wagtail/admin/urls/__init__.py

@@ -6,7 +6,6 @@ from django.urls import include, path, re_path
 from django.views.decorators.cache import never_cache
 from django.views.defaults import page_not_found
 from django.views.generic import TemplateView
-from django.views.i18n import JavaScriptCatalog
 
 from wagtail import hooks
 from wagtail.admin.api import urls as api_urls
@@ -20,6 +19,7 @@ from wagtail.admin.urls import workflows as wagtailadmin_workflows_urls
 from wagtail.admin.views import account, chooser, dismissibles, home, tags
 from wagtail.admin.views.bulk_action import index as bulk_actions
 from wagtail.admin.views.generic.preview import StreamFieldBlockPreview
+from wagtail.admin.views.i18n import localized_js_catalog
 from wagtail.admin.views.pages import listing
 from wagtail.utils.urlpatterns import decorate_urlpatterns
 
@@ -147,7 +147,7 @@ urlpatterns += [
     # JS translation catalog
     path(
         "jsi18n/",
-        JavaScriptCatalog.as_view(packages=["wagtail.admin"]),
+        localized_js_catalog,
         name="wagtailadmin_javascript_catalog",
     ),
 ]

wagtail/admin/urls/pages.py

@@ -63,7 +63,9 @@ urlpatterns = [
         search.SearchView.as_view(results_only=True),
         name="search_results",
     ),
-    path("<int:page_to_move_id>/move/", move.move_choose_destination, name="move"),
+    path(
+        "<int:page_to_move_id>/move/", move.MoveChooseDestination.as_view(), name="move"
+    ),
     path(
         "<int:page_to_move_id>/move/<int:destination_id>/confirm/",
         move.move_confirm,

wagtail/admin/userbar.py

@@ -5,7 +5,7 @@ from django.utils.translation import gettext_lazy as _
 
 from wagtail import hooks
 from wagtail.coreutils import accepts_kwarg
-from wagtail.utils.deprecation import RemovedInWagtail70Warning
+from wagtail.utils.deprecation import RemovedInWagtail80Warning
 
 
 class BaseItem:
@@ -15,9 +15,7 @@ class BaseItem:
         return {"self": self, "request": request}
 
     def render(self, request):
-        return render_to_string(
-            self.template, self.get_context_data(request), request=request
-        )
+        return render_to_string(self.template, self.get_context_data(request))
 
 
 class AdminItem(BaseItem):
@@ -314,6 +312,6 @@ def apply_userbar_hooks(request, items, page):
             warn(
                 "`construct_wagtail_userbar` hook functions should accept a `page` argument in third position -"
                 f" {fn.__module__}.{fn.__name__} needs to be updated",
-                category=RemovedInWagtail70Warning,
+                category=RemovedInWagtail80Warning,
             )
             fn(request, items)

wagtail/admin/utils.py

@@ -1,7 +1,10 @@
+import re
+from types import SimpleNamespace
 from urllib.parse import parse_qs, urlsplit, urlunsplit
 
 from django.conf import settings
 from django.utils.http import url_has_allowed_host_and_scheme, urlencode
+from django.utils.translation import gettext
 
 
 def get_admin_base_url():
@@ -30,10 +33,19 @@ def get_latest_str(obj):
     from wagtail.models import DraftStateMixin, Page
 
     if isinstance(obj, Page):
-        return obj.specific_deferred.get_admin_display_title()
-    if isinstance(obj, DraftStateMixin) and obj.latest_revision:
-        return obj.latest_revision.object_str
-    return str(obj)
+        result = obj.specific_deferred.get_admin_display_title()
+    elif isinstance(obj, DraftStateMixin) and obj.latest_revision:
+        result = obj.latest_revision.object_str
+    else:
+        result = str(obj)
+
+    if result.strip() == "":
+        result = gettext("%(classname)s object (%(id)s)") % {
+            "classname": obj.__class__.__name__,
+            "id": obj.pk,
+        }
+
+    return result
 
 
 def get_user_display_name(user):
@@ -70,3 +82,30 @@ def set_query_params(url: str, params: dict):
     querydict = {key: value for key, value in querydict.items() if value is not None}
     query = urlencode(querydict, doseq=True)
     return urlunsplit((scheme, netloc, path, query, fragment))
+
+
+def get_keyboard_key_labels_from_request(request):
+    """
+    Returns an instance of SimpleNamespace based on the user's keyboard layout
+    based on the User-Agent header in the request.
+
+    These are intentionally not translated, as they are key labels that are assumed
+    to be consistent across all languages.
+    """
+
+    user_agent = request.headers.get("User-Agent", "")
+    is_mac_os = re.search(r"Mac|iPod|iPhone|iPad", user_agent)
+
+    labels = {
+        "ALT": "⌥" if is_mac_os else "Alt",
+        "CMD": "⌘" if is_mac_os else "Ctrl",
+        "CTRL": "^" if is_mac_os else "Ctrl",
+        "DEL": "Delete",
+        "ENTER": "Return" if is_mac_os else "Enter",
+        "ESC": "Esc",
+        "MOD": "⌘" if is_mac_os else "Ctrl",
+        "SHIFT": "Shift",
+        "TAB": "Tab",
+    }
+
+    return SimpleNamespace(**labels)

wagtail/admin/views/generic/base.py

@@ -1,5 +1,4 @@
 import warnings
-from collections import namedtuple
 
 from django.contrib.admin.utils import quote, unquote
 from django.core.exceptions import ImproperlyConfigured
@@ -7,28 +6,21 @@ from django.db import models
 from django.db.models import Q
 from django.shortcuts import get_object_or_404, redirect
 from django.urls import reverse, reverse_lazy
-from django.utils.formats import date_format
 from django.utils.functional import cached_property
 from django.utils.translation import gettext_lazy as _
 from django.views import View
 from django.views.generic.base import ContextMixin, TemplateResponseMixin
 from django.views.generic.list import BaseListView
-from django_filters.filters import (
-    ChoiceFilter,
-    DateFromToRangeFilter,
-    ModelChoiceFilter,
-    ModelMultipleChoiceFilter,
-    MultipleChoiceFilter,
-)
 
 from wagtail.admin import messages
+from wagtail.admin.active_filters import filter_adapter_class_registry
 from wagtail.admin.forms.search import SearchForm
+from wagtail.admin.paginator import WagtailPaginator
 from wagtail.admin.ui.tables import Column, Table
 from wagtail.admin.utils import get_valid_next_url_from_request
 from wagtail.admin.widgets.button import ButtonWithDropdown
 from wagtail.search.backends import get_search_backend
 from wagtail.search.index import class_is_indexed
-from wagtail.utils.utils import flatten_choices
 
 
 class WagtailAdminTemplateMixin(TemplateResponseMixin, ContextMixin):
@@ -191,12 +183,6 @@ class BaseOperationView(BaseObjectMixin, View):
         return redirect(self.get_success_url())
 
 
-# Represents a django-filters filter that is currently in force on a listing queryset
-ActiveFilter = namedtuple(
-    "ActiveFilter", ["auto_id", "field_label", "value", "removed_filter_url"]
-)
-
-
 class BaseListingView(WagtailAdminTemplateMixin, BaseListView):
     template_name = "wagtailadmin/generic/listing.html"
     results_template_name = "wagtailadmin/generic/listing_results.html"
@@ -214,6 +200,7 @@ class BaseListingView(WagtailAdminTemplateMixin, BaseListView):
     default_ordering = None
     filterset_class = None
     verbose_name_plural = None
+    paginator_class = WagtailPaginator
 
     def get_template_names(self):
         if self.results_only:
@@ -317,35 +304,6 @@ class BaseListingView(WagtailAdminTemplateMixin, BaseListView):
             queryset = self.filters.filter_queryset(queryset)
         return queryset
 
-    def get_url_without_filter_param(self, param):
-        """
-        Return the index URL with the given filter parameter removed from the query string
-        """
-        base_url = self.index_results_url.split("?")[0]
-        query_dict = self.request.GET.copy()
-        query_dict.pop(self.page_kwarg, None)  # reset pagination to first page
-        if isinstance(param, (list, tuple)):
-            for p in param:
-                query_dict.pop(p, None)
-        else:
-            query_dict.pop(param, None)
-        query_dict["_w_filter_fragment"] = 1
-        return base_url + "?" + query_dict.urlencode()
-
-    def get_url_without_filter_param_value(self, param, value):
-        """
-        Return the index URL where the filter parameter with the given value has been removed
-        from the query string, preserving all other values for that parameter
-        """
-        base_url = self.index_results_url.split("?")[0]
-        query_dict = self.request.GET.copy()
-        query_dict.pop(self.page_kwarg, None)  # reset pagination to first page
-        query_dict.setlist(
-            param, [v for v in query_dict.getlist(param) if v != str(value)]
-        )
-        query_dict["_w_filter_fragment"] = 1
-        return base_url + "?" + query_dict.urlencode()
-
     @cached_property
     def active_filters(self):
         filters = []
@@ -353,6 +311,11 @@ class BaseListingView(WagtailAdminTemplateMixin, BaseListView):
         if not self.filters:
             return filters
 
+        base_url = self.index_results_url.split("?")[0]
+        query_dict = self.request.GET.copy()
+        query_dict.pop(self.page_kwarg, None)  # reset pagination to first page
+        query_dict["_w_filter_fragment"] = 1
+
         for field_name in self.filters.form.changed_data:
             filter_def = self.filters.filters[field_name]
             bound_field = self.filters.form[field_name]
@@ -364,76 +327,11 @@ class BaseListingView(WagtailAdminTemplateMixin, BaseListView):
             if value == bound_field.initial:
                 continue  # filter value is the same as the default
 
-            if isinstance(filter_def, ModelMultipleChoiceFilter):
-                field = filter_def.field
-                for item in value:
-                    filters.append(
-                        ActiveFilter(
-                            bound_field.auto_id,
-                            filter_def.label,
-                            field.label_from_instance(item),
-                            self.get_url_without_filter_param_value(
-                                field_name, item.pk
-                            ),
-                        )
-                    )
-            elif isinstance(filter_def, MultipleChoiceFilter):
-                choices = flatten_choices(filter_def.field.choices)
-                for item in value:
-                    filters.append(
-                        ActiveFilter(
-                            bound_field.auto_id,
-                            filter_def.label,
-                            choices.get(str(item), str(item)),
-                            self.get_url_without_filter_param_value(field_name, item),
-                        )
-                    )
-            elif isinstance(filter_def, ModelChoiceFilter):
-                field = filter_def.field
-                filters.append(
-                    ActiveFilter(
-                        bound_field.auto_id,
-                        filter_def.label,
-                        field.label_from_instance(value),
-                        self.get_url_without_filter_param(field_name),
-                    )
-                )
-            elif isinstance(filter_def, DateFromToRangeFilter):
-                start_date_display = date_format(value.start) if value.start else ""
-                end_date_display = date_format(value.stop) if value.stop else ""
-                widget = filter_def.field.widget
-                filters.append(
-                    ActiveFilter(
-                        bound_field.auto_id,
-                        filter_def.label,
-                        f"{start_date_display} - {end_date_display}",
-                        self.get_url_without_filter_param(
-                            [
-                                widget.suffixed(field_name, suffix)
-                                for suffix in widget.suffixes
-                            ]
-                        ),
-                    )
-                )
-            elif isinstance(filter_def, ChoiceFilter):
-                choices = flatten_choices(filter_def.field.choices)
-                filters.append(
-                    ActiveFilter(
-                        bound_field.auto_id,
-                        filter_def.label,
-                        choices.get(str(value), str(value)),
-                        self.get_url_without_filter_param(field_name),
-                    )
-                )
-            else:
-                filters.append(
-                    ActiveFilter(
-                        bound_field.auto_id,
-                        filter_def.label,
-                        str(value),
-                        self.get_url_without_filter_param(field_name),
-                    )
-                )
+            filter_adapter_class = filter_adapter_class_registry.get(filter_def)
+            filter_adapter = filter_adapter_class(
+                filter_def, bound_field, value, base_url, query_dict
+            )
+            filters.extend(filter_adapter.get_active_filters())
 
         return filters
 
@@ -573,6 +471,9 @@ class BaseListingView(WagtailAdminTemplateMixin, BaseListView):
 
         if context["is_paginated"]:
             context["items_count"] = context["paginator"].count
+            context["elided_page_range"] = context["paginator"].get_elided_page_range(
+                self.request.GET.get(self.page_kwarg, 1)
+            )
         else:
             context["items_count"] = len(context["object_list"])
 

wagtail/admin/views/generic/chooser.py

@@ -8,7 +8,7 @@ from django.core.exceptions import (
     ObjectDoesNotExist,
     PermissionDenied,
 )
-from django.core.paginator import InvalidPage, Paginator
+from django.core.paginator import InvalidPage
 from django.db.models import Model
 from django.forms.models import modelform_factory
 from django.http import Http404
@@ -28,7 +28,8 @@ from wagtail.admin.forms.choosers import (
     SearchFilterMixin,
 )
 from wagtail.admin.modal_workflow import render_modal_workflow
-from wagtail.admin.ui.tables import Column, Table, TitleColumn
+from wagtail.admin.paginator import WagtailPaginator
+from wagtail.admin.ui.tables import Column, LocaleColumn, Table, TitleColumn
 from wagtail.coreutils import resolve_model_string
 from wagtail.models import CollectionMember, TranslatableMixin
 from wagtail.permission_policies import BlanketPermissionPolicy, ModelPermissionPolicy
@@ -131,6 +132,15 @@ class BaseChooseView(
     results_template_name = "wagtailadmin/generic/chooser/results.html"
     construct_queryset_hook_name = None
     url_filter_parameters = []
+    paginator_class = WagtailPaginator
+
+    @cached_property
+    def i18n_enabled(self) -> bool:
+        return (
+            getattr(settings, "WAGTAIL_I18N_ENABLED", False)
+            and self.model_class is not None
+            and issubclass(self.model_class, TranslatableMixin)
+        )
 
     def get_object_list(self):
         return self.model_class.objects.all()
@@ -160,8 +170,7 @@ class BaseChooseView(
                 if issubclass(self.model_class, CollectionMember):
                     bases.insert(0, CollectionFilterMixin)
 
-                i18n_enabled = getattr(settings, "WAGTAIL_I18N_ENABLED", False)
-                if i18n_enabled and issubclass(self.model_class, TranslatableMixin):
+                if self.i18n_enabled:
                     bases.insert(0, LocaleFilterMixin)
 
             return type(
@@ -206,8 +215,11 @@ class BaseChooseView(
         return self.request.GET.get("multiple")
 
     @property
-    def columns(self):
-        return [self.title_column]
+    def columns(self) -> list[Column]:
+        columns = [self.title_column]
+        if self.i18n_enabled:
+            columns.append(LocaleColumn(sort_key=None))
+        return columns
 
     @property
     def title_column(self):
@@ -242,9 +254,9 @@ class BaseChooseView(
         objects = self.apply_object_list_ordering(objects)
         objects = self.filter_object_list(objects)
 
-        paginator = Paginator(objects, per_page=self.per_page)
+        self.paginator = self.paginator_class(objects, per_page=self.per_page)
         try:
-            return paginator.page(request.GET.get("p", 1))
+            return self.paginator.page(request.GET.get("p", 1))
         except InvalidPage:
             raise Http404
 
@@ -268,12 +280,17 @@ class BaseChooseView(
         # so that the pagination include can append its own parameters via the {% querystring %} template tag
         results_pagination_url = re.sub(r"\?.*$", "", results_url)
 
+        elided_page_range = self.paginator.get_elided_page_range(
+            self.request.GET.get("p", 1)
+        )
+
         context.update(
             {
                 "results": self.results,
                 "table": self.table,
                 "results_url": results_url,
                 "results_pagination_url": results_pagination_url,
+                "elided_page_range": elided_page_range,
                 "is_searching": self.filter_form.is_searching,
                 "is_filtering_by_collection": self.filter_form.is_filtering_by_collection,
                 "is_multiple_choice": self.is_multiple_choice,

wagtail/admin/views/generic/mixins.py

@@ -1,4 +1,5 @@
 import json
+from typing import Optional
 
 from django.conf import settings
 from django.contrib.admin.utils import quote
@@ -17,10 +18,11 @@ from django.utils.translation import gettext as _
 
 from wagtail import hooks
 from wagtail.admin import messages
+from wagtail.admin.forms.models import WagtailAdminModelForm
 from wagtail.admin.models import EditingSession
 from wagtail.admin.templatetags.wagtailadmin_tags import user_display_name
 from wagtail.admin.ui.editing_sessions import EditingSessionsModule
-from wagtail.admin.ui.tables import TitleColumn
+from wagtail.admin.ui.tables import LiveStatusTagColumn, TitleColumn
 from wagtail.admin.utils import get_latest_str, set_query_params
 from wagtail.locks import BasicLock, ScheduledForPublishLock, WorkflowLock
 from wagtail.log_actions import log
@@ -104,6 +106,14 @@ class BeforeAfterHookMixin(HookResponseMixin):
 
 
 class LocaleMixin:
+    @cached_property
+    def i18n_enabled(self) -> bool:
+        return (
+            getattr(settings, "WAGTAIL_I18N_ENABLED", False)
+            and (model := getattr(self, "model", None)) is not None
+            and issubclass(model, TranslatableMixin)
+        )
+
     @cached_property
     def locale(self):
         return self.get_locale()
@@ -112,19 +122,17 @@ class LocaleMixin:
     def translations(self):
         return self.get_translations() if self.locale else []
 
-    def get_locale(self):
+    def get_locale(self) -> Optional[Locale]:
         if not getattr(self, "model", None):
             return None
 
-        i18n_enabled = getattr(settings, "WAGTAIL_I18N_ENABLED", False)
-        if not i18n_enabled or not issubclass(self.model, TranslatableMixin):
+        if not self.i18n_enabled:
             return None
 
         if hasattr(self, "object") and self.object:
             return self.object.locale
 
-        selected_locale = self.request.GET.get("locale")
-        if selected_locale:
+        if selected_locale := self.request.GET.get("locale"):
             return get_object_or_404(Locale, language_code=selected_locale)
         return Locale.get_default()
 
@@ -218,6 +226,13 @@ class IndexViewOptionalFeaturesMixin:
             return queryset
         return super()._annotate_queryset_updated_at(queryset)
 
+    @cached_property
+    def list_display(self):
+        list_display = super().list_display.copy()
+        if issubclass(self.model, DraftStateMixin):
+            list_display.append(LiveStatusTagColumn())
+        return list_display
+
 
 class CreateEditViewOptionalFeaturesMixin:
     """
@@ -254,6 +269,11 @@ class CreateEditViewOptionalFeaturesMixin:
         self.lock = self.get_lock()
         self.locked_for_user = self.lock and self.lock.for_user(request.user)
         super().setup(request, *args, **kwargs)
+        self.saving_as_draft = (
+            self.draftstate_enabled
+            and request.method == "POST"
+            and self.action in ("create", "edit")
+        )
 
     @cached_property
     def workflow(self):
@@ -502,7 +522,9 @@ class CreateEditViewOptionalFeaturesMixin:
         # Save revision if the model inherits from RevisionMixin
         self.new_revision = None
         if self.revision_enabled:
-            self.new_revision = instance.save_revision(user=self.request.user)
+            self.new_revision = instance.save_revision(
+                user=self.request.user, clean=not self.saving_as_draft
+            )
 
         log(
             instance=instance,
@@ -723,9 +745,9 @@ class CreateEditViewOptionalFeaturesMixin:
         context["draftstate_enabled"] = self.draftstate_enabled
         context["workflow_enabled"] = self.workflow_enabled
         context["workflow_history_url"] = self.get_workflow_history_url()
-        context[
-            "confirm_workflow_cancellation_url"
-        ] = self.get_confirm_workflow_cancellation_url()
+        context["confirm_workflow_cancellation_url"] = (
+            self.get_confirm_workflow_cancellation_url()
+        )
         context["publishing_will_cancel_workflow"] = getattr(
             settings, "WAGTAIL_WORKFLOW_CANCEL_ON_PUBLISH", True
         ) and bool(self.workflow_tasks)
@@ -735,8 +757,20 @@ class CreateEditViewOptionalFeaturesMixin:
 
     def post(self, request, *args, **kwargs):
         form = self.get_form()
+
         # Make sure object is not locked
-        if not self.locked_for_user and form.is_valid():
+        if self.locked_for_user:
+            return self.form_invalid(form)
+
+        # If saving as draft, do not enforce full validation
+        if self.saving_as_draft and isinstance(form, WagtailAdminModelForm):
+            form.defer_required_fields()
+            form_is_valid = form.is_valid()
+            form.restore_required_fields()
+        else:
+            form_is_valid = form.is_valid()
+
+        if form_is_valid:
             return self.form_valid(form)
         else:
             return self.form_invalid(form)

wagtail/admin/views/generic/models.py

@@ -1,5 +1,3 @@
-import warnings
-
 from django.contrib.admin.utils import label_for_field, quote, unquote
 from django.contrib.contenttypes.models import ContentType
 from django.core.exceptions import (
@@ -35,6 +33,7 @@ from wagtail.admin.ui.side_panels import StatusSidePanel
 from wagtail.admin.ui.tables import (
     ButtonsColumnMixin,
     Column,
+    LocaleColumn,
     TitleColumn,
     UpdatedAtColumn,
 )
@@ -51,7 +50,6 @@ from wagtail.log_actions import registry as log_registry
 from wagtail.models import DraftStateMixin, Locale, ReferenceIndex
 from wagtail.models.audit_log import ModelLogEntry
 from wagtail.search.index import class_is_indexed
-from wagtail.utils.deprecation import RemovedInWagtail70Warning
 
 from .base import BaseListingView, WagtailAdminTemplateMixin
 from .mixins import BeforeAfterHookMixin, HookResponseMixin, LocaleMixin, PanelMixin
@@ -73,28 +71,9 @@ class IndexView(
     inspect_url_name = None
     delete_url_name = None
     any_permission_required = ["add", "change", "delete", "view"]
-    columns = None  # If not explicitly specified, will be derived from list_display
-    list_display = ["__str__", UpdatedAtColumn()]
     list_filter = None
     show_other_searches = False
 
-    def get_search_url(self):
-        # This is only used by views that do not use breadcrumbs, thus uses the
-        # legacy header.html. The search in that header template accepts both
-        # the search_url (which really should be search_url_name) and the
-        # index_results_url. This means we can advise using the latter instead,
-        # without having to instruct how to set up breadcrumbs.
-        warnings.warn(
-            "`IndexView.get_search_url` is deprecated. "
-            "Use `IndexView.get_index_results_url` instead.",
-            RemovedInWagtail70Warning,
-        )
-        return self.index_url_name
-
-    @cached_property
-    def search_url(self):
-        return self.get_search_url()
-
     @cached_property
     def is_searchable(self):
         # Do not automatically enable search if the model is not indexed and
@@ -256,8 +235,16 @@ class IndexView(
             **kwargs,
         )
 
+    @cached_property
+    def list_display(self):
+        list_display = ["__str__", UpdatedAtColumn()]
+        if self.i18n_enabled:
+            list_display.insert(1, LocaleColumn())
+        return list_display
+
     @cached_property
     def columns(self):
+        # If not explicitly overridden, derive from list_display
         columns = []
         for i, field in enumerate(self.list_display):
             if isinstance(field, Column):
@@ -405,12 +392,6 @@ class IndexView(
             context["add_url"] = context["header_action_url"] = self.add_url
             context["header_action_label"] = self.add_item_label
 
-        # RemovedInWagtail70Warning:
-        # Remove these in favor of using search_form and index_results_url
-        if self.is_searchable and not self.index_results_url:
-            context["is_searchable"] = self.is_searchable
-            context["search_url"] = self.search_url
-
         context["model_opts"] = self.model and self.model._meta
         return context
 
@@ -1124,8 +1105,13 @@ class InspectView(PermissionCheckedMixin, WagtailAdminTemplateMixin, TemplateVie
         return capfirst(label_for_field(field_name, model=self.model))
 
     def get_field_display_value(self, field_name, field):
-        # First we check for a 'get_fieldname_display' property/method on
+        # First we check for a `get_fieldname_display_value` method on the InspectView
+        # then for a 'get_fieldname_display' property/method on
         # the model, and return the value of that, if present.
+        value_func = getattr(self, f"get_{field_name}_display_value", None)
+        if value_func is not None and callable(value_func):
+            return value_func()
+
         value_func = getattr(self.object, "get_%s_display" % field_name, None)
         if value_func is not None:
             if callable(value_func):
@@ -1190,7 +1176,6 @@ class RevisionsCompareView(WagtailAdminTemplateMixin, TemplateView):
     history_label = gettext_lazy("History")
     page_title = gettext_lazy("Compare")
     template_name = "wagtailadmin/generic/revisions/compare.html"
-    _show_breadcrumbs = True
     model = None
 
     def get_breadcrumbs_items(self):

wagtail/admin/views/generic/preview.py

@@ -11,6 +11,7 @@ from django.utils.functional import cached_property
 from django.utils.translation import gettext
 from django.views.generic import TemplateView, View
 
+from wagtail.admin.forms.models import WagtailAdminModelForm
 from wagtail.admin.panels import get_edit_handler
 from wagtail.blocks.base import Block
 from wagtail.models import PreviewableMixin, RevisionMixin
@@ -77,10 +78,15 @@ class PreviewOnEdit(View):
             post_data = ""
         return QueryDict(post_data)
 
+    def validate_form(self, form):
+        if isinstance(form, WagtailAdminModelForm):
+            form.defer_required_fields()
+        return form.is_valid()
+
     def post(self, request, *args, **kwargs):
         self.remove_old_preview_data()
         form = self.get_form(request.POST)
-        is_valid = form.is_valid()
+        is_valid = self.validate_form(form)
 
         if is_valid:
             # TODO: Handle request.FILES.
@@ -89,7 +95,7 @@ class PreviewOnEdit(View):
         else:
             # Check previous data in session to determine preview availability
             form = self.get_form(self._get_data_from_session())
-            is_available = form.is_valid()
+            is_available = self.validate_form(form)
 
         return JsonResponse({"is_valid": is_valid, "is_available": is_available})
 
@@ -104,7 +110,7 @@ class PreviewOnEdit(View):
     def get(self, request, *args, **kwargs):
         form = self.get_form(self._get_data_from_session())
 
-        if not form.is_valid():
+        if not self.validate_form(form):
             return self.error_response()
 
         form.save(commit=False)