django admin: radio_admin is broken with addPopup and RelatedObjectLookups.js
|Reported by:||trbs||Owned by:||nobody|
|Severity:||Keywords:||radio_admin, relatedobjectlookups nfa_someday|
|Cc:||Triage Stage:||Design decision needed|
|Has patch:||yes||Needs documentation:||no|
|Needs tests:||no||Patch needs improvement:||no|
It seems that radio_admin is broken in trunk's admin for adding new items through popup's.
django.contrib.redirects is an easy test case, create a project, put django.contrib.redirects in it and get the error when trying to add a new (with the plus symbol) record to the database and radio selection field.
Error message: (firebug)
SelectBox is not defined http://localhost:8000/media/js/admin/RelatedObjectLookups.js Line 73
Since radio fields are no more in newforms-admin, it IMHO makes the most sense to remove the radio_admin option from the model in django.contrib.redirects (path attached)
Maybe also marking this option as deprecated in the documentation (http://www.djangoproject.com/documentation/model-api/) might be useful as it would disappear in nfa and seems to be currently broken (for a long time now? as i cannot find any revision that did handle adding items to the radio selection on page) for adding new items by popup in the admin pages.
Change History (5)
Changed 6 years ago by trbs
comment:1 Changed 6 years ago by trbs
- Needs documentation unset
- Needs tests unset
- Patch needs improvement unset
comment:2 Changed 6 years ago by Karen Tracey <kmtracey@…>
- Keywords nfa_someday added
- Triage Stage changed from Unreviewed to Design decision needed
- Version changed from SVN to newforms-admin