Here's a first attempt at a patch to add UUID support.

CommentFileSizeAuthor
#4 1340984.patch3.91 KBJohn Morahan
#3 fpp-uuid.patch4.04 KBJohn Morahan
#2 fpp-uuid.patch3.73 KBJohn Morahan
fpp-uuid.patch3.61 KBJohn Morahan
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

John Morahan’s picture

John Morahan CreditAttribution: John Morahan commented
Status: Needs review » Needs work

Hmm, the update doesn't seem to work properly.

John Morahan’s picture

John Morahan CreditAttribution: John Morahan commented
Status: Needs work » Needs review
FileSize
fpp-uuid.patch3.73 KB

Or maybe it works just fine and I messed up testing it... anyway, this version is a bit cleaner.

John Morahan’s picture

John Morahan CreditAttribution: John Morahan commented
FileSize
fpp-uuid.patch4.04 KB

One more thing needed for entity_uuid_save() to work...

John Morahan’s picture

John Morahan CreditAttribution: John Morahan commented
FileSize
1340984.patch3.91 KB

New patch that just leaves the entity keys in there whether uuid is actually enabled or not.

John Morahan’s picture

John Morahan CreditAttribution: John Morahan commented
Status: Needs review » Fixed

This is committed.

Status: Fixed » Closed (fixed)

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

jhedstrom’s picture

jhedstrom
Primary language English
Location Portland, OR
CreditAttribution: jhedstrom commented

Was it intentional that this patch add an additional call to hook_entity_presave()? I'm seeing the double calls conflict with the title module (first call sets translated title appropriately, second call unsets). Added #2098041: Double call to hook_entity_presave conflicts with title module.

alcroito’s picture

alcroito CreditAttribution: alcroito commented
Issue summary: View changes

The double hook invocation is not correct behavior, see my comment and patch in #2098041: Double call to hook_entity_presave conflicts with title module.