Opened 11 years ago

Closed 11 years ago

Last modified 11 years ago

#19714 closed Bug (fixed)

Documentation for TemplateView incorrectly refers to "params" context variable.

Reported by: Aram Dulyan Owned by: Claude Paroz
Component: Documentation Version: 1.5-beta-1
Severity: Release blocker Keywords: TemplateView params
Cc: Triage Stage: Accepted
Has patch: no Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: yes UI/UX: no

Description

The documentation for TemplateView at docs/ref/class-based-views/base.txt states that TemplateView provides a "params" context variable, which is a dictionary of the URL matched groups. However, since commit f04bb6d, TemplateView no longer creates a "params" variable, instead passing the matched groups directly, e.g. for the URL pattern r'/(?<slug>[^\]+/', the template receives {{ email }} instead of {{ params.email }}, as suggested by the documentation.

Change History (5)

comment:1 by Aram Dulyan, 11 years ago

Severity: NormalRelease blocker

comment:2 by Claude Paroz, 11 years ago

Owner: changed from nobody to Claude Paroz
Status: newassigned
Triage Stage: UnreviewedAccepted

comment:3 by Claude Paroz <claude@…>, 11 years ago

Resolution: fixed
Status: assignedclosed

In 56e553129f554f83c8e99ef3368544921dbd8a82:

Fixed #19714 -- Updated documentation about TemplateView context

Thanks Aramgutang for the report. Refs #17228.

comment:4 by Claude Paroz <claude@…>, 11 years ago

In 5ebd0452ba6ff93980e6f9a8958f6ea01c835ffe:

[1.5.x] Fixed #19714 -- Updated documentation about TemplateView context

Thanks Aramgutang for the report. Refs #17228.
Backport of 56e553129 from master.

comment:5 by Claude Paroz <claude@…>, 11 years ago

In 5ebd0452ba6ff93980e6f9a8958f6ea01c835ffe:

[1.5.x] Fixed #19714 -- Updated documentation about TemplateView context

Thanks Aramgutang for the report. Refs #17228.
Backport of 56e553129 from master.

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