From 9adf51670ee36ef558a1a144f1f73968c19ac82d Mon Sep 17 00:00:00 2001 From: jankapunkt Date: Mon, 3 Feb 2025 14:58:16 +0100 Subject: [PATCH] feature(blaze): add explicit helper name + viewname to errors, causes within helpers --- packages/blaze/lookup.js | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/packages/blaze/lookup.js b/packages/blaze/lookup.js index af30813c3..682e8b98f 100644 --- a/packages/blaze/lookup.js +++ b/packages/blaze/lookup.js @@ -75,7 +75,8 @@ Blaze._getTemplateHelper = function (template, name, tmplInstanceFunc) { if (helper === Blaze._OLDSTYLE_HELPER) { isKnownOldStyleHelper = true; } else if (helper != null) { - return wrapHelper(bindDataContext(helper), tmplInstanceFunc); + const printName = `${template.viewName} ${name}`; + return wrapHelper(bindDataContext(helper), tmplInstanceFunc, printName); } else { return null; } @@ -100,7 +101,7 @@ Blaze._getTemplateHelper = function (template, name, tmplInstanceFunc) { return null; }; -const wrapHelper = function (f, templateFunc) { +const wrapHelper = function (f, templateFunc, name = 'template helper') { if (typeof f !== "function") { return f; } @@ -109,7 +110,7 @@ const wrapHelper = function (f, templateFunc) { const self = this; return Blaze.Template._withTemplateInstanceFunc(templateFunc, function () { - return Blaze._wrapCatchingExceptions(f, 'template helper').apply(self, args); + return Blaze._wrapCatchingExceptions(f, name).apply(self, args); }); }; }; @@ -165,7 +166,8 @@ Blaze._getTemplate = function (name, templateInstance) { Blaze._getGlobalHelper = function (name, templateInstance) { if (Blaze._globalHelpers[name] != null) { - return wrapHelper(bindDataContext(Blaze._globalHelpers[name]), templateInstance); + const printName = `global helper ${name}`; + return wrapHelper(bindDataContext(Blaze._globalHelpers[name]), templateInstance, printName); } return null; };