Opened 19 years ago

Closed 19 years ago

Last modified 18 years ago

#949 closed defect (wontfix)

[patch] include template tag should support select_template type list

Reported by: amitu@… Owned by: Adrian Holovaty
Component: Template system Version: dev
Severity: normal Keywords:
Cc: upadhyay@… Triage Stage: Unreviewed
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

<% include template_name %> includes the named template which is like get_template of the Python API for loading templates, for consistancy we should have a comma seperated template list similer to select_template. (Untested) Patch attached.

Attachments (1)

django.patch (1.3 KB ) - added by upadhyay@… 19 years ago.
takes a comma seperated list as include argument

Download all attachments as: .zip

Change History (5)

by upadhyay@…, 19 years ago

Attachment: django.patch added

takes a comma seperated list as include argument

comment:1 by rjwittams, 19 years ago

I don't think commas are the right thing here - spaces would be better.

comment:2 by Amit Upadhyay <upadhyay@…>, 19 years ago

Cc: upadhyay@… added
Component: Admin interfaceTemplate system
Summary: include template tag should support select_template type list[patch] include template tag should support select_template type list
Version: SVN

Now that I think about comma vs spaces, the best option may be pipe character "|" as it already implies logic "or" which conveys what is happening.

comment:3 by rjwittams, 19 years ago

The point is that we shouldn't be proliferating random syntax within template tags.
The current method of parsing within a tag does little to discourage this, but people should explicitly *not* have to go and look up what random syntax someone made up for a particular tag.

comment:4 by Jacob, 19 years ago

Resolution: wontfix
Status: newclosed

This is out of the scope of the built-in {% include %}. Use a simple custom tag if you need this behavior.

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