Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Hi there!
In postgresql on the lastest 3.4 release I am getting an error while running drush updatedb.
I am not really sure why I have a null entry in the pid column but this is the first time a hook_upate_n has failed for me with panelizer.
Performed update: panels_update_7304 [ok]
Performed update: panels_update_7305 [ok]
Performed update: panels_mini_update_7301 [ok]
Performed update: panels_update_7306 [ok]
Performed update: panelizer_update_7118 [ok]
Performed update: panelizer_update_7119 [ok]
SQLSTATE[23502]: Not null violation: 7 ERROR: null value in column "pid" violates not-null constraint [error]
DETAIL: Failing row contains (null, 713, pearson_first, page_title, page_title, 1, a:0:{}, a:5:{s:6:"markup";s:2:"h1";s:5:"class";s:0:"";s:2:"id";s:7:"wb-c..., a:0:{}, a:1:{s:8:"settings";N;},
a:0:{}, a:0:{}, 0, a:0:{}, 1a7cd846-433f-4901-9f4c-1ecef43cad4c).
Performed update: panelizer_update_7120
Comment | File | Size | Author |
---|---|---|---|
#7 | panelizer_update_7120-2788633-7.patch | 2.09 KB | sylus |
| |||
#6 | panelizer_update_7120-2788633-6.patch | 1006 bytes | sylus |
Comments
Comment #2
sylus CreditAttribution: sylus commentedComment #3
DamienMcKennaComment #4
DamienMcKennaIt seems like this is caused by panelizer_update_7120().
Comment #5
sylus CreditAttribution: sylus commentedI am just going to update the severity as it prevents me updating to latest security release fixes.
Comment #6
sylus CreditAttribution: sylus commentedWell I think I got passed that hook_update_n issue though not sure if is correct. ^_^
I did see similar code to what I have done earlier and the pid column in panels_schema does state:
Here is my current run through now with the patch it does seem to have recovered some panes though now running into a larger issue with panelizer_update_7300 that fatals my site so probably not a good patch.
Comment #7
sylus CreditAttribution: sylus commentedOk the second issue was there was no $default_names to delete as the following logic never got entered because there were no view_modes. Perhaps because I am using beans and related to https://www.drupal.org/node/2776173 but nonetheless the code below never get entered:
I think there should be logic encapsulating the db_delete logic in hook_update_7300 so added that in the most recent patch. I now get the following when running drush updatedb. ^_^
Comment #8
sylus CreditAttribution: sylus commentedJust lowering priority since patch allows me to complete update process ^_^
Comment #10
DamienMcKennaThanks for the patches! Committed.