I was about to try out your latest changes and forgot to enable config_builder_ui, and therefore form_builder was not automatically enabled. Config builder didn't appear in admin_menu so I flushed the cache and got the following error.

PDOException: SQLSTATE[42S02]: Base table or view not found: 1146 Table 'abc_drupal.form_builder_cache' doesn't exist: DELETE FROM {form_builder_cache} WHERE (type = :db_condition_placeholder_0) ; Array ( [:db_condition_placeholder_0] => config_builder ) in config_builder_flush_caches() (line 66 of /home/abc/public_html/sites/all/modules/config_builder/config_builder/config_builder.module).

I'm guessing config_builder needs to flush form_builder_cache regardless of whether config_builder_ui is enabled or not, so I've simply added a module_exists() call in config_builder_flush_caches(). Obviously you know the code better though, so maybe you can move this to config_builder_ui or move the form_builder dependency to config_builder.info.

CommentFileSizeAuthor
config_builder-fix-pdo-exception.patch709 bytesmstrelan
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Deciphered’s picture

Deciphered CreditAttribution: Deciphered commented

Good catch, that function should have been moved into config_builder_ui, will do so now.

Deciphered’s picture

Deciphered CreditAttribution: Deciphered commented
Status: Needs review » Fixed

Fix (plus a few other things committed).

Take into account that you'll likely want to re-run the last hook_update_N(), which will possibly throw errors if you've already run it, but I'm only really concerned about beta -> beta or beta -> stable upgrade paths, dev to dev would be too much.

Cheers,
Deciphered.

Status: Fixed » Closed (fixed)

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