Duplicate Add Event about backbone-offline HOT 2 OPEN

I have found the cause of this behavior.

After an ajax 'read', items not yet in memory are passed to @collection.items.create(item, *args)

I.E. the create() method on the backbone collection. The following then happens:

1. model created from item
2. model added to collection with server attributes.
3. ( collection add event )
4. model.save()
5. Model.save() triggers an Offline.sync() 'update'.
6. model gets a local guid, and is saved to local-storage.

I'm not familiar with the framework, so may be missing something. But I thought create + PUT were for newly created entities, not all entities on the server (when they are first fetched).

Here - the purpose seems to be a way to get a 'cid' assigned by the collection, before saving the model to localstorage. I'm gussing this is a hack to avoid calling collection._prepareModel() from outside the collection.

But I can't yet think of a way to fix the 'double add-event' bug without changing the control flow.

Calling _prepareModel() outside collection is bad practice. But if you were flexible on this, would solve bug, and control flow would be much simpler + more intuitive.

One way to make this 'better' practice might be to implement Backbone.Offline using a special Collection subclass, and make a public gateway to _prepareModel().

I personally find this a nice way to implement Offline in my project anyway. I have an OnlineCollection class:

var OfflineCollection = Backbone.Collection.extend({

    storageName: 'default',

    initialize: function() {

        console.log('setting up storage for '+ this.storageName)

        this.storage = new Offline.Storage( this.storageName, this );

    }

});

Nice for DRY - just put storageName in subclass.

But maybe you guys want to touch as little of the Backbone prototypes as possible - which is understandable. Any other suggestions to fix this?

In short run I have deadline, so may have to make my own non-standard branch. In mid term, very much look forward to discussion - I'm happy to contribute to this project!

Dalip

from backbone-offline.

This sounds very similar to an issue I'm having where the success method of particular fetch is called twice when the collection is not in localStorage. This in turn, causes an error for my control flow situation because it returns an empty collection on success the first time, then later returns the populated collection after the success magic has already happened with an empty collection.

from backbone-offline.