Model proxy instance does not equal the respective model instance

[Byron Ruth]

Description here: ​http://groups.google.com/group/django-developers/browse_thread/thread/1c295b8eaf0cb73c

[Alex Gaynor]

I need to think about the semantics of this a bit more.

[Simon Meers]

Yes, it is difficult to say whether they necessarily *should* be equal. They are two distinct objects with different properties, and it is possible to compare them explicitly if necessary. It depends on what you mean by *equal*...

[Byron Ruth]

It might help to explain how I noticed this in the first place. I wrote an auth backend that sets request.user to a proxy instance of User. When I went to compare entry.author (for example) to request.user it failed. I agree that they certainly two different objects, but in this context i think it is appropriate to have them equal each other. The reality is that even though having an explicit method to do this comparison would be semantically correct, the reason for comparison is with respect to the underlying data the instance represents.

Alex, I like your idea except that for any models (like User) that are from third party apps, I don't want to have to implement the method on them.

[Russell Keith-Magee]

As DrMeers suggests, the real issue here is "What is a Proxy", and "what is equality". Is a Proxy the same object, or is it a different object? #11154 addresses a similar conceptual issue, as related to permissions.

Although the decision on #11154 was logged by Malcolm, I was with him when he made it, and we discussed it at the time, and I agree with the reasoning. The same reasoning applies here, IMHO.

That said, I'm interested in hearing community debate on this issue (on django-dev, please, not on the ticket). I'm willing to be convinced that I'm wrong or in the minority opinion.

[Byron Ruth]

I continued it back on the thread I started before ​http://groups.google.com/group/django-developers/browse_thread/thread/1c295b8eaf0cb73c

[Byron Ruth]

Latest patch attached, includes Alex's tests. Checks to ensure the object being compared against is an instance of Model.

[Byron Ruth]

Since this is a relatively minor patch, I would like to see if it can get into 1.3. There has not been anymore discussion as to why this shouldn't be added. I think per what Russell and Malcolm has stated this is a valid implementation.

[Russell Keith-Magee]

#15054 is another presentation of the same underlying issue (what is a proxy).

[Luke Plant]

If I read those other discussions correctly, there is no consensus that this is the correct design decision, so it cannot be 'Ready for checkin'.

The closest we have to consensus is on #15054, which has a valid use case for proxy models not being treated as equal (with a loose definition of 'equal') to the parent model, which goes the other way from this. So I'm knocking it back to 'design decision needed'.

[Jacob]

Milestone 1.3 deleted

[Alex Gaynor]

I'm not sure this solution is correct, but the current situation definitely is not, marking this as accepted.

[Ryan Kaskel]

Since this ticket has been accepted again, I've added a new patch: ​https://github.com/django/django/pull/1480/files

It would be great to get some feedback on the approach and how to handle backwards compatibility.

[Tim Graham]

Actually, there's already another ​pull request open which addresses this ticket.

[Byron Ruth]

Great to see this finally ready to go in.

[Anssi Kääriäinen]

Fixed in 4668c142dce77c6f29fb75532c1acfa1b2d322ff