[newforms-admin] - remove & replace radio_admin

Some discussion can be found here: http://groups.google.com/group/django-developers/browse_frm/thread/ea4d86b2b49c2647

Attached radio_admin_fields.diff, a patch which removes the old radio_admin Field attribute and replaces it with radio_admin_fields in ModelAdmin?. Here's some text for the newforms-admin branch page:

---

Moved radio_admin from the model definition

The syntax is now separated from the definition of your models.

An example:

# OLD:
class MyModel(models.Model):
    field1 = models.ForeignKey(AnotherModel, radio_admin=True)
    field2 = models.CharField(max_length=100, radio_admin=models.VERTICAL)
    class Admin:
        pass

# NEW:
class MyModel_Options(admin.ModelAdmin):
    model = MyModel
    radio_admin_fields = {'field1': admin.HORIZONTAL, 'field2': admin.VERTICAL }

---

The use of models.HORIZONTAL/VERTICAL was not documented, but it was used by django.contrib.redirects, so I maintained the old behavior. I included my best shot at some tests and doc...what doc I found for the new scheme is pretty bare bones so I just plunked down a revised version of the old doc around where it looks like the doc for ModelAdmin? specifics will go.

The second patch, 4080.diff, is a fix needed to make the display of radio_admin_fields look pretty, but it's really a separate issue so I split it out from the other patch. The code fix is part of the proposed fix for #4080 (the rest of that proposed fix has already been implemented through fixing some other ticket), plus 4080.diff fixes the testcases affected by the fix. #4080 was closed as a dup of #4117, which is a broader issue, and that one is waiting in design decision needed.

All that is needed for the radio_admin_fields display to look pretty is for the attrs specified on creation of the RadioSelect? widget to be included on the <ul> generated in the output -- right now the attrs are only being included on the individual <li>s. The simple fix proposed by #4080 seems correct to me, so that is what I used to test with for radio_admin_fields. If some other fix is made for #4117 then the radio_admin_fields code might need to change slightly. (The effect of running without any fix for this issue is that the RadioSelect? list is bulleted, which looks funny, and items are not spaced as nicely as they could be.)

(There is also the capability to sidestep the whole issue of #4080 by writing a custom renderer, but ugh, that would be 99% duplicated code with the default renderer. My impression was tht capability was added for cases where the desired output was something other than an unordered list, which is not the case for radio_admin_fields, so I did not pursue that path.)

4080.diff no longer applied cleanly, rebased on r6783

This looks like some functionality that breaks current admin. Tagging with nfa-blocker.

The patch here looks good, but I would recommend two things so this can be pushed to Ready for Checkin:

1. Don't remove the legacy radio_admin code. I rather see patches that don't change too much in the core until we know exactly how that will handled. Get in touch with Joseph about that.
2. Make a patch that includes all the changes that need to be made up to the current revision. It looks like there are two patches that are patches on top of the original.

I'd say go ahead and yank the old code if it isn't being used. We have trunk and version control if we need to look at the old stuff.

unified patch against current newforms-admin

radio-admin-fields.diff applied "fuzzily" against current newforms-admin. The unified patch includes both radio-admin-fields.diff and 4080-rebase6783.diff

(In [7626]) newforms-admin: Fixed #5731 -- Implemented ModelAdmin?.radio_fields to match trunk's radio_admin. Removed legacy code and added tests. Thanks Karen Tracey for the initial work.