Not working with multiple files

[timkelty]

options: {
    templatePath: 'src/templates/',
    data: 'src/data/',
    punctuation: '',
  },
  dist: {
    files: {
      'dist/index.html': ['src/templates/index.tpl'],
      'dist/product.html': ['src/templates/product.tpl'],
      'dist/category.html': ['src/templates/category.tpl'],
    }
  }

All files get created, but all but dist/index.html are empty, no matter what they have in the src.

[timkelty]

This seems to happen only when the files have an extends tag, e.g.:
{extends file='layouts/default.tpl'}

[hereandnow]

i'll look into it. got a lot of work at the moment, so this could take a few days.

[timkelty]

Thanks, I appreciate it.

[hereandnow]

sorry for taking so long... as you have already noticed (a saw your opened issue on jsmart-repo) this is an error with jsmart itself. i'll close this issue for now. as long as jsmart isn't publishing a new major version, this should be incorporated automatically if fixed (dependency in package.json is "^2.13.0"

[Avcajaraville]

I know this issue is closed, and I know its related to jsmart. I have posted an issue there, but the project seems not maintained anymore.

With this problem, how is this project even useful ? Isn't that one CAPITAL feature ?

Does anybody knows how to fix it (without modifying the src code) ?

[hereandnow]

Hi Avcajaraville,

i understand your point. I developed this Grunt-Plugin because i needed it for 1 File, so i did not recognize the issue in jsmart.

To fix it i only see 2 possible solutions:

1. someone needs to ask the author of jsmart if he would make another one a contributor and give him the rights to publish jsmart to npm. then he can fix the issue there
2. someone needs to fork jsmart, fix the bug and publish a version under a new name to npm so that i can change the dependency

i would to it, but my time is very very limited. if you are willing to do give it a try, go ahead, i will integrate if the bug is fixed!

[hereandnow]

I took a look into it, it should probably be fairly easy to fix:

just enable the "nocache" option param when calling getTemplate in the built-in extend parse function:

before:

getTemplate(trimQuotes(params.file?params.file:params[0]), tree);

after:

getTemplate(trimQuotes(params.file?params.file:params[0]), tree, true);

[Avcajaraville]

Problem here is how to get this module updated. It doesn't expose anything, and in order to call that function, I will have to fork it, leaving all further support unavailable. I was thinking maybe the author fix it or there is anyway we can activate that option without overwriting the source code.

Thanks a lot for your answer :)

[hereandnow]

i dont see any possibility to do this without forking and merging back/or creating a new npm-module from it. the extend-functionality is not exposed, so "monkey-patching" it is not an option

go ahead and try to fork and make a pull request, maybe he answers... if not publish a module like "jsmart-multiextend" and i will integrate that one

[Avcajaraville]

@hereandnow

I just created a fork and send a pull request.

You can introduce this fork on your package.json as dependency by referencing it like:

"jsmart": "Avcajaraville/jsmart#master"

Waiting for updates on the original repository owner.

[hereandnow]

done, and published v0.1.4: https://www.npmjs.com/package/grunt-jsmart

would be great if you ping me when the owner merged your pull request

[Avcajaraville]

@hereandnow

You can count with that, as soon as it gets merge (or dismiss, or whatever), I will ping you :)

Thanks for merge it !