If you install UUID and then install a module that implements an entity it would break UUID, as UUID didn't add the uuid schema to the newly installed module.

This patch fixes this by acting upon module install and then modify it's schema to add the required schema modifications.

CommentFileSizeAuthor
uuid-added_modules_installed_hook.patch1.07 KBlslinnet
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

lslinnet’s picture

lslinnet CreditAttribution: lslinnet commented
Assigned: lslinnet » Unassigned

Ups, didn't mean to assign it to my self.

dixon_’s picture

dixon_
he/him
Primary language English
Location France
CreditAttribution: dixon_ commented
Status: Needs review » Needs work

Good point! One minor issue though:

+++ b/uuid.install
@@ -35,6 +35,30 @@ function uuid_schema_alter(&$schema = array()) {
+    // get entity_info so we can get the
+    if (module_implements('entity_info', $module)) {

Please clarify the comments here a little bit.

lslinnet’s picture

lslinnet CreditAttribution: lslinnet commented

Can't seem to recall what I thinking to write there, will take a closer look at it doing the coming week.

lslinnet’s picture

lslinnet CreditAttribution: lslinnet commented

Have been pondering what I was thinking should go into that comment line but can't seem to recall, will update the patch on friday

dixon_’s picture

dixon_
he/him
Primary language English
Location France
CreditAttribution: dixon_ commented
Status: Needs work » Fixed

This should now be fixed in 7.x-1.x-dev and the alpha3 release that I'm rolling right now. Read the upcoming release announcement for more info.

Status: Fixed » Closed (fixed)

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