Opened 10 years ago

Closed 10 years ago

Last modified 7 years ago

#9992 closed (fixed)

get_profile / get_model issue, capitalisation of AUTH_PROFILE_MODULE

Reported by: anonymous Owned by: Jacob
Component: Documentation Version: 1.0
Severity: Keywords:
Cc: Triage Stage: Accepted
Has patch: no Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description (last modified by Jacob)

The docs lead one to expect that given an app name Profile, a AUTH_PROFILE_MODULE setting of profile.modelname would work, but Profile.modelname is actually correct.

http://docs.djangoproject.com/en/dev/topics/auth/#auth-profiles claims that AUTH_PROFILE_MODULE is "The (normalized to lower-case) name of the application in which the user profile model is defined (in other words, an all-lowercase version of the name which was passed to manage.py startapp to create the application)."

This is wrong; the app_label part is the a python module, and hence case-sensitive.

Attachments (2)

debug.tar.gz (6.6 KB) - added by h.fangohr@… 10 years ago.
Tar file of complete Django site to demonstrate the problem.
9992.patch (1.2 KB) - added by Peter Bengtsson 10 years ago.
made from r10257

Download all attachments as: .zip

Change History (9)

Changed 10 years ago by h.fangohr@…

Attachment: debug.tar.gz added

Tar file of complete Django site to demonstrate the problem.

comment:1 Changed 10 years ago by Ramiro Morales

Description: modified (diff)

(edited description)

comment:2 Changed 10 years ago by Jacob

Description: modified (diff)
milestone: 1.1
Triage Stage: UnreviewedAccepted

[trimmed excessive explanation; this is actually a very simple doc fix]

comment:3 Changed 10 years ago by Peter Bengtsson

Resolution: fixed
Status: newclosed

Check how it works. Conclusion:

  1. the name of the application is case sensitive
  1. the name of the model is case insensitive

I've attached a patch to auth.txt and also concluded that the text in chapter 12 of the book is fine and doesn't need to be updated.

Changed 10 years ago by Peter Bengtsson

Attachment: 9992.patch added

made from r10257

comment:4 Changed 10 years ago by Karen Tracey

Resolution: fixed
Status: closedreopened

It's not fixed until the patch is committed to the code base.

comment:5 Changed 10 years ago by Jacob

Owner: changed from nobody to Jacob
Status: reopenedassigned

comment:6 Changed 10 years ago by Jacob

Resolution: fixed
Status: assignedclosed

(In [10371]) Fixed a whole bunch of small docs typos, errors, and ommissions.

Fixes #8358, #8396, #8724, #9043, #9128, #9247, #9267, #9267, #9375, #9409, #9414, #9416, #9446, #9454, #9464, #9503, #9518, #9533, #9657, #9658, #9683, #9733, #9771, #9835, #9836, #9837, #9897, #9906, #9912, #9945, #9986, #9992, #10055, #10084, #10091, #10145, #10245, #10257, #10309, #10358, #10359, #10424, #10426, #10508, #10531, #10551, #10635, #10637, #10656, #10658, #10690, #10699, #19528.

Thanks to all the respective authors of those tickets.

comment:18 Changed 7 years ago by Jacob

milestone: 1.1

Milestone 1.1 deleted

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