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)
Change History (2)
by , 17 years ago
| Attachment: | RelatedObjectLookups.js.patch added |
|---|
comment:1 by , 17 years ago
| Resolution: | → duplicate |
|---|---|
| Status: | new → closed |
Note:
See TracTickets
for help on using tickets.
Duplicate of #6100.