Django

Code

Ticket #6104 (closed: duplicate)

Opened 1 year ago

Last modified 1 year ago

admin ManyToMany widget: js broken for 'add another' popup

Reported by: Steven Armstrong 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

In RelatedObjectLookups?.js dismissRelatedLookupPopup get's passed an integer which is then run through html_unescape.

html_unescape errors out while trying to call a .replace method on the int.

The effect of this is that the change-form popup stays open and the value in the select element doesn't get updated.

The attached patch fixes that by ensuring the arguments of html_unescape are string.

Attachments

RelatedObjectLookups.js.patch (0.6 kB) - added by Steven Armstrong on 12/02/07 14:08:51.

Change History

12/02/07 14:08:51 changed by Steven Armstrong

  • attachment RelatedObjectLookups.js.patch added.

12/03/07 16:20:46 changed by brosner

  • status changed from new to closed.
  • needs_better_patch changed.
  • resolution set to duplicate.
  • needs_tests changed.
  • needs_docs changed.

Duplicate of #6100.

Add/Change #6104 (admin ManyToMany widget: js broken for 'add another' popup)




Change Properties
Action