pub/mongoengine
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fixes for django Q query rendering bug

Browse Source 
Ensures that the QNodes haven't already been processed
Fixes #185
This commit is contained in:
Ross Lawley 2011-06-16 15:00:10 +01:00
parent cae3f3eeff
commit 5e8604967c
2 changed files with 6 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
mongoengine/queryset.py
View File

@ -251,7 +251,8 @@ class QCombination(QNode):
def accept(self, visitor):
for i in range(len(self.children)):
self.children[i] = self.children[i].accept(visitor)
if isinstance(self.children[i], QNode):
self.children[i] = self.children[i].accept(visitor)
return visitor.visit_combination(self)

5
tests/django_tests.py
View File

@ -53,4 +53,7 @@ class QuerySetTest(unittest.TestCase):
t = Template("{% for o in ol %}{{ o.name }}-{{ o.age }}:{% endfor %}")
d = {"ol": self.Person.objects.filter(Q(age=10) | Q(name="C"))}
self.assertEqual(t.render(Context(d)), u'D-10:C-30:')
self.assertEqual(t.render(Context(d)), 'D-10:C-30:')
# Check double rendering doesn't throw an error
self.assertEqual(t.render(Context(d)), 'D-10:C-30:')