Closed (fixed)
Project:
Group
Version:
4.0.x-dev
Component:
Code
Priority:
Normal
Category:
Task
Assigned:
Reporter:
Created:
27 Apr 2026 at 12:54 UTC
Updated:
11 May 2026 at 13:25 UTC
Jump to comment: Most recent
Not really a bug, but GroupRelationshipStorage::createForEntityInGroup() has:
// If the entity is config, we need to use the wrapper for it.
if ($group_relation_type->handlesConfigEntityType()) {
$storage = $this->entityTypeManager->getStorage('group_config_wrapper');
assert($storage instanceof ConfigWrapperStorageInterface);
$entity = $storage->wrapEntity($entity);
}
However, when we further down create the GroupRelationship and assign the target entity, it will always get wrapped regardless because of RelationshipEntityReferenceItem::setValue(). So let's remove the extra call to wrapEntity() and make the storage code easier to read.
Start within a Git clone of the project using the version control instructions.
Or, if you do not have SSH keys set up on git.drupalcode.org:
Comments
Comment #3
kristiaanvandeneyndeComment #5
kristiaanvandeneynde