fix: STRF-9673 Fix null params passed to helpers (#240)

helpers/all.js

@@ -15,7 +15,7 @@ const factory = () => {
         const opts = args.pop();
 
         // Check if all the args are valid / truthy
-        const result = args.every( function (arg) {
+        const result = args.every(function (arg) {
             if (utils.isArray(arg)) {
                 return !!arg.length;
             }

helpers/contains.js

@@ -14,7 +14,7 @@ const factory = () => {
     return function(container, value) {
         const options = arguments[arguments.length - 1];
         const preparedContainer = utils.isObject(container) ? Object.values(container) : container;
-        const contained = preparedContainer.includes(value);
+        const contained = preparedContainer ? preparedContainer.includes(value) : false;
 
         // Yield block if true
         if (contained) {

helpers/deprecated/pick.js

@@ -9,7 +9,7 @@ const factory = () => {
         const target = args.shift();
         const toReturn = {};
         const paths = args[0];
-        if (paths) {
+        if (paths && Array.isArray(paths)) {
             paths.forEach((key) => {
                 if (target.hasOwnProperty(key)) {
                     toReturn[key] = target[key];

helpers/join.js

@@ -5,6 +5,9 @@ const factory = () => {
         const options = arguments[arguments.length - 1];
         var config = options.hash || {};
 
+        if (!Array.isArray(array)) {
+            throw new TypeError("Non-array passed to join helper");
+        }
         array = array.slice();
 
         // Truncate array

helpers/pluck.js

@@ -2,7 +2,10 @@
 
 const factory = () => {
     return function(collection, path) {
-        return collection.map(item => item.hasOwnProperty(path) ? item[path] : undefined);
+        if (collection) {
+            return collection.map(item => item.hasOwnProperty(path) ? item[path] : undefined);
+        }
+        return [];
     };
 };
 

spec/helpers/all.js

@@ -16,7 +16,8 @@ describe('all helper', function() {
         emptyObject: {},
         itemArray: [1,2],
         emptyString: "",
-        big: 'big'
+        big: 'big',
+        alwaysNull: null,
     };
 
     // Build a test runner that uses a default context
@@ -114,6 +115,10 @@ describe('all helper', function() {
                 input: '{{#all true "" alwaysTrue}}big{{/all}}',
                 output: '',
             },
+            {
+                input: '{{#all alwaysNull}}big{{/all}}',
+                output: '',
+            },
         ], done);
     });
 });

spec/helpers/any.js

@@ -65,6 +65,10 @@ describe('any helper (with option hash)', function() {
                     foobar: undefined
                 }
             },
+            {
+                input: '{{#any null num=null}}{{big}}{{/any}}',
+                output: '',
+            },
         ], done);
     });
 });

spec/helpers/join.js

@@ -6,7 +6,8 @@ const Lab = require('lab'),
 
 describe('join helper', function() {
     const context = {
-        list: ['Mario', 'Chris', 'Mick', 'Hau', 'Cody']
+        list: ['Mario', 'Chris', 'Mick', 'Hau', 'Cody'],
+        notArray: null,
     };
 
     const runTestCases = testRunner({context});

spec/helpers/pluck.js

@@ -9,7 +9,9 @@ describe('pluck helper', function() {
         users: [
           { 'user': 'barney', 'age': 36 },
           { 'user': 'fred',   'age': 40 }
-        ]
+        ],
+        null: null,
+        undefined: undefined,
     };
 
     const runTestCases = testRunner({context});
@@ -39,4 +41,17 @@ describe('pluck helper', function() {
             },
         ], done);
     });
+
+    it('should return empty array when null is provided', function(done) {
+        runTestCases([
+            {
+                input: '{{pluck null "age"}}',
+                output: '',
+            },
+            {
+                input: '{{pluck undefined "age"}}',
+                output: '',
+            },
+        ], done);
+    });
 });

spec/helpers/thirdParty.js

@@ -7,6 +7,7 @@ const Lab = require('lab'),
 describe('third party handlebars-helpers', function() {
     const context = {
         array: [1, 2, 3, 4, 5],
+        null: null,
         object: {"a" : 1, "b" : 2},
         options: { a: { b: { c: 'd' } } }
     };
@@ -90,6 +91,14 @@ describe('third party handlebars-helpers', function() {
                     },
                 ], done);
             });
+            it('renders the else block if it evaluates to false. null as input', function(done) {
+                runTestCases([
+                    {
+                        input: `{{#contains null '3'}}This will not be rendered.{{else}}This will be rendered.{{/contains}}`,
+                        output: 'This will be rendered.',
+                    },
+                ], done);
+            });
         });
 
     });