Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

NextEntityTypeConfig is not deleted when its correspond entity type is deleted. #282

Open
jrockowitz opened this issue Aug 22, 2022 · 2 comments
Assignees
Labels
area: next module bug Something isn't working

Comments

@jrockowitz
Copy link

Steps to reproduce

  • Create a plain vanilla standard instance of Drupal with the Article content type.
  • Install the Next.js module
  • Configure Next.js Entity Type for Content: Article (node.article) - /admin/config/services/next/entity-types/add
  • Delete the Article content type - /admin/structure/types/manage/article/delete
  • View 'Next.js entity types' page - /admin/config/services/next/entity-types
  • Confirm that Warning: Undefined array key "article" in Drupal\next\NextEntityTypeConfigListBuilder->buildRow() is displayed or logged.

Thoughts/Notes

  • The Next.js Entity Type config file has no dependencies listed.
  • Adding the '{entity_type_id}.{bundle}' as a dependency might fix the issue.
@shadcn
Copy link
Collaborator

shadcn commented Aug 24, 2022

Confirmed this is a bug. Thanks for the suggested fix. Looking into it.

Happy to have you here @jrockowitz :)

@shadcn shadcn added bug Something isn't working area: next module labels Aug 24, 2022
@shadcn shadcn self-assigned this Aug 24, 2022
@mglaman
Copy link
Contributor

mglaman commented Dec 7, 2023

I didn't see this when I opened #599 but I have a proposed fix in #620

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: next module bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants