New template filter suggestion (wrap)

[david.arakelian@…]

We came across a use case today for wrapping variables in styles If they are not None/Empty/False...

E.g. If a variable exists add a break tag after it or if a value exists then wrap it in a <p></p> tag.
So we wrote a simple template tag that might be of interest to the community.

Template tag:

from django import template
from django.template.defaultfilters import stringfilter
from django.utils.safestring import mark_safe

register = template.Library()

@register.filter
@stringfilter
def wrap(value, arg):
    if value and value != '':
        return mark_safe(arg % value) # decision needed with |safe
    return value

Usage:

{{ var|wrap:"%s<br />" }} or
{{ var|wrap:"<p>%s</p>" }}

[Russell Keith-Magee]

I'm afraid I don't see the benefit here. What you describe is already possible with:

{% if var %}<p>{{ var }}</p>{% endif %}

I'm not convinced that a template filter is an inhernetly cleaner way of representing this.