django.core.context_processors.PermWrapper should be moved to django.contrib.auth.context_processors

[shailesh]

django.core.context_processors.PermWrapper specifically is meant for Permissions framework and doesn't really belong to django.core.

It would be a good idea to shift it to django.contrib.auth.context_processors.py

[Russell Keith-Magee]

This ticket requires a couple of things:

• Adding a deprecation warning to the existing instances
• Copying the code to the contrib.auth context processor module
• Updating the references in the docs (two references at present, in ref/templates/api and topics/auth)

It would be good to get this in for 1.3 -- we're already going to have to keep the alias around for a release longer than the auth processor that originally used it.

[v1v3kn]

Changed to std DeprecationWarning, updated deprecation policy

[Jannis Leidel]

Hm, shouldn't this use a PendingDeprecationWarning in 1.3 first?

[Russell Keith-Magee]

In [15635]:

Fixed #15299 -- Started the process of migrating the auth context processor support classes into the auth context processor module. Thanks to shailesh for the report, and v1v3kn for the draft patch.

[Russell Keith-Magee]

@jezdez - Yes, it should; I've made a couple of tweaks to the patch as submitted, including using a PendingDeprecationWarning, adding a release note, and correcting the location of the deprecation note.

[Jacob]

Milestone 1.3 deleted