UUID handles dependencies by creating required entities before their parent entities.
This doesn't work for field_collection which requires the host entity to be present for field_collection_item entities to be saved properly.

Because of this there needs to be a way to alter the list of entities before they are processed. I propose adding an alter hook to uuid_entities_rebuild(). This will allow field_collection to remove any field_collection entities from the list to add them back later in hook_field_uuid_presave().

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

ciss’s picture

Patch against commit 4f00a9d (version 1.0-alpha5 + 18 commits, 2014-05-15).

ciss’s picture

Status: Active » Needs review
ciss’s picture

paulmartin84’s picture

Would be good to get this committed, its still applying cleanly to latest dev, have tested it with field collections and its working well. I am also using it with paragraphs module as well.

Arnaud Kali’s picture

Having trouble to save field collections with UUID enabled but this patch from #1 solved it!
+1 to get this committed as field_collection is now working fine together with patch from https://www.drupal.org/node/2074599#comment-7798611

ciss’s picture

Status: Needs review » Reviewed & tested by the community

Patch still applies. Setting to RTBC as there seem to be no objections.

ciss’s picture

@skwashd what is required to get this committed? Patch still applies to 7.x-1.x.

  • Manuel Garcia committed d78095b on 7.x-1.x authored by ciss
    Issue #2308983 by ciss: Add hook_uuid_entities_pre_rebuild_alter()
    
Manuel Garcia’s picture

Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.