diff --git a/public/javascripts/collections/list.js b/public/javascripts/collections/list.js index bb4c415..2cdb87c 100644 --- a/public/javascripts/collections/list.js +++ b/public/javascripts/collections/list.js @@ -18,22 +18,6 @@ localforageBackbone model: Task, offlineSync: Backbone.localforage.sync("tasks"), - - /** Override backbone parse **/ - // parse: function(models){ - // if (this._isModel(models)) - // return models; - // var parsedModels =[]; - // for (var i = 0; i < models.length; i++) { - // if (this._isModel(models[i])) - // parsedModels.push(models[i]); - // else - // parsedModels.push(new this.model(models[i])); - // } - // return parsedModels; - // - // }, - initialize: function(){ // update order on add, remove. // Ref: http://stackoverflow.com/a/11665085/221742 @@ -69,9 +53,11 @@ localforageBackbone /** Updated priority of each member of list **/ updateModelPriority: function() { this.each(function(model, index) { - if (model && model.get('priority')!==index) - model.save({'priority': index}); - + if (model && model.get('priority')!==index){ + model.set({'priority': index}); + if (model.id) + model.save({'priority': index}); + } }, this); }, diff --git a/public/javascripts/views/task.js b/public/javascripts/views/task.js index 62645f3..0e483c4 100644 --- a/public/javascripts/views/task.js +++ b/public/javascripts/views/task.js @@ -230,7 +230,7 @@ define( /** Finish editing an item **/ close: function () { var value = this.$('.edit:first').val().trim(); - if (this.model.get('content')!=value) + if (this.model.get('content')!=value && value!=='') this.model.save({content: value}); this.$el.removeClass('editing'); },