When i create content type and delete that content type full configuration content_access.settings object is missing. This cause failing of everything.

CommentFileSizeAuthor
#6 2868399-content_access-missing-config-6.patch1.84 KBid.tarzanych
#2 content_access-config_missing_after_node_type_delete-2868399-2.patch648 byteskmajzlik
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

karlos007 created an issue. See original summary.

kmajzlik’s picture

kmajzlik CreditAttribution: kmajzlik at Ciklum Western Europe for BurdaForward commented
FileSize
content_access-config_missing_after_node_type_delete-2868399-2.patch648 bytes

And here is my patch which seems to be working.

kmajzlik’s picture

kmajzlik CreditAttribution: kmajzlik at Ciklum Western Europe for BurdaForward commented
Status: Active » Needs review
kmajzlik’s picture

kmajzlik CreditAttribution: kmajzlik at Ciklum Western Europe for BurdaForward commented

After hours of thinking about bug... Maybe that can be issue for Drupal Core. Deleting unexisting item in config object should not cause deleting of whole object. But after 2 days spent on finding this i have no desire to work on that NOW.

albertski’s picture

albertski CreditAttribution: albertski at Xeno Media, Inc. commented

I also ran into this issue after removing a content type. I debugged it and found that the content_access.settings.yml file was blank. To fix I just reverted that file to prior the content type deletion and removed the content types gids from the list.

id.tarzanych’s picture

id.tarzanych CreditAttribution: id.tarzanych commented
FileSize
2868399-content_access-missing-config-6.patch1.84 KB

Not a Core error. It looks that wrong method was used.
Please review my patch

sathish.redcrackle’s picture

sathish.redcrackle CreditAttribution: sathish.redcrackle at Red Crackle commented
Status: Needs review » Reviewed & tested by the community

I tested and it works perfectly.

LpSolit’s picture

LpSolit CreditAttribution: LpSolit as a volunteer commented

Yes, the patch from comment 6 looks correct, per https://api.drupal.org/api/drupal/core%21lib%21Drupal%21Core%21Config%21...
Please commit asap.

kmajzlik’s picture

kmajzlik CreditAttribution: kmajzlik at Ciklum Western Europe for BurdaForward commented

+1 for commit.

fago’s picture

fago
Primary language German
Location Vienna
CreditAttribution: fago at drunomics commented
Status: Reviewed & tested by the community » Fixed

Thanks, committed.

  • fago committed cc2de75 on 8.x-1.x authored by id.tarzanych 
    Issue #2868399 by karlos007, id.tarzanych: content_access.settings...

Status: Fixed » Closed (fixed)

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