Opened 16 years ago

Closed 16 years ago

Last modified 12 years ago

#6491 closed (invalid)

Improvements to User and Permission models

Reported by: Bastian Kleineidam <calvin@…> Owned by: nobody
Component: Uncategorized Version: dev
Severity: Keywords:
Cc: Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

The Permission model increases the maximal permission name length
from 50 to 100 (I actually hit this limit with some model permission
names some of my projects).
The User model will be sorted by last name, first name and then
username. And the string representation of a user will include the
last and first name if defined.

Attachments (1)

0019-Improvements-to-User-and-Permission-models.patch (1.9 KB ) - added by Bastian Kleineidam <calvin@…> 16 years ago.

Download all attachments as: .zip

Change History (5)

by Bastian Kleineidam <calvin@…>, 16 years ago

comment:1 by Malcolm Tredinnick, 16 years ago

Triage Stage: UnreviewedAccepted

These are two unrelated changes, so should really have been two tickets.

The Permissions one would be nice to have, but is backwards incompatible, so we need to judge the right moment to include it. Shouldn't be too harmful, though, and can be included in the list of database changes required to go from 0.96 to 1.0.

The User changes are a little less clear. The extra ordering is very much personal preference and adds a small performance hit. I don't see a huge need for that. Similarly, the string format change is a bit "take it or leave it" -- again, some people want one thing, others are going to want something else. Probably not worth changing just so that things continue to display as they always have. There are very few cases where you're going to be forced to use Django's version of the string format and cannot replace it with your own if you want (admin might be one place, but we have to make a trade-off in those cases).

So, I'm +1 on the Permissions changes and -1 or -0 on the User changes at the moment, at a first impression.

comment:2 by anonymous, 16 years ago

milestone: 1.0

comment:3 by Julia, 16 years ago

Resolution: invalid
Status: newclosed

Splitting into one separate ticket. ref: #8162

comment:4 by Jacob, 12 years ago

milestone: 1.0

Milestone 1.0 deleted

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