Think I finally figured out that stupid race condition.

This commit is contained in:
Elf M. Sternberg 2010-12-08 17:30:54 -08:00
parent 8180650836
commit bedb20bb0b
1 changed files with 6 additions and 24 deletions

View File

@ -50,9 +50,7 @@ var ProductView = Backbone.View.extend({
}, },
initialize: function(options) { initialize: function(options) {
if (options.cart) {
this.cart = options.cart; this.cart = options.cart;
}
}, },
update: function(e) { update: function(e) {
@ -80,7 +78,7 @@ var ProductView = Backbone.View.extend({
}); });
return this; return this;
} }
}) });
var IndexView = Backbone.View.extend({ var IndexView = Backbone.View.extend({
@ -121,14 +119,9 @@ var Workspace = Backbone.Controller.extend({
success: function(data) { success: function(data) {
ws._cart = new Cart(); ws._cart = new Cart();
new CartView({model: ws._cart}); new CartView({model: ws._cart});
ws._products = new ProductCollection(); ws._products = new ProductCollection(data);
ws._products.refresh(data);
ws._index = new IndexView({model: ws._products}); ws._index = new IndexView({model: ws._products});
if (Backbone.history) { Backbone.history.loadUrl();
Backbone.history.loadUrl()
} else {
ws.index();
}
} }
}); });
return this; return this;
@ -137,21 +130,10 @@ var Workspace = Backbone.Controller.extend({
}, },
index: function() { index: function() {
/* Race sentinel. : View loaded before data ready.
* Backbone.history will load as appropriate. */
if (this._index === null) {
return;
}
this._index.render(); this._index.render();
}, },
item: function(id) { item: function(id) {
if (this._products === null) {
/* Race sentinel. That's two. I'm Doing Something Wrong. */
return;
}
if (_.isUndefined(this._products.getByCid(id)._view)) { if (_.isUndefined(this._products.getByCid(id)._view)) {
this._products.getByCid(id)._view = new ProductView({model: this._products.getByCid(id), this._products.getByCid(id)._view = new ProductView({model: this._products.getByCid(id),
cart: this._cart}); cart: this._cart});