Can't use loop var "class" with asyncEach

[softshape]

We've used {% include %} not only for templates, but also to insert SVG to HTML -

{% include "img/face" + user.face + ".svg" %}

It worked fine until we switched nunjucks to async mode. Here's our simple loader:

var NunjucksLoader = nunjucks.Loader.extend({
  async: true,
  getSource: function (name, callback)
  {
    $.ajax(name, {dataType: "text"})
      .done(function (data)
      {
    console.log('NunjucksLoader loaded ', name)
        var res = {src: {type: "string", obj: data}, path: name};
        callback(null, res);
      })
      .fail(function (e) {callback(e, null);});
  }
});

It works, generally, we can see 'NunjucksLoader loaded ' messages in the browser console. But SVG content won't appear on the resulting HTML. Included templates are fine, but included SVGs aren't.

Is there any fix for that?

[devoidfury]

Is the include inside a loop? Make sure you use the asyncEach tag instead of for if you are, as the for tag compiles directly into a javascript for loop and does not work with async.

http://mozilla.github.io/nunjucks/api.html#asynchronous

[softshape]

I've tested templates both with for and asyncEach, no difference beetwen them.

[carljm]

possibly related to #372?

[softshape]

I've found the reason.

{% for class in json %} - works in sync mode
{% asyncEach class in json %} - doesn't work
{% asyncEach users in json %} - works!

So the name "class" used as loop variable works in the sync and fails in async mode. I've changed it and the template is OK now.

[carljm]

This still seems like a bug that ideally would be fixed, if your analysis of the problem is correct.