Opened 12 years ago

Closed 12 years ago

#18780 closed Cleanup/optimization (duplicate)

Set stacklevel when warning about depreciated function views

Reported by: paulb@… Owned by: nickmartini
Component: Generic views Version: 1.4
Severity: Normal Keywords:
Cc: Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: yes UI/UX: no

Description

It would be handy if the depreciation warning messages gave a clue as to the cause of warning. This can be done by adding severity to the warning.

Eg

#django.views.generic.simple.py

import warnings
warnings.warn(
    'Function-based generic views have been deprecated; use class-based views instead.',
    DeprecationWarning, stacklevel=2
)

Change History (4)

comment:1 by Aymeric Augustin, 12 years ago

Triage Stage: UnreviewedAccepted
Type: UncategorizedCleanup/optimization

Actually, it may be a good idea to always set stacklevel appropriately (is 2 always the "right" value?) wherever Django raises a DeprecationWarning.

comment:2 by nickmartini, 12 years ago

Owner: changed from nobody to nickmartini

comment:3 by nickmartini, 12 years ago

Has patch: set

All DeprecationWarning are now stacklevel=2. I made no changes to PendingDeprecationWarning.

https://github.com/django/django/pull/349

Last edited 12 years ago by nickmartini (previous) (diff)

comment:4 by Aymeric Augustin, 12 years ago

Resolution: duplicate
Status: newclosed

This is actually a duplicate of #18127, the same pull request was posted to both tickets.

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