Skip to content

Commit

Permalink
Merge pull request #7 from gastonrobledo/master
Browse files Browse the repository at this point in the history
Adding error propagation
  • Loading branch information
gastonrobledo committed Jun 25, 2019
2 parents 057fa56 + 9744de2 commit 0bba9b4
Show file tree
Hide file tree
Showing 2 changed files with 14 additions and 7 deletions.
19 changes: 13 additions & 6 deletions lib/handlebars.runtime.js
Original file line number Diff line number Diff line change
Expand Up @@ -25,13 +25,20 @@ function wrapTemplate(env, compiledFuncTemplate) {
return async(data, options) => {
/* eslint-disable no-param-reassign */
env.stream = options.stream || env.stream
const dataParams = Object.assign(data, {
async callback(item) {
const value = await processResult(item, env.stream)
return value
try {
const dataParams = Object.assign(data, {
async callback(item) {
const value = await processResult(item, env.stream)
return value
}
})
return await compiledFuncTemplate(dataParams)
} catch (e) {
if (env.stream) {
env.stream.emit('error', e)
}
})
return await compiledFuncTemplate(dataParams)
return false
}
}
}
// For compatibility and usage outside of module systems, make the Handlebars object a namespace
Expand Down
2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"name": "handlebars-stream-render",
"version": "1.1.0",
"version": "1.1.2",
"description": "Handlebars modification to work with stream outputs",
"keywords": [
"handlebars",
Expand Down

0 comments on commit 0bba9b4

Please sign in to comment.