Merge pull request #767 from marionettejs/sjs/move-options-assignment

[fix] Move the instatiation of view options above the constructor

spec/javascripts/view.spec.js

@@ -144,6 +144,19 @@ describe("base view", function(){
     });
   });
 
+  describe("should expose its options in the constructor", function() {
+    var View = Marionette.View.extend({
+       initialize: function() {
+        this.info = this.options;
+      }
+    });
+
+    it("should be able to access instance options", function() {
+      var myView = new View({name: "LeChuck"});
+      expect(myView.info.name).toBe("LeChuck");
+    });
+  });
+
   describe("when closing a view that is already closed", function(){
     var close, view;
 

src/marionette.view.js

@@ -8,8 +8,13 @@ Marionette.View = Backbone.View.extend({
     _.bindAll(this, "render");
 
     var args = Array.prototype.slice.apply(arguments);
-    Backbone.View.prototype.constructor.apply(this, args);
+
+    // this exposes view options to the view initializer
+    // this is a backfill since backbone removed the assignment
+    // of this.options
+    // at some point however this may be removed
     this.options = options || {};
+    Backbone.View.prototype.constructor.apply(this, args);
 
     Marionette.MonitorDOMRefresh(this);
     this.listenTo(this, "show", this.onShowCalled, this);