Hi Anssi,

Thanks for that quick fix.

On Wednesday 17 December 2014 10:16:27 Anssi Kääriäinen wrote:
> 
> It turns out this is easy to fix. There is a patch for this at
> https://github.com/django/django/pull/3743.
> 
(All: In case you're interested but lazy -- this was committed)

> The generic relations still work a bit differently to that of reverse
> foreign key - where reverse foreign keys can be filtered
> with .filter(reverse_relation=some_related_object), that doesn't work
> with reverse relations, instead you have to explicitly provide the
> primary key value by doing .filter(generic_rel=some_related_object.pk).

So, I'm not sure if we should document this behavior or try to fix it as well.

I opened #24019 for fixing the outdated "GenericRelation+aggregation don't 
work" documentation, and added that as a point for consideration as well.

Thanks again,
        Shai.

Reply via email to