Opened 7 years ago

Closed 6 years ago

Last modified 4 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: UI/UX:

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@… 7 years ago.
Tar file of complete Django site to demonstrate the problem.
9992.patch (1.2 KB) - added by peterbe 6 years ago.
made from r10257

Download all attachments as: .zip

Change History (9)

Changed 7 years ago by h.fangohr@…

Tar file of complete Django site to demonstrate the problem.

comment:1 Changed 7 years ago by ramiro

  • Description modified (diff)
  • Needs documentation unset
  • Needs tests unset
  • Patch needs improvement unset

(edited description)

comment:2 Changed 7 years ago by jacob

  • Description modified (diff)
  • milestone set to 1.1
  • Triage Stage changed from Unreviewed to Accepted

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

comment:3 Changed 6 years ago by peterbe

  • Resolution set to fixed
  • Status changed from new to closed

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 6 years ago by peterbe

made from r10257

comment:4 Changed 6 years ago by kmtracey

  • Resolution fixed deleted
  • Status changed from closed to reopened

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

comment:5 Changed 6 years ago by jacob

  • Owner changed from nobody to jacob
  • Status changed from reopened to assigned

comment:6 Changed 6 years ago by jacob

  • Resolution set to fixed
  • Status changed from assigned to closed

(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 4 years ago by jacob

  • milestone 1.1 deleted

Milestone 1.1 deleted

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