From 03974d81220ffd237754a82c77913799dd5909a4 Mon Sep 17 00:00:00 2001 From: Vasiliy Bolshakov Date: Fri, 1 Dec 2017 03:19:02 +0700 Subject: [PATCH] Fixed #28866 -- Made InlineAdminFormSet include InlineModelAdmin's Media before its formset's Media. This provides better backwards compatibility following refs #28377. --- django/contrib/admin/helpers.py | 2 +- tests/admin_inlines/admin.py | 9 +++++++++ tests/admin_inlines/tests.py | 10 ++++++++++ 3 files changed, 20 insertions(+), 1 deletion(-) diff --git a/django/contrib/admin/helpers.py b/django/contrib/admin/helpers.py index 2b47ed07fa2c0..58e92c9689b07 100644 --- a/django/contrib/admin/helpers.py +++ b/django/contrib/admin/helpers.py @@ -304,7 +304,7 @@ def non_form_errors(self): @property def media(self): - media = self.formset.media + self.opts.media + media = self.opts.media + self.formset.media for fs in self: media = media + fs.media return media diff --git a/tests/admin_inlines/admin.py b/tests/admin_inlines/admin.py index 4a34d65d83e35..1370484313e5d 100644 --- a/tests/admin_inlines/admin.py +++ b/tests/admin_inlines/admin.py @@ -1,5 +1,6 @@ from django import forms from django.contrib import admin +from django.db import models from .models import ( Author, BinaryTree, CapoFamiglia, Chapter, ChildModel1, ChildModel2, @@ -73,8 +74,16 @@ class Media: js = ('my_awesome_inline_scripts.js',) +class CustomNumberWidget(forms.NumberInput): + class Media: + js = ('custom_number.js',) + + class InnerInline3(admin.StackedInline): model = Inner3 + formfield_overrides = { + models.IntegerField: {'widget': CustomNumberWidget}, + } class Media: js = ('my_awesome_inline_scripts.js',) diff --git a/tests/admin_inlines/tests.py b/tests/admin_inlines/tests.py index 13b179fc15542..67ffa7648fb62 100644 --- a/tests/admin_inlines/tests.py +++ b/tests/admin_inlines/tests.py @@ -453,6 +453,16 @@ def test_inline_media_only_inline(self): Inner3(dummy=42, holder=holder).save() change_url = reverse('admin:admin_inlines_holder3_change', args=(holder.id,)) response = self.client.get(change_url) + self.assertEqual( + response.context['inline_admin_formsets'][0].media._js, + [ + 'admin/js/vendor/jquery/jquery.min.js', + 'admin/js/jquery.init.js', + 'admin/js/inlines.min.js', + 'my_awesome_inline_scripts.js', + 'custom_number.js', + ] + ) self.assertContains(response, 'my_awesome_inline_scripts.js') def test_all_inline_media(self):