Updating or inserting an item in an observable array about knockout.mapping HOT 7 CLOSED

The reason the mapping plugin does not do this, is that there is no clean way to signal deletions.

from knockout.mapping.

I think it would be alright to not be expected to support deletions this way. I mean, there really isn't a clean way to do it. So, one might be expected to explicitly call a mappedRemove(key).

but I can dig it if you don't want to add it to the library.

from knockout.mapping.

No, I think this alternate behavior could also be a nice feature. But it
would have to be explicitly enabled using a mapping configuration option.
This way it wouldn't break any existing code.
If you can add a pull request implementing that, it would be very welcome :)

On Fri, Dec 9, 2011 at 17:40, Matthew Taylor <
[email protected]

wrote:

I think it would be alright to not be expected to support deletions this
way. I mean, there really isn't a clean way to do it. So, one might be
expected to explicitly call a mappedRemove(key).

but I can dig it if you don't want to add it to the library.

---

Reply to this email directly or view it on GitHub:

#41 (comment)

from knockout.mapping.

turn it on like this?

var vm = ko.mapping.fromJS([], { arrayUpdate: true, arrayInsert:true });

from knockout.mapping.

well, more something along the lines of { incrementalArrayUpdates: true }
to indicate that what you receive from the server is an incremental update,
instead of a 'full' state.
additionally, if the data from the server contains a "_destroy" property in
an array, that could be an indication to delete (similar to what you can do
with array.destroy(item) in an observable array).

On Fri, Dec 9, 2011 at 19:13, Matthew Taylor <
[email protected]

wrote:

turn it on like this?

var vm = ko.mapping.fromJS([], { arrayUpdate: true, arrayInsert:true });

---

Reply to this email directly or view it on GitHub:

#41 (comment)

from knockout.mapping.

I gave this a shot; but I'm out of time. the updateViewModel function is hard to get my mind around. I kept breaking other unit tests without actually getting my change to work. I strongly feel that the function is too large and needs to be refactored into smaller chunks.

I'm giving up for now. I might come back to it later.

from knockout.mapping.

Fair enough, development of the plugin has been fairly, shall we say, iterative :) So there is definitely a lot to be gained by cleaning it up. Sadly, with limited time on my side this has fallen by the wayside.

By the way, if you have any suggestions on restructuring the code and so on, by all means let me know. Good suggestions are always welcome!

from knockout.mapping.