Opened 6 years ago

Closed 4 years ago

#14451 closed New feature (wontfix)

add related_name to contrib.auth.models.User for 'groups'

Reported by: Tom von Schwerdtner Owned by: anonymous
Component: contrib.auth Version: 1.2
Severity: Normal Keywords:
Cc: Triage Stage: Design decision needed
Has patch: no Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

Would allow for

from django.contrib.auth.models import Group
group = Group.objects.all()[0]
group.members.all()

instead of the current

group.users_set.all()

You know, nexttime someone feels like kicking backwards compatibility in the groin...

Change History (7)

comment:1 Changed 6 years ago by Luke Plant

Needs documentation: unset
Needs tests: unset
Patch needs improvement: unset
Triage Stage: Unreviewed → Someday/Maybe

comment:2 Changed 6 years ago by anonymous

Owner: changed from nobody to anonymous
Status: new → assigned

comment:3 Changed 5 years ago by Julien Phalip

Severity: → Normal
Type: → New feature

comment:4 Changed 5 years ago by Thejaswi Puthraya

Component: Uncategorized → contrib.auth
Easy pickings: unset
UI/UX: unset

comment:5 Changed 5 years ago by Jacob

Milestone 2.0 deleted

comment:6 Changed 4 years ago by Aymeric Augustin

Triage Stage: Someday/Maybe → Design decision needed

This can be done with a deprecation path right now if we want to. During the transition, Group.users_set would be a property raising a deprecation warning and returning members.

However I don't know if there's much value in forcing this change upon all users of Django.

comment:7 Changed 4 years ago by Aymeric Augustin

Resolution: → wontfix
Status: assigned → closed

Actually, I cannot convinced myself there's enough value.

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