#28866: merge method of class Media doesn't work properly
---------------------------------+--------------------------------------
Reporter: clincher | Owner: nobody
Type: Bug | Status: new
Component: Forms | Version: 2.0
Severity: Release blocker | Resolution:
Keywords: forms, media | Triage Stage: Unreviewed
Has patch: 0 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
---------------------------------+--------------------------------------
Comment (by clincher):
Thanks for the pointing to my fault, @Tim
Thanks a lot for explanaition, @ Johannes
Then everything works fine in the merge method and the actual issue is not
in the forms app, but django.admin.helpers in the method media of
InlineAdminFormSet, line 307 shall be just changed from to
{{{
media = self.formset.media + self.opts.media
}}}
to
{{{
media = self.opts.media + self.formset.media
}}}
Because this is a place where it put opts.media (jquery.js,
jquery.init.js, inlines.js) before the formset.media where it actually
utilized.
Shall I create a patch or?
Cheers, Vasiliy
--
Ticket URL: <https://code.djangoproject.com/ticket/28866#comment:6>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.
--
You received this message because you are subscribed to the Google Groups
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
To post to this group, send email to [email protected].
To view this discussion on the web visit
https://groups.google.com/d/msgid/django-updates/066.61977004411c96bd70026e09f47cbdcd%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.
