Commit 36ce5d0b authored by Jannis Leidel's avatar Jannis Leidel
Browse files

[1.2.X] Fixed #14363 -- Fixed render method of ManyToManyRawIdWidget to set... 

[1.2.X] Fixed #14363 -- Fixed render method of ManyToManyRawIdWidget to set attrs to an empty dict. Thanks, tyrion and dpn.

Backport from trunk (r13971).

git-svn-id: http://code.djangoproject.com/svn/django/branches/releases/1.2.X@13975 bcc190cf-cafb-0310-a4f2-bffc1f526a37
parent 6d30b284

django/contrib/admin/widgets.py

+2 −0
Original line number Diff line number Diff line
@@ -164,6 +164,8 @@ class ManyToManyRawIdWidget(ForeignKeyRawIdWidget): 
    in a <select multiple> box.

    """

    def render(self, name, value, attrs=None):

        if attrs is None:

            attrs = {}

        attrs['class'] = 'vManyToManyRawIdAdminField'

        if value:

            value = ','.join([force_unicode(v) for v in value])

tests/regressiontests/admin_widgets/models.py

+2 −0
Original line number Diff line number Diff line
@@ -133,6 +133,8 @@ Currently: <a target="_blank" href="%(STORAGE_URL)salbums/hybrid_theory.jpg">alb 
>>> w = ManyToManyRawIdWidget(rel)

>>> print conditional_escape(w.render('test', [m1.pk, m2.pk], attrs={}))

<input type="text" name="test" value="1,2" class="vManyToManyRawIdAdminField" /><a href="../../../admin_widgets/member/" class="related-lookup" id="lookup_id_test" onclick="return showRelatedObjectLookupPopup(this);"> <img src="%(ADMIN_MEDIA_PREFIX)simg/admin/selector-search.gif" width="16" height="16" alt="Lookup" /></a>

>>> print conditional_escape(w.render('test', [m1.pk]))

<input type="text" name="test" value="1" class="vManyToManyRawIdAdminField" /><a href="../../../admin_widgets/member/" class="related-lookup" id="lookup_id_test" onclick="return showRelatedObjectLookupPopup(this);"> <img src="%(ADMIN_MEDIA_PREFIX)simg/admin/selector-search.gif" width="16" height="16" alt="Lookup" /></a>

>>> w._has_changed(None, None)

False

>>> w._has_changed([], None)