Opened 17 years ago

Closed 17 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

Description

Before [6289], {% load satchmo.google %} was identical to {% load google %} (because everything before the last segment was ignored). With [6289] or later, {% load satchmo.google %} 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 by John Shaffer, 17 years ago

comment:2 by Simon G. <dev@…>, 17 years ago

Triage Stage: UnreviewedAccepted

comment:3 by Malcolm Tredinnick, 17 years ago

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 by Malcolm Tredinnick, 17 years ago

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