Auth should use primary key rather than 'id' when referencing user

[wiz]

Got two problems on using profile system:

1) user.get_profile() fails with "AttributeError: NoneType has no _default_manager" when using shell in
/usr/local/src/django_trunk/django/contrib/auth:

self._profile_cache = model._default_manager.get(useridexact=self.id)

Error is gone after execution of models.get_models()

2) i have profile class with no 'id' field, but 'user_idnr' as primary key.
But useridexact is hardcoded in the same line above.

[wiz]

my workover for both issues

[Malcolm Tredinnick]

The second part of this patch is not correct. That queryset is referring to the User model, not the Profile model, so user__id__exact is correct (it is finding the profile that belongs to the given user id).

For the first part of the problem, I don't really understand what you are doing to see the problem. Can you paste a few commands I can run to replicate it, please? I want to make sure we are fixing the real problem, not just covering up the cause.

Removing the patch keyword for now, since it can't be applied as is.

[Malcolm Tredinnick]

It just occurred to me that the first part of this ticket is probably the problem from #2684.

[Chris Beaven]

I'll change the summary to reference the second part of this ticket since it does look like #2684 handles part 1.

I think that this would solve it, but haven't tested it:
self._profile_cache = model._default_manager.get(user=self)

[Malcolm Tredinnick]

I'm going to claim that this is a non-bug. The first paragraph of comment 3, above, is still correct.