Opened 9 years ago

Closed 9 years ago

Last modified 8 years ago

#949 closed defect (wontfix)

[patch] include template tag should support select_template type list

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

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@… 9 years ago.
takes a comma seperated list as include argument

Download all attachments as: .zip

Change History (5)

Changed 9 years ago by upadhyay@…

takes a comma seperated list as include argument

comment:1 Changed 9 years ago by rjwittams

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

comment:2 Changed 9 years ago by Amit Upadhyay <upadhyay@…>

  • Cc upadhyay@… added
  • Component changed from Admin interface to Template system
  • Summary changed from include template tag should support select_template type list to [patch] include template tag should support select_template type list
  • Version set to 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 Changed 9 years ago by rjwittams

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 Changed 9 years ago by jacob

  • Resolution set to wontfix
  • Status changed from new to closed

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