import django.contrib.User as something other than User in users/models.py

RESOLVED FIXED in 5.6

Status

RESOLVED FIXED
9 years ago
3 years ago

People

(Reporter: jbalogh, Assigned: wenzel)

Tracking

Details

(Whiteboard: [z][goodfirstbugifyou'reGerman])

(Reporter)

Description

9 years ago
Otherwise, people are going to try to link to auth.User instead of UserProfile, which is our table.
(Assignee)

Comment 1

9 years ago
how about "DjangoUser"
(Reporter)

Comment 2

9 years ago
(In reply to comment #1)
> how about "DjangoUser"

Sounds good to me!  I hope the right things break when you take away the User import.
Assignee: nobody → fwenzel
Whiteboard: [z] → [z][goodfirstbugifyou'reGerman]
(Assignee)

Comment 3

9 years ago
Can I use an Umlaut? ;)
(Assignee)

Comment 4

9 years ago
(In reply to comment #2)
> I hope the right things break when you take away the User import.

Yup! Promptly, the Version model broke.

http://github.com/jbalogh/zamboni/commit/6d04900f79f72aa2120d0cffd4fa39bbe3150b88
Status: NEW → RESOLVED
Last Resolved: 9 years ago
Resolution: --- → FIXED
(Reporter)

Comment 5

9 years ago
It doesn't count as fixed if you broke tests: http://sm-hudson01.mozilla.org:8080/job/addons.mozilla.org/91/testReport/?
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
(Assignee)

Comment 6

9 years ago
Oh, you are so right, thanks. Guess I missed this amidst Hudson and Firebot's little discussion (bug 540436).

Fixed:
http://github.com/jbalogh/zamboni/commit/ce2fa7b9b713d92d65610450b3be1925295f1a73

For reals:
http://sm-hudson01.mozilla.org:8080/job/addons.mozilla.org/92/
Status: REOPENED → RESOLVED
Last Resolved: 9 years ago9 years ago
Resolution: --- → FIXED
Product: addons.mozilla.org → addons.mozilla.org Graveyard
You need to log in before you can comment on or make changes to this bug.