maas-devel team mailing list archive

[Julian Edwards <julian.edwards@xxxxxxxxxxxxx>]

On 20/11/12 00:51, Raphaël Badin wrote:
> I think there are still some parts that are missing.
> 
> Running the test suite of the 1.2 branch on a precise instance (with the
> django package from the experimental ppa), I'm getting exceptions:
> AttributeError: 'ManyRelatedManager' object has no attribute
> 'query_field_name' (http://paste.ubuntu.com/1369927/).
> 
> These failures go away if I comment out the calls to prefetch_related in
> MAAS' code.
> 
> This is the change that your patch does to the definition of the class
> 'ManyRelatedManager':
> 
>     http://paste.ubuntu.com/1370197/
> 
> And in Django's sources, the commit
> 662eea116f5a188b6983f5f7c01c8247e3b6b309 (which fixes
> https://code.djangoproject.com/ticket/16937 and introduces
> prefetch_related) changes the same code this way:
> 
>     http://paste.ubuntu.com/1370200/
> 
> I think you've missed part of the diff (namely the introduction of the
> field 'query_field_name').

Indeed I had.  Try again :)