Synchronous requests [solved]

MichMich

@paviro Yeah, but try to understand what I did. Thats the best way to learn. Let me know what you don’t understand. I’ll explain.

paviro

I do not quite understand how the itemsReceivedCallbackworks :confused:

paviro

Does work great now! The only problem is sometimes a list is empty on a refresh even tough it should not be :/

{ inbox: [],
  'ViRO Entertainment': 
   [ 'Todo1',
     'Todo2',
     'Todo3' ]
 }

MichMich

I’m on my phone now, so not really able to explain the callbacks. Will do that later.

Regarding the empty lists: try debugging by putting some log statements on places where You receive and send data.

paviro

It is not the socket connection. The fetcher sometimes returns an empty list :/

MichMich

@paviro try debugging the Fetchet on the same way. ;)

paviro

I am on it but I can’t find the problem… :confounded:
When broadcastItems is fired and therefore fetcher.onReceive triggered the lists are populated. But once broadcastTodos() collects the tasks by looping over the fetchers some lists appear to be empty. Any chance you could have a look?

MichMich

Will fork your module tomorrow and give it a try.

paviro

Thanks! :)

MichMich

Ok, I looked into it, and it really is pretty simple and obvious when I’ll explain you. So prepare for a “Doh!” moment … ;)

Right before you do the request to WunderList, you empty the items array:
https://github.com/paviro/MMM-Wunderlist/blob/master/fetcher.js#L38

Then if Wunderlist responds, you fill the items array with new items.

This means that as long as your fetcher is waiting for wunderlist to respond, the items array is empty. If in the meantime an other fetcher is finished fetching, it uses the items array to make a full list. But that array is then still empty, waiting to be filled after the wunderlust response.

The solution? Move line 38 (items = [];) to line 51 (above for (var i = 0; i < JSON.parse(body).length; i++) {).

Thats all! :)