uuid_schema_alter() needs to live in uuid.module [#1584626]

I've been running into issues with drush updb and uuid, resulting in

Error: Call to undefined function uuid_schema_field_definition()

which is triggered by line 19 in the .install file.

The discussion at http://drupal.org/node/1309302 suggests that implementations of hook_schema_alter() should live in the .module file.

Comment	File	Size	Author
#3	1584626-move-schema-helper-to-install.patch	1.25 KB	dave reid
#1	move-uuid-schema-alter-1584626-1.patch	1.45 KB	mcjim

Comments

Status	File	Size
new	move-uuid-schema-alter-1584626-1.patch	1.45 KB

Comment #2

dave reid

he/him

English

Nebraska USA

commented 16 May 2012 at 13:00

Status:

Needs review

» Needs work

We're fixing the wrong problem here. The actual problem is that the uuid_schema_field_definition() function is defined in uuid.module. When a module's hook_install is run, it *cannot* assume that the .module file is loaded yet and must manually use drupal_load() to make the .module's files available.

Comment #3

dave reid

he/him

English

Nebraska USA

commented 16 May 2012 at 13:05

Status:

Needs work

» Needs review

Status	File	Size
new	1584626-move-schema-helper-to-install.patch	1.25 KB

Since the schema helper function is called from the .install file only, I think we need to move the uuid_schema_field_definition() function to uuid.install instead.