Opened 12 years ago

Closed 12 years ago

#18038 closed Cleanup/optimization (fixed)

Remove the 'supports_inactive_user' backwards-compatibility flag

Reported by: Aymeric Augustin Owned by: nobody
Component: contrib.auth Version: dev
Severity: Release blocker 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

As explained in the deprecation timeline.

Attachments (3)

18038-wip.patch (3.4 KB ) - added by Aymeric Augustin 12 years ago.
18038-2.diff (8.2 KB ) - added by Claude Paroz 12 years ago.
Tests included
18038-3.diff (8.2 KB ) - added by Claude Paroz 12 years ago.
Added one sentence in the docs

Download all attachments as: .zip

Change History (9)

by Aymeric Augustin, 12 years ago

Attachment: 18038-wip.patch added

comment:1 by Aymeric Augustin, 12 years ago

Has patch: set
Patch needs improvement: set

Several backends defined in d.c.auth's test define supports_inactive_user = True; this isn't fixed in the patch I attached. Also, the docs could be improved.

comment:2 by Aymeric Augustin, 12 years ago

Severity: NormalRelease blocker

by Claude Paroz, 12 years ago

Attachment: 18038-2.diff added

Tests included

comment:3 by Claude Paroz, 12 years ago

I didn't improve the docs, still.

by Claude Paroz, 12 years ago

Attachment: 18038-3.diff added

Added one sentence in the docs

comment:4 by Claude Paroz, 12 years ago

Patch needs improvement: unset

In the last patch, I only added:
"Do not forget to test for the is_active attribute of the user in your own
backend permission methods."

If Aymeric wanted to add more, I think we should then open a seperate ticket.

comment:5 by Ramiro Morales, 12 years ago

Triage Stage: AcceptedReady for checkin

comment:6 by Claude Paroz, 12 years ago

Resolution: fixed
Status: newclosed

In [17938]:

Fixed #18038 -- Removed the 'supports_inactive_user' backwards-compatibility flag. Thanks Aymeric Augustin for the initial patch and Ramiro Morales for the review.

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