The extra_context keyword argument should be available in the auth app views

[peadalo@…]

Enhancement Request

When using the views in the auth app (django.contrib.auth.views), it would be nice to have the option of passing extra context variables through the views and to the templates. This seems to be possible with django generic views by passing a dictionary to the "extra_context" keyword argument.

This would be useful for passing context variables to the login/logout templates which are specific to only those templates and therefore don't belong in the RequestContext context processors.

[Simon G. <dev@…>]

Want to write a patch?

[Russell Keith-Magee]

Seems like a reasonable idea.

[jordanl]

Adds support for passing extra_context to the views in django.contrib.auth.views

[jordanl]

The patch is pretty trivial. Not sure if this even needs tests.

[durdinator]

patch_5298.diff has the same semantics for extra_context as the generic views: i.e. it calls callables in the dict.

patch_5298.2.diff adds a paragraph to the docs about it.

[Russell Keith-Magee]

Mostly fine - but why the 'update_context_from_dict' method? Why not just use Context.update()?

[durdinator]

I was following the example of the extra_context argument to generic views, whereby callables are called -- for comparison see ​http://code.djangoproject.com/browser/django/trunk/django/views/generic/list_detail.py#L88

[Jj <jjdelc@…>]

extra_context added to views

[Jj <jjdelc@…>]

I just added the django.contrib.auth.views.patch patch that adds extra_context support to the auth views, It has minimal impact in the overall code and should not affect current code.

[CrashRoX]

This is still not as of 1.1...
(r'^{login/$', 'django.contrib.auth.views.login', {'template_name': 'login.html', 'extra_context': {'test': 'test'}})}

[Luke Plant]

The patch should use None, not {} as default value, due to potential 'mutable defaults' issue.

[efloehr]

This patch no longer works with trunk, due to changes in auth. I've attached an updated patch, current as of [12938]. I've also changed the patch to default extra_context to None instead of the original {}, as suggested by lukeplant in previous comment. I've also changed version to SVN as this patch isn't in 1.1.

[efloehr]

Patch for extra_context, current as of changeset [13042]

[efloehr]

Replying to efloehr:
Patch current as of [13042].

[Adam Vandenberg]

Patch updated to r14586

[Jannis Leidel]

(In [14768]) Fixed #5298 -- Added extra_context to contrib auth views.

[Tim Graham <timograham@…>]

In 61ecb5f48a4732f1471f858c64904ec1c4763925:

Fixed #20855 -- Added documentation of current_app and extra_context params to django.contrib.auth views

refs #5298 and refs #8342

[Tim Graham <timograham@…>]

In 77293f9354774e2631087935f1550c674093c433:

[1.6.x] Fixed #20855 -- Added documentation of current_app and extra_context params to django.contrib.auth views

refs #5298 and refs #8342

Backport of 61ecb5f48a from master

[Tim Graham <timograham@…>]

In 145e0a14f0c098d7bbf2bfb593f7b6cfe1b02d72:

[1.5.x] Fixed #20855 -- Added documentation of current_app and extra_context params to django.contrib.auth views

refs #5298 and refs #8342

Backport of 61ecb5f48a from master.