Problem/Motivation

Field config for EntityReferenceItem currently does not provide config dependencies to its configured target_bundles.

Proposed resolution

Add them.

Remaining tasks

User interface changes

API changes

Data model changes

CommentFileSizeAuthor
#2 entityreference-target-bundles-2571187-1.patch1.65 KBnedjo
#5 interdiff.txt2.41 KBnedjo
#5 entityreference-target-bundles-2571187-5.patch3.72 KBnedjo
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

nedjo created an issue. See original summary.

nedjo’s picture

nedjo
he/him/his
Primary language English
CreditAttribution: nedjo as a volunteer commented
Issue summary: View changes
Status: Active » Needs review
FileSize
entityreference-target-bundles-2571187-1.patch1.65 KB

Noticed this while reviewing #2567183: Re-export all built-in configuration in core.

Cloned the issue tags etc. from #2569741: [PP-1] FilterFormat should add the roles as config dependencies; they could use review.

Status: Needs review » Needs work

The last submitted patch, 2: entityreference-target-bundles-2571187-1.patch, failed testing.

The last submitted patch, 2: entityreference-target-bundles-2571187-1.patch, failed testing.

nedjo’s picture

nedjo
he/him/his
Primary language English
CreditAttribution: nedjo as a volunteer commented
Status: Needs work » Needs review
FileSize
entityreference-target-bundles-2571187-5.patch3.72 KB
interdiff.txt2.41 KB

Extend onDependencyRemoval for entity type bundle dependencies.

The last submitted patch, 5: entityreference-target-bundles-2571187-5.patch, failed testing.

The last submitted patch, 5: entityreference-target-bundles-2571187-5.patch, failed testing.

claudiu.cristea’s picture

claudiu.cristea
Primary language Romanian
Location Arad 🇷🇴
CreditAttribution: claudiu.cristea at Webikon commented
Status: Needs review » Closed (duplicate)

This is a duplicate of #1978714: Entity reference doesn't update its field settings when referenced entity bundles are deleted. We should fix the dependencies and reaction on removal there.