[patch, new-admin] Allow html tags if "allow_tags" model's method attribute is present

[plisk]

Subj, to use like shown below. It allows to include html in object list. Patch against revision 1153, change is in function items_for_result.

class Policy(meta.Model):
        subject = meta.CharField(maxlength=100)
        date_added = meta.DateTimeField('Date Added', default=datetime.today())

        def link_view(self):
                return '<a href="' + str(self.id) + '/view/' + '">view</a>'
        link_view.short_description = 'Action'
        link_view.allow_tags = True

        class META:
                admin = meta.Admin(
                                        list_display = ('subject', 'date_added', 'link_view'),
                                )

[plisk]

Add support for allow_tags attribute

[rjwittams]

Hm, not sure about applying this in new-admin (changes admin behaviour). I can see it could be useful.
Adrian, Jacob, any thoughts?

[Adrian Holovaty]

(In [1174]) Fixed #766 -- Custom methods in admin.list_display can now have an allow_tags attribute, which doesn't strip tags in the methods' output. Thanks, plisk

[anonymous]

Sorry to bother, but after revising the code i guess its written with potential exception catch missing. Seems like "func = getattr(result, field_name)" should be in try block and exception is AttributeError and not ObjectDoesNotExist ? Maybe i'm not right, but just want to be sure it will be always working correctly. Below is patch for this.

[plisk]

correct exception handling ?

[rjwittams]

Applied exception fix to new-admin.

Should be applied in trunk if new-admin isn't merged soon....

[Adrian Holovaty]

Fixed in [1434].