django_compressor killed my 'name' variable

[peterbe]

My basic template looked something like this:

{% extends "base.html" %}
{% block body %} Hi {{ name }} {% endblock %}

All was fine until I added django_compressor. Now template context variable name becomes None.

I know that "name" is a poor choice of a variable but django_compressor shouldn't interfere anyway. Going to do some more debugging into django_compressor to confirm that this is the case. I changed from "name" to "foobar_name" (which is better anyway) and that solved my immediate problem.

[peterbe]

Culprit: https://github.com/jezdez/django_compressor/blob/develop/compressor/templatetags/compress.py#L81

it's a much to generic name and too risky to squeeze in to the context.

[judy2k]

Just dropping a note to say that I also have this issue.

[jezdez]

Agreed, this is kinda related to #151. A combined fix would be appreciated.

[judy2k]

I'm coding overflow the weekend. I'll see if I can pull together a fix.
Cheers Jannis!

This message was sent from my phone, so please excuse its brevity,
spelling, and punctuation.
On 17 Nov 2011 22:14, "Jannis Leidel" <
reply@reply.github.com>
wrote:

Agreed, this is kinda related to #151. A combined fix would be appreciated.

---

Reply to this email directly or view it on GitHub:
#156 (comment)

[judy2k]

I have no idea why my phone decided I meant overflow instead of over...

This message was sent from my phone, so please excuse its brevity,
spelling, and punctuation.
On 17 Nov 2011 22:16, "Mark Smith" mark.smith@practicalpoetry.co.uk wrote:

I'm coding overflow the weekend. I'll see if I can pull together a fix.
Cheers Jannis!

This message was sent from my phone, so please excuse its brevity,
spelling, and punctuation.
On 17 Nov 2011 22:14, "Jannis Leidel" <
reply@reply.github.com>
wrote:

Agreed, this is kinda related to #151. A combined fix would be
appreciated.

---

Reply to this email directly or view it on GitHub:

#156 (comment)