Improper escaping of path (precompiler issue)

When using the below command to precompile:

  node node_modules\nunjucks\bin\precompile app/views/js/presales > app/storage/assets/compiled/js/presale_templates.js

The output in presale_templates.js:

  (function() {(window.nunjucksPrecompiled = window.nunjucksPrecompiled || {})["flooring\index.html"] = (function() {function root(env, context, frame, runtime, cb) {

As you can see this is effectively escaping the character ‘i’ "flooring\index.html" which causes issues in loading the template with the render function.

[jlongster]

@mattbasta that would fix it, but it seems like that actual template names should always use the forward slash /. That way you always say nunjucks.render("foo/bar.html") and it runs on every platform.

Still, no harm doing the JSON thing too for other edge cases.

[garygreen]

This has been fixed a while ago, closing