You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository has been archived by the owner on Nov 27, 2022. It is now read-only.
It seems that this packages is pretty well adjusted for Django1.7, except for an issue with the query.promote_joins() method.
I'm not 100% sure, but it appears that in Django1.6, promote_joins allows for an unconditional kwarg to be passed:
def promote_joins(self, aliases, unconditional=False):
"""
Promotes recursively the join type of given aliases and its children to
an outer join. If 'unconditional' is False, the join is only promoted if
it is nullable or the parent join is an outer join.
Note about join promotion: When promoting any alias, we make sure all
joins which start from that alias are promoted, too. When adding a join
in join(), we make sure any join added to already existing LOUTER join
is generated as LOUTER. This ensures we don't ever have broken join
chains which contain first a LOUTER join, then an INNER JOIN, that is
this kind of join should never be generated: a LOUTER b INNER c. The
reason for avoiding this type of join chain is that the INNER after
the LOUTER will effectively remove any effect the LOUTER had.
"""
but in Django1.7, this kwarg is removed:
def promote_joins(self, aliases):
"""
Promotes recursively the join type of given aliases and its children to
an outer join. If 'unconditional' is False, the join is only promoted if
it is nullable or the parent join is an outer join.
The children promotion is done to avoid join chains that contain a LOUTER
b INNER c. So, if we have currently a INNER b INNER c and a->b is promoted,
then we must also promote b->c automatically, or otherwise the promotion
of a->b doesn't actually change anything in the query results.
"""
aggregate_if.py seems to choke on this at line 122
# Django 1.5+
if hasattr(query, 'promote_joins'):
query.promote_joins(join_list, True)
Removing the True arg seems to make it work, but I'm not sure if it passes all the relevant tests...
The text was updated successfully, but these errors were encountered:
It seems that this packages is pretty well adjusted for Django1.7, except for an issue with the query.promote_joins() method.
I'm not 100% sure, but it appears that in Django1.6, promote_joins allows for an unconditional kwarg to be passed:
but in Django1.7, this kwarg is removed:
aggregate_if.py seems to choke on this at line 122
Removing the True arg seems to make it work, but I'm not sure if it passes all the relevant tests...
The text was updated successfully, but these errors were encountered: