Merge pull request #1036 from geogolem/filteringFix

im not too sure why this fixes the problem
pull/2/head
geogolem 8 years ago committed by GitHub
commit 593fa4239f

@ -150,10 +150,10 @@ module.exports = Marionette.Layout.extend({
this._showFooter(); this._showFooter();
}); });
this.listenTo(this.seriesCollection, 'sync', function(model, collection, options) { /*this.listenTo(this.seriesCollection, 'sync', function(model, collection, options) {
this._renderView(); this._renderView();
//MoviesCollectionClient.fetch(); //MoviesCollectionClient.fetch();
}); });*/
this.listenTo(this.seriesCollection, "change", function(model) { this.listenTo(this.seriesCollection, "change", function(model) {
if (model.get('saved')) { if (model.get('saved')) {
model.set('saved', false); model.set('saved', false);
@ -292,13 +292,15 @@ module.exports = Marionette.Layout.extend({
}, },
onShow : function() { onShow : function() {
this.listenToOnce(this.seriesCollection, 'sync', function() {
this._showToolbar(); this._showToolbar();
//this._fetchCollection(); //this._fetchCollection();
if (window.shownOnce) { if (window.shownOnce) {
this._fetchCollection(); //this._fetchCollection();
this._showFooter(); this._showFooter();
} }
window.shownOnce = true; window.shownOnce = true;
});
}, },
_showTable : function() { _showTable : function() {

Loading…
Cancel
Save