Opened 7 years ago

Closed 4 years ago

Last modified 4 years ago

#6766 closed New feature (needsinfo)

TEMPLATE_STRING_IF_INVALID should work for loops

Reported by: guettli 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 guettli 7 years ago.
template_if_invalid.diff (7.0 KB) - added by guettli 7 years ago.
Previous patch included the function twice.

Download all attachments as: .zip

Change History (8)

Changed 7 years ago by guettli

Changed 7 years ago by guettli

Previous patch included the function twice.

comment:1 Changed 7 years ago by guettli

  • Needs documentation unset
  • Needs tests unset
  • Patch needs improvement unset

comment:2 Changed 7 years ago by programmerq

  • Triage Stage changed from Unreviewed to Accepted

comment:3 Changed 6 years ago by SmileyChris

  • Triage Stage changed from Accepted to Design decision needed

comment:4 Changed 4 years ago by julien

  • Severity set to Normal
  • Type set to New feature

comment:5 Changed 4 years ago by carljm

  • Easy pickings unset
  • Resolution set to needsinfo
  • Status changed from new to closed
  • 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 4 years ago by guettli

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