MySQL syntax error on object.all().count(), when called from template

[gumuz]

I have a blogpost model which is referenced by a Comment class. Thus, the Blogpost model has a comment_set attribute.

I gave the Blogpost model a method like this:

def comments(self):
    return self.comment_set.all()

When i access it from the shell and use the .count() method on the QuerySet, everything is fine:

>>> from devlog.weblog.models import Post
>>> p = Post.objects.all()[4]
>>> p.comments()
[<Comment: gumuz - 2006-07-12 07:10:30>, <Comment: gumuz - 2006-07-12 07:09:56>]
>>> p.comments().count()
2L

When i do the same from my template:

{{ object.comments.count }}

I get the following error:

Exception Type:  	ProgrammingError
Exception Value: 	(1064, "You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'count,1' at line 1")
Exception Location: 	/usr/lib/python2.4/site-packages/MySQLdb/connections.py in defaulterrorhandler, line 33

The length-filter works fine though:

{{ object.comments|length }}

[racter@…]

this error only happens for me when nesting -- ie if i try:

{% for object in category.object_set.all %}
{{ object.subobject_set.count }}
{% endfor %}

i get the same error.

[Jonas von Poser]

I get the same error. count fails from template but |length works fine.

[Michael Radziej <mir@…>]

Please, send the full bracktrace.

[marcin@…]

I just ran into the same problem with Postgresql.

The cause is a bug in QuerySet.getitem: it assumes that the parameter it gets is either a slice or an integer, but does not really check the latter. When you put {{ somequeryset.qwe }} in your template and template.resolve_variable tries to resolve it using a dictionary lookup, QuerySet.getitem treats qwe like an integer index, which results in queries ending with "limit 1 offset qwe".

The attached patch fixes the problem.

[mir@…]

It would be rather nice if someone posts the full backtrace. I personally refuse to triage a bug report without the full information, and it's rather tedious to reconstruct the problem when you're dealing with a lot of tickets.

[marcin@…]

mir: here you are. Note the resulting SQL query at the end.

All the interesting parts of code:

models.py:

    class Article(models.Model):
        text = models.TextField()

urls.py:

    urlpatterns = patterns('',
        (r'^$', 'django.views.generic.simple.direct_to_template',
         {'extra_context': {'object_list': Article.objects.all()},
          'template': 'index.html'}),
    )

index.html:

{{ object_list.count }}

I tried to attach the detailed backtrace in HTML, but Akismet said it was spam.

[Chris Beaven]

I'll accept. Probably could do with some tests showing the failure.

[Noah Slater <nslater@…>]

I can confirm identical behaviour.

[James Bennett]

#2945 was a duplicate of this one.

[James Bennett]

And for the record, #2945 has a full traceback of this bug. And I've seen it happen on SQLite as well.

[Malcolm Tredinnick]

(In [4772]) Fixed #2351 -- Fixed problem with using ".count" attribute of QuerySets in
templates.