remotestorage.js icon indicating copy to clipboard operation
remotestorage.js copied to clipboard

Empty files leads to infinite fetching during sync process

Open untitaker opened this issue 8 years ago • 5 comments

This has the same root cause as https://github.com/remotestorage/remotestorage.js/issues/187

I wonder whether we should introduce a node type, with methods that do the rigorous checking for us. Currently the code of sync.js is littered with typeof-checks, unfortunately only partially.

untitaker avatar Oct 15 '15 23:10 untitaker

Good idea.

raucao avatar Oct 16 '15 12:10 raucao

Where would I define such a type?

untitaker avatar Oct 16 '15 12:10 untitaker

It's a fairly complex topic, as you also have to implement it with the caching in mind.

raucao avatar Oct 16 '15 12:10 raucao

It would only replace the {...} construction I see in makeNode. It's not even a real design change?

untitaker avatar Oct 16 '15 12:10 untitaker

Oh, ok. Haven't looked into it specifically. Just explore on your own.

raucao avatar Oct 16 '15 12:10 raucao