Opened 16 years ago

Closed 15 years ago

Last modified 12 years ago

#5270 closed (fixed)

Template tags won't accept an empty string as an argument

Reported by: stephen Owned by: Jeremy Dunck
Component: Template system Version: dev
Severity: Keywords: empty string tplrf-fixed
Cc: jdunck@… Triage Stage: Ready for checkin
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

There currently appears to be no easy to pass an empty string to a template tag.

{% url admin "" %} results in a TemplateSyntaxError.

Some template tags (notable reverse URL lookups for the root of a newforms-admin instance) require an empty string to be passed in.

Attachments (2)

empty-string.patch​ (782 bytes) - added by rnairn@… 15 years ago.
Patch to allow empty constant strings as filter arguments
5270-empty-string-template-w-tests.diff​ (2.9 KB) - added by Jeremy Dunck 15 years ago.
Added tests, and now handling the empty i18n constant specially for translation.

Download all attachments as: .zip

Change History (12)

comment:1 Changed 16 years ago by Simon G. <dev@…>

Triage Stage: Unreviewed → Design decision needed

comment:2 Changed 16 years ago by miracle2k

This is due to 

if i18n_constant:
    var = '"%s"' %  _(i18n_constant.replace(r'\"', '"'))
elif constant:
    var = '"%s"' % constant.replace(r'\"', '"')

in FilterExpression.init(), where the elif-clause should probably check for is None.

Would be nice if this could be fixed.

Changed 15 years ago by rnairn@…

Attachment: empty-string.patch​ added

Patch to allow empty constant strings as filter arguments

comment:3 Changed 15 years ago by Rowan Nairn

Has patch: set
Owner: changed from nobody to Rowan Nairn
Status: new → assigned

Added a patch

comment:4 Changed 15 years ago by Johannes Dollinger

Keywords: tplrf-fixed added

This would be fixed by the refactoring proposed in #7806.

comment:5 Changed 15 years ago by Jacob

milestone: → 1.0

Changed 15 years ago by Jeremy Dunck

Attachment: 5270-empty-string-template-w-tests.diff​ added

Added tests, and now handling the empty i18n constant specially for translation.

comment:6 Changed 15 years ago by Jeremy Dunck

Cc: jdunck@… added
Triage Stage: Design decision needed → Ready for checkin

comment:7 Changed 15 years ago by Jeremy Dunck

Owner: changed from Rowan Nairn to Jeremy Dunck
Status: assigned → new

comment:8 Changed 15 years ago by Jeremy Dunck

Status: new → assigned

comment:9 Changed 15 years ago by Gary Wilson

Resolution: → fixed
Status: assigned → closed

(In [8393]) Fixed #5270 -- Allow template tags and filters to accept an emtpy string, patch from jdunck.

comment:10 Changed 12 years ago by Jacob

milestone: 1.0

Milestone 1.0 deleted

Note: See TracTickets for help on using tickets.
Back to Top