RestWS module has now support for UUIDs and adds them in for all entity references. However, I ran into an issue where it dies because it tries to lookup the UUID for an vocabulary. This is not support by UUID module, what is fine - however entity_get_uuid_by_id() should bail out properly. Right now it produces a broken query and so a FATAL.

Attached patch makes entity_get_uuid_by_id() work for entity types that are not supported.

CommentFileSizeAuthor
#2 1820786-2-uuid-unsupported-entities.patch865 byteswodenx
uuid_entity_type_support.patch448 bytesfago

Comments

rodmaz’s picture

rodmaz commented

I can confirm this is a major bug. Using 7.x-1.0-alpha3 and applied the patch successfully.

wodenx’s picture

wodenx commented
StatusFileSize
new 1820786-2-uuid-unsupported-entities.patch865 bytes

I also can confirm that this is a major bug.

While the patch above will prevent the fatal, it still generates a PHP notice for entities which don't support uuids. This version fixes that, and extends the fix to cover entity_get_id_by_uuid() as well.

subspaceeddy’s picture

subspaceeddy commented
Issue summary: View changes

I ran in to the issue fago mentioned and can confirm that wodenx's patch in #2 cleanly fixes it

skwashd’s picture

skwashd commented
Status: Needs review » Closed (duplicate)
Related issues: +#2356305: Some contrib entities has no support for UUID, let's skip them.

This is a duplicate of #2356305: Some contrib entities has no support for UUID, let's skip them., which is currently RTBC. Let's run with that one.