harvest-dev team mailing list archive

[Daniel Holbach <daniel.holbach@xxxxxxxxxx>]

> Opportunity.objects.only_valid().count()
> 
> is not the same as:
> 
> Opportunity.objects.filter(valid=True,
> last_updated=F('opportunitylist__last_updated')).count()
> 
> It adds an extra filter: opportunitylist__active=True


I know, but it's intentional. We use the same check in a couple of others places, but we missed to change it here as well.
-- 
https://code.launchpad.net/~nigelbabu/harvest/cache-add/+merge/71904
Your team harvest-dev is requested to review the proposed merge of lp:~nigelbabu/harvest/cache-add into lp:harvest.