Updating Items from JsonRestStore about dojo-calendar

[cjolif]

Can you provide a sample example that does reproduce your issue? Or at least a code excerpt that shows how you do your polling / refresh?
Thanks.

[httpete]

Christophe,

Thank you for helping us. We are very excited about this new control you have built.

In my startup() I do: (

var o = {
start: stamp.toISOString(interval[0], opts),
end: stamp.toISOString(interval[1], opts)
};
this.query = o;
calendar.set("store", new Observable(new JsonRestStore({target: "appointments/"})));

and then, whenever the interval changes,

var o = {
start: stamp.toISOString(interval[0], opts),
end: stamp.toISOString(interval[1], opts)
};
var results = this.store.query(o, { overwrite: true });
results = results.map(lang.hitch(this, function(item){
return this.itemToRenderItem(item, this.store);
}));
when(results, lang.hitch(this, "_initItems"));

and I poll that (refresh that call ) every 30 seconds. At first I thought all was well, and then I noticed my Firebug HTML inspector scrollbar grow and grow every 30 seconds. It is overlaying the events, instead of refreshing them.

We can do a teamviewer if this would help you. Let me know. We can also support this project financially.

Pete

[cjolif]

Thanks for your kind words. So you are re-executing that piece of code:

var results = this.store.query(o, { overwrite: true }); 
results = results.map(lang.hitch(this, function(item){ return this.itemToRenderItem(item, this.store); })); when(results, lang.hitch(this, "_initItems"));

on each poll request?

If yes then I understand why you get that behavior. For the items to be correctly updated automatically and not re-created you need to rely on the store observation mechanism. And that's not what you are apparently doing. I guess because JsonRest does not provide a polling mechanism so you implemented it yourself.

So you either needs to wrap JsonRest and implement a polling mechanism to correctly notify Observable of data changes or if you still want to manage that in your code you are probably better of calling _updateItems than _initItems as you do now. Indeed _updateItems will make sure to not create items but just updates them.

[damiengarbarino]

I'm closing this issue as Christophe seems to have answered correctly. If something is still unclear, we can reopen it.