comments app template tags don't resolve context variables

[James Addison]

From the django comments app docs, you're able to specify the app.model and object_pk if you don't have an object in the template context, like this:

{% get_comment_list for blog.post object_pk as comment_list %}

However, I have a scenario where I want to pass in the app.model value in a context variable. eg. say I have a variable called object_content_type containing 'blog.post', this errors out:

{% get_comment_list for object_content_type object_pk as comment_list %}

as does this:

{% get_comment_list for 'blog.post' object_pk as comment_list %}

Error info:

TemplateSyntaxError at /my-comments-page/
Third argument in u'get_comment_list' must be in the format 'app.model'

Near as I can figure it, the comments app template tags don't seem to attempt variable resolution of the token. Shouldn't it?

[James Addison]

Just a note: isn't this similar to the url vs url from future issue that required deprecation over 2 or 3 versions?

[Collin Anderson]

I'm not a core developer, but as far as I can tell, everything you have reported is correct and probably a good idea.

However, the entire comments app is deprecated and has been pulled out to a separate project. I suggest proposing this idea there instead.

​https://github.com/django/django-contrib-comments

More info about the deprecation here:
​https://docs.djangoproject.com/en/dev/ref/contrib/comments/

[Tim Graham]

Correct, bugs for contrib.comments are no longer reported here.