Opened 15 years ago

Closed 13 years ago

Last modified 12 years ago

#9666 closed (fixed)

ssi tag don't works with context variables

Reported by: Andrews Medina Owned by: Eric Holscher
Component: Template system Version: dev
Severity: Keywords: tag, ssi, template
Cc: Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: yes Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

if I want use this:

{% ssi url %}


and url to be a context variable, this don't works. The tag {% ssi %} works only with strings.

Attachments (5)

patche.txt (527 bytes ) - added by Andrews Medina 15 years ago.
ssi-variable.diff (1.2 KB ) - added by Ulrich Petri 15 years ago.
fixed ssi variable resolving
9666-with-tests.2.diff (5.3 KB ) - added by Eric Holscher 15 years ago.
'"' +This makes me cry a little inside + '"'
9666-with-tests.diff (5.6 KB ) - added by Eric Holscher 15 years ago.
Actually attach template this time.
future_ssi-1.diff (4.2 KB ) - added by Sean Brant 13 years ago.
future ssi tag replacement

Download all attachments as: .zip

Change History (16)

by Andrews Medina, 15 years ago

Attachment: patche.txt added

comment:1 by Andrews Medina, 15 years ago

Version: 1.0SVN

comment:2 by Jacob, 15 years ago

milestone: 1.1
Triage Stage: UnreviewedAccepted

comment:3 by Ulrich Petri, 15 years ago

The above patch has a bug if the ssi tag is used with a variable inside a loop.
Since the variable is only "resolved" once subsequent .render() calls (to the same node) always return the result of the first invocation.

I'll attach a fixed patch.

by Ulrich Petri, 15 years ago

Attachment: ssi-variable.diff added

fixed ssi variable resolving

comment:4 by mcroydon, 15 years ago

Needs tests: set

comment:5 by Eric Holscher, 15 years ago

Owner: changed from nobody to Eric Holscher
Status: newassigned

Note that currently I don't see any tests associated with ssi. I'll work on this later, if nobody else wants to.

by Eric Holscher, 15 years ago

Attachment: 9666-with-tests.2.diff added

'"' +This makes me cry a little inside + '"'

by Eric Holscher, 15 years ago

Attachment: 9666-with-tests.diff added

Actually attach template this time.

comment:6 by Russell Keith-Magee, 15 years ago

milestone: 1.1

This borders on being a feature; deferring from v1.1

comment:7 by Eric Holscher, 13 years ago

milestone: 1.3

by Sean Brant, 13 years ago

Attachment: future_ssi-1.diff added

future ssi tag replacement

comment:8 by Sean Brant, 13 years ago

#14389 is trying to introduce a from future concept for the url tag fixes. My latest patch does the same thing for the ssi tag. See #14389 for more details.

comment:9 by Russell Keith-Magee, 13 years ago

The RC1 patch on #14389 includes a fix for this ticket.

comment:10 by Russell Keith-Magee, 13 years ago

Resolution: fixed
Status: assignedclosed

(In [14643]) Fixed #14389, #9666 -- Started the migration path to make the first argument to url and ssi template tags syntactically consistent with other tags. Thanks to Sean Brant for the draft patch.

comment:11 by Jacob, 12 years ago

milestone: 1.3

Milestone 1.3 deleted

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