fix: Fixed bug that causes ChangeView not to use SnippetForm (#80)

Author: adam-murray

djangocms_snippet/admin.py

@@ -5,6 +5,7 @@
 from django.forms import Textarea
 
 from .cms_config import SnippetCMSAppConfig
+from .forms import SnippetForm
 from .models import Snippet
 from .views import SnippetPreviewView
 
@@ -33,7 +34,7 @@ class SnippetAdmin(*snippet_admin_classes):
         'data-mode': getattr(settings, 'DJANGOCMS_SNIPPET_THEME', 'html'),
         'data-theme': getattr(settings, 'DJANGOCMS_SNIPPET_MODE', 'github'),
     }
-
+    form = SnippetForm
     formfield_overrides = {
         models.TextField: {'widget': Textarea(attrs=text_area_attrs)}
     }

djangocms_snippet/forms.py

@@ -28,6 +28,7 @@ class Meta:
     def __init__(self, *args, **kwargs):
         super().__init__(*args, **kwargs)
         self.fields["snippet_grouper"].required = False
+        self.fields["snippet_grouper"].widget = forms.HiddenInput()
 
     def clean(self):
         data = super().clean()

tests/test_admin.py

@@ -7,6 +7,7 @@
 
 from djangocms_snippet import admin as snippet_admin
 from djangocms_snippet import cms_config
+from djangocms_snippet.forms import SnippetForm
 from djangocms_snippet.models import Snippet
 
 from .utils.factories import SnippetWithVersionFactory
@@ -51,3 +52,10 @@ def test_admin_list_display_with_versioning(self):
         )
         self.assertEqual(list_display[-1].short_description, 'actions')
         self.assertIn("function ExtendedVersionAdminMixin._list_actions", list_display[-1].__str__())
+
+    def test_admin_uses_form(self):
+        """
+        The SnippetForm provides functionality to make SnippetGroupers irrelevant to the user,
+        ensure the admin uses this.
+        """
+        self.assertEqual(self.snippet_admin.form, SnippetForm)