Django

Code

Ticket #6059 (closed: fixed)

Opened 11 months ago

Last modified 11 months ago

newforms-admin extra javascript is getting escaped

Reported by: Karen Tracey <kmtracey@gmail.com> Assigned to: nobody
Milestone: Component: django.contrib.admin
Version: newforms-admin Keywords:
Cc: Triage Stage: Unreviewed
Has patch: 1 Needs documentation: 0
Needs tests: 0 Patch needs improvement: 0

Description

On change and add pages, I'm seeing escaped script tags on the top line:

<script type="text/javascript" src="/media/js/core.js"></script> <script type="text/javascript" src="/media/js/admin/RelatedObjectLookups.js"></script>

Attached patch fixes this by marking the 'media' template variable as safe.

Attachments

safe_media.diff (0.9 kB) - added by Karen Tracey <kmtracey@gmail.com> on 11/30/07 08:50:15.

Change History

11/30/07 08:50:15 changed by Karen Tracey <kmtracey@gmail.com>

  • attachment safe_media.diff added.

11/30/07 11:53:34 changed by jkocherhans

  • status changed from new to closed.
  • resolution set to fixed.

(In [6779]) newforms-admin: Fixed #6059. Javascript in the admin is no longer escaped.

11/30/07 11:54:36 changed by jkocherhans

  • needs_better_patch changed.
  • needs_tests changed.
  • needs_docs changed.

Oops. Forgot to thank Karen Tracy in the commit message. Sorry Karen, and thanks. :)

Add/Change #6059 (newforms-admin extra javascript is getting escaped)




Change Properties
Action