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.
Trying to upgrade my child distro to the latest version of Panopoly, gives me the following on BOA / Aegir.
Upgrading Executing panopoly_pages_update_7101
SQLSTATE[23000]: Integrity constraint violation: 1048 Column 'rid' cannot be null
Comment | File | Size | Author |
---|---|---|---|
#13 | panopoly_pages-integrity-constraint-error-2456635-13.patch | 860 bytes | johnrosswvsu |
#6 | panopoly_pages-sqlstate-23000-2456635-5.patch | 511 bytes | dsnopek |
Comments
Comment #1
candelas CreditAttribution: candelas commentedHello
Same here. I found #2296899-3: PDOException: SQLSTATE[23000]: Integrity constraint violation: 1048 Column 'module' cannot be null. Since I had Devel installed, I uninstalled but the error still happens.
The patch is #2293947-41: Make "Landing Page" into node + Panelizer (rather than a Page Manager page)
I was trying to update Open Atrium 2.32 to 2.33 that makes the update from Panopoly 7.x-1.18 to 7.x-1.19
I hope is solved soon because I can't update OA
Searching, I think this can be related #2183937: Error on install of Panopoly-based install profile: "Column 'module' cannot be null" when profile starts with "e"
Comment #2
candelas CreditAttribution: candelas commentedthe update that gives the error is panopoly_pages_update_7101
Comment #3
candelas CreditAttribution: candelas commentedI realize that the error comes because I have not an editor role, so its rid == NULL.
So I made a minimal patch where
changes to
I run the update and no problem now :)
I upload here that patch with the doubt that if I should upload it to the Panopoly Pages module instead than here...
Also I doubt in the module naming. In https://www.drupal.org/project/panopoly_pages/git-instructions they say the patch should be named git diff > [description]-[issue-number]-[comment-number].patch
but in https://www.drupal.org/node/707484 they say git diff > [project_name]-[short-description]-[issue-number]-[comment-number].patch
I choose the second, but please, tell to me if it is right :)
Comment #4
candelas CreditAttribution: candelas commenteddoubt solved:
I upload here that patch with the doubt that if I should upload it to the Panopoly Pages module instead than here...
I see that the Panopoly Pages issue cue directs to here :)
Comment #5
candelas CreditAttribution: candelas commentedComment #6
dsnopekHere is a much simplified version of this patch. Can you double check that this works for you as well? Thanks!
Comment #7
candelas CreditAttribution: candelas commented:)
Yes, it works. Thanks
Comment #8
dsnopekGreat! Thanks, @candelas, for finding the problem and writing the original patch. The patch name was fine, but I usually use underscores in the project name (so, 'panopoly_pages-" rather than "panopoly-pages-").
Committed!
Comment #10
candelas CreditAttribution: candelas commentedThanks for correcting to me :)
I just realize a couple of days that I was forgetting that (in the Drupal instructions they say it clear, but I am learning so many things... and two informations). Thanks for teaching.
I am very very happy to have being able to collaborate :)
Comment #12
boabjohn CreditAttribution: boabjohn commentedSorry guys, no intentiion to backtrack here, but:
I've just tried to go from Panopoly 1.21 to 1.23 and got the error @candelas reports in #1 :
1048 Column 'module' cannot be [error] null ...
Which is different to the 'rid' problem later described, or not?
In any event, after Dave's commit to dev 3 months ago, I'm wondering why the point release 1.23 from 12 days ago is causing this issue?
Since my 1.23 install is not fully updated now (update .._7101 is still outstanding), I'm not sure what to patch (or not?)
Comment #13
johnrosswvsu CreditAttribution: johnrosswvsu as a volunteer and at Promet Source commentedI have this for testing purposes only. Seems a check on the role object is needed.