Fix / Add support for radiobuttons after adding item with popup in Django Admin

[trbs]

The following patch adds support for dynamically adding radiobutton items to the admin page.

Without this patch the newly added item through the popup menu does not get added to the page and
the user must reload the page before being able to select the new item.

The patch adds an extra case to dismissAddAnotherPopup in RelatedObjectsLookup.js which creates and appends the necessary html to the list of radiobuttons on the page.

Tested with django.contrib.redirects and a slightly modified version of django-podcast.
Works (for me) on Firefox, IE and Opera.

[(none)]

Milestone post-1.0 deleted

[bodiddlie]

Fixes some errors in previous patch and updates to work with latest svn

[bodiddlie]

Made some changes to the patch to make it work properly with IE and to correctly check for the radiolist class whether it's inline or not. Also checks whether added content is a tag or not since I've been using it to render a list of images as radio buttons so the user doesn't have to remember file names.

[trbs]

For clarity / future reference; What was the reason for resolution: worksforme ?

[Karen Tracey]

Replying to trbs:

For clarity / future reference; What was the reason for resolution: worksforme ?

Reopening as it doesn't appear that it should have been closed. There's a recently updated patch so that implies something needs to be changed in the code base -- until this code is actually committed in the SVN repository the ticket state should not be set to closed. Presence of a working patch is not enough as work still needs to be done to make the fix or new function available in general.

If it was closed "worksforme" because further investigation revealed no changes were needed in the code base, then that needs to be made clear by a comment on the close. As it is this looks to have been closed prematurely.

[trbs]

Thanks kmtracey.

I imaging admin-ui branch impacting on this ticket as well.

Since they appear to use JQuery and thus all those dom-operations, like adding entries to radiobuttons, comboboxes, etc
could hopefully be done much cleaner by using the js framework.

[bodiddlie]

Doh, that was a total misread on how the ticket system works on my part. First time I've ever submitted patch for anything. My bad. :( Definitely not meant to be closed yet.

[James Bennett]

1.2 is feature-frozen, moving this feature request off the milestone.

[Julien Phalip]

This is probably related to #14660 (which also has a patch).

[Tim Graham]

Patch will need to be updated to apply cleanly + tests.