Opened 17 years ago

Closed 17 years ago

#6104 closed (duplicate)

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

Reported by: Steven Armstrong Owned by: nobody
Component: contrib.admin Version: newforms-admin
Severity: Keywords:
Cc: Triage Stage: Unreviewed
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

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 (1)

RelatedObjectLookups.js.patch (607 bytes ) - added by Steven Armstrong 17 years ago.

Download all attachments as: .zip

Change History (2)

by Steven Armstrong, 17 years ago

comment:1 by Brian Rosner, 17 years ago

Resolution: duplicate
Status: newclosed

Duplicate of #6100.

Note: See TracTickets for help on using tickets.
Back to Top