Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Re-add doT - global leaks fixed #92

Merged
merged 1 commit into from Feb 23, 2013

Conversation

Projects
None yet
4 participants
Contributor

Sannis commented Feb 9, 2013

//cc @olado

inDream commented Feb 16, 2013

I'm interested in using doT with consolidate too. But I wondered how to use it with 'partials' feature.
As the example in doT is not clear, I only know how to output the code in console. Here's my steps:

/views/index.dot

<html>
<body>
{{#def.test}}
</body>
</html>

/views/test.def

<p>Test</p>

Use the following code in node

var dots = require('dot').process({path: "./views"});
var result = dots.index();

And the content of 'result' will be:

<html><body><p>Test</p></body></html>

But how to use this feature with consolidate?

Contributor

ForbesLindesay commented Feb 16, 2013

This code looks very similar to the code I'm using for transformers although I seem to be using template where you're using compile. Is there a differnece?

@inDream It would be good to see a pair of templates and the code that results in that error so we can work out what causes it?

inDream commented Feb 17, 2013

@ForbesLindesay I've updated my comment.
In fact I've typed {{def.test}} instead of {{#def.test}} which caused the error.

tj added a commit that referenced this pull request Feb 23, 2013

Merge pull request #92 from Sannis/dot-1.x-support
Re-add doT - global leaks fixed

@tj tj merged commit 579da6b into tj:master Feb 23, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment