Merge pull request #11260 from csantero/array-arg

[BUGFIX beta] fix passing array argument to action helper

packages/ember-routing-htmlbars/lib/keywords/closure-action.js

@@ -63,7 +63,8 @@ function createClosureAction(target, action, valuePath, actionArguments) {
     closureAction = function() {
       var args = actionArguments;
       if (arguments.length > 0) {
-        args = actionArguments.concat(...arguments);
+        var passedArguments = Array.prototype.slice.apply(arguments);
+        args = actionArguments.concat(passedArguments);
       }
       if (valuePath && args.length > 0) {
         args[0] = get(args[0], valuePath);

packages/ember-routing-htmlbars/tests/helpers/closure_action_test.js

@@ -164,6 +164,42 @@ if (Ember.FEATURES.isEnabled("ember-routing-htmlbars-improved-actions")) {
     innerComponent.fireAction();
   });
 
+  QUnit.test("array arguments are passed correctly to action", function(assert) {
+    assert.expect(3);
+
+    const first = 'foo';
+    const second = [3, 5];
+    const third = [4, 9];
+
+    innerComponent = EmberComponent.extend({
+      fireAction() {
+        this.attrs.submit(second, third);
+      }
+    }).create();
+
+    outerComponent = EmberComponent.extend({
+      layout: compile(`
+        {{view innerComponent submit=(action outerSubmit first)}}
+      `),
+      innerComponent,
+      value: '',
+      outerSubmit(actualFirst, actualSecond, actualThird) {
+        assert.equal(actualFirst, first, 'action has the correct first arg');
+        assert.equal(actualSecond, second, 'action has the correct second arg');
+        assert.equal(actualThird, third, 'action has the correct third arg');
+      }
+    }).create();
+
+    runAppend(outerComponent);
+
+    run(function() {
+      outerComponent.set('first', first);
+      outerComponent.set('second', second);
+    });
+
+    innerComponent.fireAction();
+  });
+
   QUnit.test("mut values can be wrapped in actions, are settable", function(assert) {
     assert.expect(1);