Fixed #28357 -- Modified JavaScript for prepopulated fields

[Zotyamester]

Fixes #28357: The prepopulated_fields will work for the extra forms specified, in the admin.StackedInline, but in case of any additional forms that are added via "Add another " will not run the JS that pre populates the field. admin.TabularInline seems to work fine and is unaffected by this.

[github-actions]

Hello @Zotyamester! Thank you for your contribution 💪

As it's your first contribution be sure to check out the patch review checklist.

If you're fixing a ticket from Trac make sure to set the "Has patch" flag and include a link to this PR in the ticket!

If you have any design or process questions then you can ask in the Django forum.

Welcome aboard ⛵️!

[felixxm]

@Zotyamester Thanks for this patch 👍 A regression test is required.

[Zotyamester]

Sorry for that. I was silly, and missed the semicolon at the end of the function expression. 😅

[Zotyamester]

@felixxm Is it okay now? Or should I add more tests?

[felixxm]

@felixxm Is it okay now? Or should I add more tests?

I don't see any new tests in this patch 🤔

[Zotyamester]

Forgot to commit those. 😅
But here they are now.

[felixxm]

Forgot to commit those. sweat_smile But here they are now.

This test works without a patch, so it's not a proper regression test.

[Zotyamester]

And then how exactly should I write it? I'm asking it, because I haven't seen one test case that relied on Django models with prepopulated fields.

[felixxm]

And then how exactly should I write it? I'm asking it, because I haven't seen one test case that relied on Django models with prepopulated fields.

Have you seen the previous PR for this ticket? It contains a selenium test can be an inspiration for you.

• #28357 -- Prepopulated_fields doesn't work for admin.StackedInline #11221

[felixxm]

Updated in #15091.