handling edge case of empty view cache file

[pkamps]

Alternative solution to this pull request:
#325

[bdunogier]

It looks okay to me, I'd almost merge it right away.

Any counter opinion ?

[nfrp]

@bdunogier : you are the Clustering boss, I can only follow your recommendation here. Is this fix also fixing @alexpagnoni 's issue ?

Thanks for the contributions @pkamps !

[pkamps]

Yes - you want to merge only one of the two pull requests.

[bdunogier]

Well, an empty cache file will generate an error when it gets parsed by the
ViewCache handler, so yes, I would say it fixes it as well.

It would be great if Alex could comment, though, but the time may not be
the best :-)
Le 2 août 2012 19:11, "Nicolas Pastorino" <
reply@reply.github.com>
a écrit :

@bdunogier : you are the Clustering boss, I can only follow your
recommendation here. Is this fix also fixing @alexpagnoni 's issue ?

Thanks for the contributions @pkamps !

---

Reply to this email directly or view it on GitHub:
#386 (comment)

[nfrp]

+1 for a merge then.
@alexpagnoni can step-in even afterwards to confirm whether the fix works in his case too, or not.

[pkamps]

He mentioned that they changed their setup - I'm not sure if he'll be able to confirm it.

[nfrp]

Ok.
Merging now.

[nfrp]

Thank you Philipp for the contribution!
And thanks Bertrand for the review!