Merge pull request #666 from bckohan/issue546

Add playwright test capability, fix autocomplete bug on new form creation on stacked admin inline pages.

Author: bckohan

.gitignore

@@ -218,3 +218,6 @@ __marimo__/
 .DS_Store
 
 .python-version
+
+test1.db
+test2.db

conftest.py

@@ -0,0 +1,5 @@
+def pytest_configure(config):
+    # stash it somewhere global-ish
+    from polymorphic import tests
+
+    tests.HEADLESS = not config.getoption("--headed")

docs/changelog.rst

@@ -1,9 +1,10 @@
 Changelog
 =========
 
-v4.2.0 (2025-XX-XX)
+v4.2.0 (2025-12-01)
 -------------------
 
+* Fixed `Issue with Autocomplete Fields in StackedPolymorphicInline.Child Inline <https://github.com/jazzband/django-polymorphic/issues/546>`_
 * Support Python 3.14 and Django 6.0, drop support for EOL python 3.9, Django 3.2, 4.0, 4.1 and 5.0.
 * `Modernized package management with new build, test, docs tooling and improved CI 
   <https://github.com/jazzband/django-polymorphic/issues/651>`_.

justfile

@@ -16,6 +16,7 @@ manage *COMMAND:
     from django.core import management
     sys.path.append(os.getcwd())
     os.environ["DJANGO_SETTINGS_MODULE"] = "polymorphic.tests.settings"
+    os.environ["SQLITE_DATABASES"] = "test1.db,test2.db"
     management.execute_from_command_line(sys.argv + "{{ COMMAND }}".split(" "))
 
 # install the uv package manager
@@ -29,10 +30,11 @@ install_uv:
 install_uv:
     powershell -ExecutionPolicy ByPass -c "irm https://astral.sh/uv/install.ps1 | iex"
 
-# setup the venv and pre-commit hooks
+# setup the venv, pre-commit hooks and playwright dependencies
 setup python="python":
     uv venv -p {{ python }}
     @just run pre-commit install
+    @just run playwright install
 
 # install git pre-commit hooks
 install-precommit:

pyproject.toml

@@ -151,6 +151,7 @@ dev = [
     "pytest>=8.3.4",
     "pytest-cov>=5.0.0",
     "pytest-django>=4.10.0",
+    "pytest-playwright>=0.7.2",
     "ruff>=0.9.8",
     "tomlkit>=0.13.3",
     "tox>=4.24.1",

src/polymorphic/static/polymorphic/js/polymorphic_inlines.js

@@ -140,7 +140,11 @@
                     if (options.removed) {
                         options.removed(row);
                     }
-                    $(document).trigger('formset:removed', [row, options.prefix]);
+                    document.dispatchEvent(new CustomEvent("formset:removed", {
+                        detail: {
+                            formsetName: options.prefix
+                        }
+                    }));
                     // Update the TOTAL_FORMS form count.
                     var forms = $("." + options.formCssClass);
                     $("#id_" + options.prefix + "-TOTAL_FORMS").val(forms.length);
@@ -163,7 +167,12 @@
                 if (options.added) {
                     options.added(row);
                 }
-                $(document).trigger('formset:added', [row, options.prefix]);
+                row.get(0).dispatchEvent(new CustomEvent("formset:added", {
+                    bubbles: true,
+                    detail: {
+                        formsetName: options.prefix
+                    }
+                }));
             });
         }
         return this;

src/polymorphic/tests/__init__.py

@@ -0,0 +1 @@
+HEADLESS = True

src/polymorphic/tests/admin.py

@@ -0,0 +1,27 @@
+from django.contrib.admin import register, ModelAdmin
+from polymorphic.admin import StackedPolymorphicInline, PolymorphicInlineSupportMixin
+from polymorphic.tests.models import PlainA, InlineModelA, InlineModelB, InlineParent
+
+
+@register(PlainA)
+class PlainAAdmin(ModelAdmin):
+    search_fields = ["field1"]
+
+
+class InlineModelAChild(StackedPolymorphicInline.Child):
+    model = InlineModelA
+
+
+class InlineModelBChild(StackedPolymorphicInline.Child):
+    model = InlineModelB
+    autocomplete_fields = ["plain_a"]
+
+
+class Inline(StackedPolymorphicInline):
+    model = InlineModelA
+    child_inlines = (InlineModelAChild, InlineModelBChild)
+
+
+@register(InlineParent)
+class InlineParentAdmin(PolymorphicInlineSupportMixin, ModelAdmin):
+    inlines = (Inline,)

src/polymorphic/tests/migrations/0001_initial.py

@@ -1,4 +1,4 @@
-# Generated by Django 4.2.23 on 2025-08-25 13:28
+# Generated by Django 4.2.26 on 2025-12-01 11:40
 
 from django.db import migrations, models
 import django.db.models.deletion
@@ -89,17 +89,6 @@ class Migration(migrations.Migration):
             },
             bases=(polymorphic.showfields.ShowFieldType, models.Model),
         ),
-        migrations.CreateModel(
-            name='InlineModelA',
-            fields=[
-                ('id', models.AutoField(auto_created=True, primary_key=True, serialize=False, verbose_name='ID')),
-                ('field1', models.CharField(max_length=30)),
-            ],
-            options={
-                'abstract': False,
-                'base_manager_name': 'objects',
-            },
-        ),
         migrations.CreateModel(
             name='InlineParent',
             fields=[
@@ -352,18 +341,6 @@ class Migration(migrations.Migration):
             },
             bases=('tests.inittestmodel',),
         ),
-        migrations.CreateModel(
-            name='InlineModelB',
-            fields=[
-                ('inlinemodela_ptr', models.OneToOneField(auto_created=True, on_delete=django.db.models.deletion.CASCADE, parent_link=True, primary_key=True, serialize=False, to='tests.inlinemodela')),
-                ('field2', models.CharField(max_length=30)),
-            ],
-            options={
-                'abstract': False,
-                'base_manager_name': 'objects',
-            },
-            bases=('tests.inlinemodela',),
-        ),
         migrations.CreateModel(
             name='Middle',
             fields=[
@@ -820,15 +797,18 @@ class Migration(migrations.Migration):
             },
             bases=(polymorphic.showfields.ShowFieldType, models.Model),
         ),
-        migrations.AddField(
-            model_name='inlinemodela',
-            name='parent',
-            field=models.ForeignKey(on_delete=django.db.models.deletion.CASCADE, related_name='inline_children', to='tests.inlineparent'),
-        ),
-        migrations.AddField(
-            model_name='inlinemodela',
-            name='polymorphic_ctype',
-            field=models.ForeignKey(editable=False, null=True, on_delete=django.db.models.deletion.CASCADE, related_name='polymorphic_%(app_label)s.%(class)s_set+', to='contenttypes.contenttype'),
+        migrations.CreateModel(
+            name='InlineModelA',
+            fields=[
+                ('id', models.AutoField(auto_created=True, primary_key=True, serialize=False, verbose_name='ID')),
+                ('field1', models.CharField(max_length=30)),
+                ('parent', models.ForeignKey(on_delete=django.db.models.deletion.CASCADE, related_name='inline_children', to='tests.inlineparent')),
+                ('polymorphic_ctype', models.ForeignKey(editable=False, null=True, on_delete=django.db.models.deletion.CASCADE, related_name='polymorphic_%(app_label)s.%(class)s_set+', to='contenttypes.contenttype')),
+            ],
+            options={
+                'abstract': False,
+                'base_manager_name': 'objects',
+            },
         ),
         migrations.CreateModel(
             name='Duck',
@@ -1070,6 +1050,19 @@ class Migration(migrations.Migration):
             ],
             bases=('tests.uuidplainb',),
         ),
+        migrations.CreateModel(
+            name='InlineModelB',
+            fields=[
+                ('inlinemodela_ptr', models.OneToOneField(auto_created=True, on_delete=django.db.models.deletion.CASCADE, parent_link=True, primary_key=True, serialize=False, to='tests.inlinemodela')),
+                ('field2', models.CharField(max_length=30)),
+                ('plain_a', models.ForeignKey(blank=True, default=None, null=True, on_delete=django.db.models.deletion.SET_NULL, to='tests.plaina')),
+            ],
+            options={
+                'abstract': False,
+                'base_manager_name': 'objects',
+            },
+            bases=('tests.inlinemodela',),
+        ),
         migrations.CreateModel(
             name='BlogEntry',
             fields=[

src/polymorphic/tests/models.py

@@ -15,6 +15,9 @@
 class PlainA(models.Model):
     field1 = models.CharField(max_length=30)
 
+    def __str__(self):
+        return self.field1
+
 
 class PlainB(PlainA):
     field2 = models.CharField(max_length=30)
@@ -448,6 +451,10 @@ class InlineModelA(PolymorphicModel):
 class InlineModelB(InlineModelA):
     field2 = models.CharField(max_length=30)
 
+    plain_a = models.ForeignKey(
+        PlainA, null=True, blank=True, default=None, on_delete=models.SET_NULL
+    )
+
 
 class AbstractProject(PolymorphicModel):
     topic = models.CharField(max_length=30)