.replace() works once, then never again. about horizon HOT 4 CLOSED

Hmm, maybe the listener is getting killed

from horizon.

Nope, all my fault. The changed.new_val got me again. The onChanged function above should be this:

onChanged: (changed) => {
     this.todos = this.todos.map((todo) => {
      return todo.id !== changed.new_val.id ? todo : Utils.extend({}, todo, changed.new_val);
     });
     this.inform();
},

from horizon.

I think onChanged should call a callback with two arguments (new and old values), instead of a single argument that combines both into an object.

from horizon.

Err, these are observables and I should be embarrassed and ashamed of myself for the comment above (which I am).

from horizon.