id	summary	reporter	owner	description	type	status	component	version	severity	resolution	keywords	cc	stage	has_patch	needs_docs	needs_tests	needs_better_patch	easy	ui_ux
7133	django admin: radio_admin is broken with addPopup and RelatedObjectLookups.js	trbs	nobody	"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
}}}

After a little inspection of RelatedObjectLookups it seems to me that the entire javascript is not capable of handling radio_fields when adding new objects through the popup. The reason this error get's raised is because the javascript code handles it like it would be a combobox using the SelectBox javascript object instead of as a radio field. (The entire radio-field handling code is 'missing')

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. 

"		closed	contrib.admin	newforms-admin		duplicate	radio_admin, relatedobjectlookups nfa_someday		Design decision needed	1	0	0	0	0	0