Opened 9 years ago

Closed 5 years ago

Last modified 5 years ago

#6766 closed New feature (needsinfo)

TEMPLATE_STRING_IF_INVALID should work for loops

Reported by: Thomas Güttler Owned by: nobody
Component: Template system Version: master
Severity: Normal Keywords:
Cc: Triage Stage: Design decision needed
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

This small patch enables to render TEMPLATE_STRING_IF_INVALID in loops.

Attachments (2)

template_invalid__loop.diff (2.2 KB) - added by Thomas Güttler 9 years ago.
template_if_invalid.diff (7.0 KB) - added by Thomas Güttler 9 years ago.
Previous patch included the function twice.

Download all attachments as: .zip

Change History (8)

Changed 9 years ago by Thomas Güttler

Attachment: template_invalid__loop.diff added

Changed 9 years ago by Thomas Güttler

Attachment: template_if_invalid.diff added

Previous patch included the function twice.

comment:1 Changed 9 years ago by Thomas Güttler

Needs documentation: unset
Needs tests: unset
Patch needs improvement: unset

comment:2 Changed 8 years ago by Jeff Anderson

Triage Stage: UnreviewedAccepted

comment:3 Changed 7 years ago by Chris Beaven

Triage Stage: AcceptedDesign decision needed

comment:4 Changed 6 years ago by Julien Phalip

Severity: Normal
Type: New feature

comment:5 Changed 5 years ago by Carl Meyer

Easy pickings: unset
Resolution: needsinfo
Status: newclosed
UI/UX: unset

This report doesn't clearly state what the existing behavior is, why it's problematic, and what the desired alternative behavior is. The "small patch" changes quite a lot of existing tests and doesn't clarify matters. Closing "needsinfo".

comment:6 Changed 5 years ago by Thomas Güttler

Cc: hv@… removed
Note: See TracTickets for help on using tickets.
Back to Top