[BUGFIX beta] Fix initial render of {{input type=bound}} for checkboxes

packages/ember-handlebars/lib/controls.js

@@ -6,12 +6,21 @@ import Ember from "ember-metal/core"; // Ember.assert
 // var emberAssert = Ember.assert;
 
 import EmberHandlebars from "ember-handlebars-compiler";
+import { handlebarsGet } from "ember-handlebars/ext";
 var helpers = EmberHandlebars.helpers;
 /**
 @module ember
 @submodule ember-handlebars-compiler
 */
 
+function _resolveOption(context, options, key) {
+  if (options.hashTypes[key] === "ID") {
+    return handlebarsGet(context, options.hash[key], options);
+  } else {
+    return options.hash[key];
+  }
+}
+
 /**
 
   The `{{input}}` helper inserts an HTML `<input>` tag into the template,
@@ -192,11 +201,9 @@ export function inputHelper(options) {
 
   var hash = options.hash,
       types = options.hashTypes,
-      inputType = hash.type,
+      inputType = _resolveOption(this, options, 'type'),
       onEvent = hash.on;
 
-  Ember.assert('You can only use a string as a `type` parameter, not a variable', types.type === 'STRING' || types.type === undefined);
-
   delete hash.type;
   delete hash.on;
 

packages/ember-handlebars/tests/controls/checkbox_test.js

@@ -89,26 +89,35 @@ test("It works", function() {
   }, /you must use `checked=/);
 });
 
-QUnit.module("{{input type='checkbox'}} - prevent dynamic type", {
+QUnit.module("{{input type=boundType}}", {
   setup: function() {
+    controller = {
+      inputType: "checkbox",
+      isChecked: true,
+    };
+
     checkboxView = EmberView.extend({
       controller: controller,
-      inputType: "checkbox",
-      template: compile('{{input type=inputType}}')
+      template: compile('{{input type=inputType checked=isChecked}}')
     }).create();
+
+    append();
   },
 
   teardown: function() {
     destroy(checkboxView);
   }
 });
 
-test("It works", function() {
-  expectAssertion(function() {
-    append();
-  }, /not a variable/);
+test("should append a checkbox", function() {
+  equal(checkboxView.$('input[type=checkbox]').length, 1, "A single checkbox is added");
 });
 
+// Checking for the checked property is a good way to verify that the correct
+// view was used.
+test("checkbox checked property is updated", function() {
+  equal(checkboxView.$('input').prop('checked'), true, "the checkbox is checked");
+});
 
 QUnit.module("{{input type='checkbox'}} - static values", {
   setup: function() {