Skip to content

removed AUTH_USER_MODEL compat property #4176

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
lovelydinosaur merged 4 commits into from
Jun 7, 2016
Merged

removed AUTH_USER_MODEL compat property #4176

Changes from 1 commit
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
7dca49c
removed AUTH_USER_MODEL compat property
auvipy Jun 7, 2016
45e38dd
imported and updated to contrib auth User model
auvipy Jun 7, 2016
2c0bcc6
reverted settings removal
auvipy Jun 7, 2016
f82cf3e
used overridable AUTH_USER_MODEL
auvipy Jun 7, 2016
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
imported and updated to contrib auth User model
  • Loading branch information
@auvipy
auvipy committed Jun 7, 2016
commit 45e38ddacc5f8d96609ff58cfc32f3c50d560906
4 changes: 2 additions & 2 deletions rest_framework/authtoken/models.py
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
import binascii
import os

from django.conf import AUTH_USER_MODEL
from django.contrib.auth.models import User
Copy link
Contributor

@xordoquy xordoquy Jun 7, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Doesn't this remove the ability to use a custom user ?

Copy link
Collaborator Author

@auvipy auvipy Jun 7, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

then should stick with AUTH_USER_MODEL instead?

Copy link
Contributor

@xordoquy xordoquy Jun 7, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes.

from django.db import models
from django.utils.encoding import python_2_unicode_compatible
from django.utils.translation import ugettext_lazy as _
Expand All @@ -14,7 +14,7 @@ class Token(models.Model):
"""
key = models.CharField(_("Key"), max_length=40, primary_key=True)
user = models.OneToOneField(
AUTH_USER_MODEL, related_name='auth_token',
User, related_name='auth_token',
on_delete=models.CASCADE, verbose_name=_("User")
)
created = models.DateTimeField(_("Created"), auto_now_add=True)
Expand Down