Opened 17 years ago

Closed 17 years ago

#5468 closed (fixed)

[patch] direct_to_template uses literal dict in keyword arg

Reported by: durdinator Owned by: nobody
Component: Template system Version: dev
Severity: Keywords:
Cc: Triage Stage: Ready for checkin
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

This is not a real problem, just bad practice, with a trivial fix.

def direct_to_template(request, template, extra_context={}, mimetype=None, **kwargs)

Attachments (1)

patch_5468.diff​ (861 bytes ) - added by durdinator 17 years ago.
Fix

Download all attachments as: .zip

Change History (4)

by durdinator, 17 years ago

Attachment: patch_5468.diff​ added

Fix

comment:1 by durdinator, 17 years ago

Has patch: set

comment:2 by flother, 17 years ago

Summary: direct_to_template uses literal dict in keyword arg → [patch] direct_to_template uses literal dict in keyword arg
Triage Stage: Unreviewed → Ready for checkin

All the other views in the django.views.generic package do this, so it seems fair to accept this patch.

comment:3 by Russell Keith-Magee, 17 years ago

Resolution: → fixed
Status: new → closed

(In [6278]) Fixed #5468 -- Fixed the handling of the context argument in direct_to_template generic views. Thanks, durdinator.

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