allow afterRender on module binding to be defined later when passing …

…an observable and initial value is null/undefined.

fixes #16

bower.json

@@ -1,6 +1,6 @@
 {
     "name": "knockout-amd-helpers",
-    "version": "0.6.1",
+    "version": "0.6.2",
     "main": "build/knockout-amd-helpers.min.js",
     "ignore": [
         "examples",

build/knockout-amd-helpers.js

@@ -1,4 +1,4 @@
-// knockout-amd-helpers 0.6.1 | (c) 2014 Ryan Niemeyer |  http://www.opensource.org/licenses/mit-license
+// knockout-amd-helpers 0.6.2 | (c) 2014 Ryan Niemeyer |  http://www.opensource.org/licenses/mit-license
 define(["knockout"], function(ko) {
 
 //helper functions to support the binding and template engine (whole lib is wrapped in an IIFE)
@@ -45,18 +45,16 @@ ko.bindingHandlers.module = {
             disposeMethod = ko.bindingHandlers.module.disposeMethod;
 
         //build up a proper template binding object
-        if (options && typeof options === "object") {
-            templateBinding.templateEngine = options.templateEngine;
+        templateBinding.templateEngine = options && options.templateEngine;
 
-            //afterRender could be different for each module, create a wrapper
-            templateBinding.afterRender = function() {
-                var options = unwrap(valueAccessor());
+        //afterRender could be different for each module, create a wrapper
+        templateBinding.afterRender = function() {
+            var options = unwrap(valueAccessor());
 
-                if (options && typeof options.afterRender === "function") {
-                    options.afterRender.apply(this, arguments);
-                }
-            };
-        }
+            if (options && typeof options.afterRender === "function") {
+                options.afterRender.apply(this, arguments);
+            }
+        };
 
         //if this is not an anonymous template, then build a function to properly return the template name
         if (!isAnonymous(element)) {

package.json

@@ -1,6 +1,6 @@
 {
   "name": "knockout-amd-helpers",
-  "version": "0.6.1",
+  "version": "0.6.2",
   "devDependencies": {
     "grunt": "~0.4.1",
     "grunt-contrib-uglify": "~0.2.0",

spec/amdBindings.spec.js

@@ -469,6 +469,22 @@ define(["knockout", "knockout-amd-helpers"], function(ko) {
                     });
                 });
 
+                it("should call afterRender, even when initial value was null", function(done) {
+                    var observable = ko.observable(null),
+                        afterRender = jasmine.createSpy("afterRender");
+
+                    applyBindings("module: test", { test: observable }, null, function() {
+                        updateObservable(observable, {
+                            name: "static-no-initialize",
+                            afterRender: afterRender
+                        }, function() {
+                            expect(container.innerText).toEqual("static-no-initialize: Bob Smith");
+                            expect(afterRender.calls.count()).toEqual(1);
+                            done();
+                        });
+                    });
+                });
+
                 describe("observable is empty", function() {
                     it("should not error out initially", function(done) {
                         var observable = ko.observable();

src/amdBindings.js

@@ -9,18 +9,16 @@ ko.bindingHandlers.module = {
             disposeMethod = ko.bindingHandlers.module.disposeMethod;
 
         //build up a proper template binding object
-        if (options && typeof options === "object") {
-            templateBinding.templateEngine = options.templateEngine;
+        templateBinding.templateEngine = options && options.templateEngine;
 
-            //afterRender could be different for each module, create a wrapper
-            templateBinding.afterRender = function() {
-                var options = unwrap(valueAccessor());
+        //afterRender could be different for each module, create a wrapper
+        templateBinding.afterRender = function() {
+            var options = unwrap(valueAccessor());
 
-                if (options && typeof options.afterRender === "function") {
-                    options.afterRender.apply(this, arguments);
-                }
-            };
-        }
+            if (options && typeof options.afterRender === "function") {
+                options.afterRender.apply(this, arguments);
+            }
+        };
 
         //if this is not an anonymous template, then build a function to properly return the template name
         if (!isAnonymous(element)) {