Ensure apply is not called with undefined.

Fixes a bug in IE8 Fixes #2225
marionettejs · Jan 30, 2015 · be6f1b4 · be6f1b4 · jridgewell · Jan 30, 2015
1 parent 2478c5d
commit be6f1b4
Showing 1 changed file with 4 additions and 0 deletions.
diff --git a/src/helpers.js b/src/helpers.js
@@ -43,6 +43,10 @@ Marionette.proxyGetOption = function(optionName) {
 // undefined return a default value
 Marionette._getValue = function(value, context, params) {
   if (_.isFunction(value)) {
+    // We need to ensure that params is not undefined
+    // to prevent `apply` from failing in ie8
+    params = params || [];
+
     value = value.apply(context, params);
   }
   return value;