0
0
mirror of https://github.com/wagtail/wagtail.git synced 2024-12-01 11:41:20 +01:00

Rename snippets view classes to be more explicit

This commit is contained in:
Sage Abdullah 2022-09-27 08:27:48 +01:00 committed by Matt Westcott
parent f3b8a7df27
commit e7cb9b1c45
2 changed files with 39 additions and 42 deletions

View File

@ -26,15 +26,12 @@ from wagtail.admin.ui.tables import (
TitleColumn, TitleColumn,
UserColumn, UserColumn,
) )
from wagtail.admin.views.generic import CreateView, DeleteView, EditView, IndexView from wagtail.admin.views import generic
from wagtail.admin.views.generic.mixins import RevisionsRevertMixin from wagtail.admin.views.generic.mixins import RevisionsRevertMixin
from wagtail.admin.views.generic.models import RevisionsCompareView, UnpublishView
from wagtail.admin.views.generic.permissions import PermissionCheckedMixin from wagtail.admin.views.generic.permissions import PermissionCheckedMixin
from wagtail.admin.views.generic.preview import ( from wagtail.admin.views.generic.preview import PreviewOnCreate as PreviewOnCreateView
PreviewOnCreate, from wagtail.admin.views.generic.preview import PreviewOnEdit as PreviewOnEditView
PreviewOnEdit, from wagtail.admin.views.generic.preview import PreviewRevision
PreviewRevision,
)
from wagtail.admin.views.reports.base import ReportView from wagtail.admin.views.reports.base import ReportView
from wagtail.admin.viewsets.base import ViewSet from wagtail.admin.viewsets.base import ViewSet
from wagtail.log_actions import log from wagtail.log_actions import log
@ -78,7 +75,7 @@ def get_snippet_edit_handler(model):
# == Views == # == Views ==
class Index(IndexView): class ModelIndexView(generic.IndexView):
template_name = "wagtailadmin/generic/index.html" template_name = "wagtailadmin/generic/index.html"
page_title = gettext_lazy("Snippets") page_title = gettext_lazy("Snippets")
header_icon = "snippet" header_icon = "snippet"
@ -150,7 +147,7 @@ class SnippetTitleColumn(TitleColumn):
cell_template_name = "wagtailsnippets/snippets/tables/title_cell.html" cell_template_name = "wagtailsnippets/snippets/tables/title_cell.html"
class List(IndexView): class IndexView(generic.IndexView):
view_name = "list" view_name = "list"
index_results_url_name = None index_results_url_name = None
delete_multiple_url_name = None delete_multiple_url_name = None
@ -203,7 +200,7 @@ class List(IndexView):
return ["wagtailsnippets/snippets/type_index.html"] return ["wagtailsnippets/snippets/type_index.html"]
class Create(CreateView): class CreateView(generic.CreateView):
view_name = "create" view_name = "create"
preview_url_name = None preview_url_name = None
permission_required = "add" permission_required = "add"
@ -301,7 +298,7 @@ class Create(CreateView):
return context return context
class Edit(EditView): class EditView(generic.EditView):
view_name = "edit" view_name = "edit"
history_url_name = None history_url_name = None
preview_url_name = None preview_url_name = None
@ -414,7 +411,7 @@ class Edit(EditView):
return context return context
class Delete(DeleteView): class DeleteView(generic.DeleteView):
view_name = "delete" view_name = "delete"
delete_multiple_url_name = None delete_multiple_url_name = None
permission_required = "delete" permission_required = "delete"
@ -494,7 +491,7 @@ class Delete(DeleteView):
return context return context
class Usage(IndexView): class UsageView(generic.IndexView):
view_name = "usage" view_name = "usage"
template_name = "wagtailsnippets/snippets/usage.html" template_name = "wagtailsnippets/snippets/usage.html"
paginate_by = 20 paginate_by = 20
@ -594,7 +591,7 @@ class ActionColumn(Column):
return context return context
class History(ReportView): class HistoryView(ReportView):
view_name = "history" view_name = "history"
index_url_name = None index_url_name = None
edit_url_name = None edit_url_name = None
@ -650,11 +647,11 @@ class History(ReportView):
) )
class RevisionsView(PermissionCheckedMixin, PreviewRevision): class PreviewRevisionView(PermissionCheckedMixin, PreviewRevision):
permission_required = "change" permission_required = "change"
class RevisionsCompare(PermissionCheckedMixin, RevisionsCompareView): class RevisionsCompareView(PermissionCheckedMixin, generic.models.RevisionsCompareView):
permission_required = "change" permission_required = "change"
header_icon = "snippet" header_icon = "snippet"
@ -671,7 +668,7 @@ class RevisionsCompare(PermissionCheckedMixin, RevisionsCompareView):
) )
class Unpublish(PermissionCheckedMixin, UnpublishView): class UnpublishView(PermissionCheckedMixin, generic.models.UnpublishView):
permission_required = "publish" permission_required = "publish"
@ -683,38 +680,38 @@ class SnippetViewSet(ViewSet):
#: A subclass of ``wagtail.admin.filters.WagtailFilterSet``, which is a subclass of `django_filters.FilterSet <https://django-filter.readthedocs.io/en/stable/ref/filterset.html>`_. This will be passed to the ``filterset_class`` attribute of the index view. #: A subclass of ``wagtail.admin.filters.WagtailFilterSet``, which is a subclass of `django_filters.FilterSet <https://django-filter.readthedocs.io/en/stable/ref/filterset.html>`_. This will be passed to the ``filterset_class`` attribute of the index view.
filterset_class = None filterset_class = None
#: The view class to use for the list view; must be a subclass of ``wagtail.snippet.views.snippets.List``. #: The view class to use for the index view; must be a subclass of ``wagtail.snippet.views.snippets.IndexView``.
index_view_class = List index_view_class = IndexView
#: The view class to use for the create view; must be a subclass of ``wagtail.snippet.views.snippets.Create``. #: The view class to use for the create view; must be a subclass of ``wagtail.snippet.views.snippets.CreateView``.
add_view_class = Create add_view_class = CreateView
#: The view class to use for the edit view; must be a subclass of ``wagtail.snippet.views.snippets.Edit``. #: The view class to use for the edit view; must be a subclass of ``wagtail.snippet.views.snippets.EditView``.
edit_view_class = Edit edit_view_class = EditView
#: The view class to use for the delete view; must be a subclass of ``wagtail.snippet.views.snippets.Delete``. #: The view class to use for the delete view; must be a subclass of ``wagtail.snippet.views.snippets.DeleteView``.
delete_view_class = Delete delete_view_class = DeleteView
#: The view class to use for the usage view; must be a subclass of ``wagtail.snippet.views.snippets.Usage``. #: The view class to use for the usage view; must be a subclass of ``wagtail.snippet.views.snippets.UsageView``.
usage_view_class = Usage usage_view_class = UsageView
#: The view class to use for the history view; must be a subclass of ``wagtail.snippet.views.snippets.History``. #: The view class to use for the history view; must be a subclass of ``wagtail.snippet.views.snippets.HistoryView``.
history_view_class = History history_view_class = HistoryView
#: The view class to use for previewing revisions; must be a subclass of ``wagtail.snippet.views.snippets.RevisionsView``. #: The view class to use for previewing revisions; must be a subclass of ``wagtail.snippet.views.snippets.PreviewRevisionView``.
revisions_view_class = RevisionsView revisions_view_class = PreviewRevisionView
#: The view class to use for comparing revisions; must be a subclass of ``wagtail.snippet.views.snippets.RevisionsCompare``. #: The view class to use for comparing revisions; must be a subclass of ``wagtail.snippet.views.snippets.RevisionsCompareView``.
revisions_compare_view_class = RevisionsCompare revisions_compare_view_class = RevisionsCompareView
#: The view class to use for unpublishing a snippet; must be a subclass of ``wagtail.snippet.views.snippets.Unpublish``. #: The view class to use for unpublishing a snippet; must be a subclass of ``wagtail.snippet.views.snippets.UnpublishView``.
unpublish_view_class = Unpublish unpublish_view_class = UnpublishView
#: The view class to use for previewing on the create view; must be a subclass of ``wagtail.snippet.views.snippets.PreviewOnCreate``. #: The view class to use for previewing on the create view; must be a subclass of ``wagtail.snippet.views.snippets.PreviewOnCreateView``.
preview_on_add_view_class = PreviewOnCreate preview_on_add_view_class = PreviewOnCreateView
#: The view class to use for previewing on the edit view; must be a subclass of ``wagtail.snippet.views.snippets.PreviewOnEdit``. #: The view class to use for previewing on the edit view; must be a subclass of ``wagtail.snippet.views.snippets.PreviewOnEditView``.
preview_on_edit_view_class = PreviewOnEdit preview_on_edit_view_class = PreviewOnEditView
@property @property
def revisions_revert_view_class(self): def revisions_revert_view_class(self):
@ -723,11 +720,11 @@ class SnippetViewSet(ViewSet):
By default, this class is generated by combining the edit view with By default, this class is generated by combining the edit view with
``wagtail.admin.views.generic.mixins.RevisionsRevertMixin``. As a result, ``wagtail.admin.views.generic.mixins.RevisionsRevertMixin``. As a result,
this class must be a subclass of ``wagtail.snippet.views.snippets.Edit`` this class must be a subclass of ``wagtail.snippet.views.snippets.EditView``
and must handle the reversion correctly. and must handle the reversion correctly.
""" """
revisions_revert_view_class = type( revisions_revert_view_class = type(
"_RevisionsRevert", "_RevisionsRevertView",
(RevisionsRevertMixin, self.edit_view_class), (RevisionsRevertMixin, self.edit_view_class),
{"view_name": "revisions_revert"}, {"view_name": "revisions_revert"},
) )

View File

@ -21,7 +21,7 @@ from wagtail.snippets.widgets import SnippetListingButton
def register_admin_urls(): def register_admin_urls():
snippet_index_patterns = ( snippet_index_patterns = (
[ [
path("", snippet_views.Index.as_view(), name="index"), path("", snippet_views.ModelIndexView.as_view(), name="index"),
], ],
"wagtailsnippets", "wagtailsnippets",
) )