Since jquery_ui_requirements() calls drupal_get_path(), which in turns calls drupal_get_filename(), which then looks at whether db_is_active(), and it is active, so it tries to do a SELECT on the system table. However, a system table is not yet installed, since Drupal just checks whether the default set of modules for the profile can be installed. So an 'install' phase of a profile should not have DB queries, because it will be called before any DB table is actually installed.

Attached relevant patch. Same issue would also apply to D6.

CommentFileSizeAuthor
jquery-ui-no-install.patch854 bytesgábor hojtsy

Comments

gábor hojtsy’s picture

gábor hojtsy
he/him
Primary language Hungarian
Location Hungary
commented
Status: Active » Needs work

Uhm actually one should return an array from here.

sun’s picture

sun
Primary language German
Location Karlsruhe
commented

This should already be fixed in CVS, no?

sun’s picture

sun
Primary language German
Location Karlsruhe
commented

oh, btw, since you're working on HEAD/core, it might be worth to also look at #454382: Provide jquery_ui_elements() for easier usage of plugins in other modules.

gábor hojtsy’s picture

gábor hojtsy
he/him
Primary language Hungarian
Location Hungary
commented
Status: Needs work » Closed (duplicate)

This was indeed fixed in #325831: url() in hook_requirements breaks install profiles, but we were not tracking had closely.