Opened 15 years ago

Closed 15 years ago

#5504 closed (fixed)

Backwards-incompatible change introduced by [6289]

Reported by: John Shaffer Owned by: Malcolm Tredinnick
Component: Template system Version: dev
Severity: Keywords: sprintsept14
Cc: Triage Stage: Accepted
Has patch: no Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no


Before [6289], {% load %} was identical to {% load google %} (because everything before the last segment was ignored). With [6289] or later, {% load %} raises a TemplateSyntaxError.

The change in behavior should be reverted or documented. I personally prefer the new behavior (although the old was nice to give an indication of where the tags really came from, this change is overall for the better).

References Satchmo #155.

Change History (4)

comment:1 Changed 15 years ago by John Shaffer

comment:2 Changed 15 years ago by Simon G. <dev@…>

Triage Stage: UnreviewedAccepted

comment:3 Changed 15 years ago by Malcolm Tredinnick

Owner: changed from nobody to Malcolm Tredinnick

Should be documented as backwards-incompatible. I hadn't thought that anybody would be relying on the old behaviour for useful purposes.

I'll fix this on the wiki page.

comment:4 Changed 15 years ago by Malcolm Tredinnick

Resolution: fixed
Status: newclosed
Note: See TracTickets for help on using tickets.
Back to Top