EntityMetadataWrapperException: This node is no book page

Thanks for reporting, problem confirmed.

Looks like we need to catch exceptions when we filter out properties that should not be accessed.

Patch attached, please test.

This should also come with an automated test case using the book module.

Tagging. Needs work for a test case, any volunteers?

Keeping the needs tests tag.

We had precisely the same error, this patch works as a charm. we recommend this will be added to the main branch.
Thanks for the patch!

Glad it works for you - can you write a simpletest for this?

Well at this moment i'm experiencing other issues, which make it a bit hard to test anything. i have created another issue about that, so if i can get this module to work, i'm willing to give it a try.

This patch looks fine and is an incredibly simple fix.

It took me a good while with my debugger to even realise I had the book.module problem that this fixes. I suspect many would have given up, blaming the restws module.

Surely it's not worth holding up including this patch for almost two years just because there's no test? I admire the sentiment but pragmatically, I don't think anyone's going to write a test for this.

Can you please roll it in?

This has been running on our production environment for almost a year without issue. Please merge it.

Alright, committed.