- Issue created by @Anybody
- π©πͺGermany Anybody Porta Westfalica
Okay finally found it. The issue is, that currently the config is missing the entity type entirely! Just the bundle is added!
So if two entities have the same bundle name, they will conflict!
Example:
entity_extra_field.extra_field.entity_extra_field.glas_plate.boilerplate_body.yml
The bundle
glas_plate
exists both incommerce_product
andcommerce_product_variation
entities!The correct configuration name would be:
entity_extra_field.extra_field.entity_extra_field.commerce_product.glas_plate.boilerplate_body.yml
entity_extra_field.extra_field.entity_extra_field.commerce_product_variation.glas_plate.boilerplate_body.yml
So the bundle gets prefixed by the entity type!
Setting priority to major as this is a risky bug.
- Merge request !30Update EntityExtraField.php: Added getBaseEntityTypeId to use the entity type in the config id. β (Open) created by Anybody
- π©πͺGermany Anybody Porta Westfalica
Please check, if this is the correct approach. For non-bundleable entity types this will (correctly) use the entity type id twice.
- πΊπΈUnited States droath
@anybody, this change appears to be a duplication of https://www.drupal.org/project/entity_extra_field/issues/3440365 π Unable to edit existing extra field or render any public site pages Active . Can you confirm?
- Status changed to Postponed: needs info
3 months ago 11:49pm 29 January 2025 - π©πͺGermany Anybody Porta Westfalica
@droath yes, nearly.
Please note that
π Unable to edit existing extra field or render any public site pages Active replaces
$this->getEntityTypeId(),
while the MR here keeps it and adds$this->getBaseEntityTypeId()
.I think we should use this issue to check the result of π Unable to edit existing extra field or render any public site pages Active and see what it returns for bundleable and for non-bundleable entities to be sure it's totally correct yet.
My assumption is, that π Unable to edit existing extra field or render any public site pages Active does it correct, but we should check. Best would be to implement a kernel test?