executing app initializers with the app object as the context

latentflip · Jan 24, 2012 · 96a0c8e · 96a0c8e
1 parent 9ac12ed
commit 96a0c8e
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/backbone.marionette.js b/backbone.marionette.js
@@ -376,7 +376,7 @@ Backbone.Marionette = (function(Backbone, _, $){
       this.trigger("initialize:before", options);
       for(var i=0; i<this.initializers.length; i++){
         var initializer = this.initializers[i];
-        initializer(options);
+        initializer.call(this, options);
       }
       this.trigger("initialize:after", options);
     },

diff --git a/spec/javascripts/application.spec.js b/spec/javascripts/application.spec.js
@@ -11,6 +11,7 @@ describe("marionette application", function(){
         var module = {};
         module.initializer = function(options){
           module.options = options;
+          module.context = this;
         };
 
         spyOn(module, "initializer").andCallThrough();
@@ -39,6 +40,10 @@ describe("marionette application", function(){
     it("should pass the options through to the initializer", function(){
       expect(MyModule.options).toEqual(someOptions);
     });
+
+    it("should run the initializer with the context of the app object", function(){
+      expect(MyModule.context).toEqual(MyApp);
+    });
   });
 
   describe("when instantiating an app with options specified", function(){