Column 'module' cannot be null [#2064489]

Looks like this is back again. Normal install of the latest -dev is giving this since the Panopoly rc5 update. When I test it via drush install-site, I get a bit more detail:

PDOException: SQLSTATE[23000]: Integrity constraint [error]
violation: 1048 Column 'module' cannot be null: INSERT INTO
{role_permission} (rid, permission, module) VALUES
(:db_insert_placeholder_0, :db_insert_placeholder_1,
:db_insert_placeholder_2); Array
(
[:db_insert_placeholder_0] => 3
[:db_insert_placeholder_1] => use text format panopoly_html_text
[:db_insert_placeholder_2] =>
)
in user_role_grant_permissions() (line 3076 of
/opt/development/test_install/drupal/modules/user/user.module).

So it looks like another one of these cases where a permission is being installed (in this case by defaultconfig in panopoly_wysiwyg) before the module itself is enabled.

Comments

Comment #1

mpotter CreditAttribution: mpotter commented 14 August 2013 at 13:31

Status:

Active

» Fixed

Looks like a problem in the defaultconfig module. Posted a patch here: #2008178: Fatal error when adding a permission that doesn't exist

Added this to the OA2 make file in commit c0e4c6d. Will give it some testing.

Comment #2

mpotter CreditAttribution: mpotter commented 14 August 2013 at 14:59

Status:

Fixed

» Needs work

Well, it's better. But now we get this:

Notice: Undefined index: download own files in user_role_grant_permissions() (line 3074 of /opt/clients/demo.openatrium.com/drupal/modules/user/user.module).

so it's time to investigate where "download own files" permission is being used.

Comment #3

mpotter CreditAttribution: mpotter commented 14 August 2013 at 15:22

Hmm, that might have been bogus from the installer. Using drush site-install gives this:

PDOException: SQLSTATE[23000]: Integrity constraint violation: 1048 Column 'rid' cannot be [error]
null: INSERT INTO {role_permission} (rid, permission, module) VALUES (:db_insert_placeholder_0,
:db_insert_placeholder_1, :db_insert_placeholder_2); Array
(
[:db_insert_placeholder_0] =>
[:db_insert_placeholder_1] => administer panelizer node panopoly_page breadcrumbs
[:db_insert_placeholder_2] => panelizer
)
in user_role_grant_permissions() (line 3076 of
/opt/clients/demo.openatrium.com/drupal/modules/user/user.module).

This is something the panopoly_pages module is doing: #2008762: Panopoly Pages has missing dependencies

Oh, and just noticed this is a bit different than the original post. In this case it's the 'rid' and not the 'module' field that is null. Since this is open over in panopoly_pages, I'm closing this issue again. Will work on a patch in the other issue queue.

Comment #4

mpotter CreditAttribution: mpotter commented 14 August 2013 at 15:22

Status:

Needs work

» Fixed

Comment #5

mpotter CreditAttribution: mpotter commented 14 August 2013 at 19:38

OK, #3 was being caused by #2043307: 'features.defaultconfig.inc' file never added during install -- what is 'group' key for? not applying when using an older version of git (make file was not specifying git as the download method and no branch was specified). Should be fixed in commit 1e03e75.