Persisting Cargo needs improvement

[paulrayner]

Currently the store method in CargoRepository does a find first, then a delete if it finds a document. It does this because it had trouble detecting that a Cargo document already existing when doing an update after the Delivery object had been updated.

There should be a way to make this work without the find and delete step, and without having to store an extra Mongoid attribute in Cargo.

[drsharp]

Can't you do a Cargo create within a rescue block? It's not ideal but that would work and would save you the "find + delete + create" dance.

[paulrayner]

Sure. I'll try that. Thanks.