Remove Entity Reference UUID dependency

Open on Drupal.org →

Created on 14 February 2025, 2 months ago

Problem/Motivation

Currently we have a dependency on entity_reference_uuid which is only used on two base fields in grant entity (user, assignee). Because both fields are meant to be filled via code there is currently only one benefit I know which is in views as easy to use relation.
This dependency is curretnly the main blocker for D11 version of grant incl. the grant_base sub module which doen'nt need entity_reference_uuid but we can only set one composer dependency: 📌 Automated Drupal 11 compatibility fixes for entity_reference_uuid Needs review

Steps to reproduce

-

Proposed resolution

Change basefield definition to pure string also with 128 max char (as also used by entity_reference_uuid).
Create also for user data a computed field as already done for referenced entity (see related issue).
Remove info and composer dependencies.

Remaining tasks

change code

User interface changes

Sadly existing views configs which uses entity_reference_uuid based relation are breaking with WSOD.
Warning: Undefined array key "relationship" in Drupal\views\Plugin\views\HandlerBase->getEntityType() (line 800 of core/modules/views/src/Plugin/views/HandlerBase.php).
Warning: Trying to access array offset on value of type null in Drupal\views\Plugin\views\HandlerBase->getEntityType() (line 800 of core/modules/views/src/Plugin/views/HandlerBase.php).
The views configs needed to be updated e.g. by using new strategy. If something is missing please open additional issues.

API changes

-

Data model changes

No, the data is stored identically. Because of this we don't need an update hook.

📌 Task

Status

Active

Version

1.0

Component

Code

Created by

🇩🇪Germany c-logemann Frankfurt/M, Germany

Live updates comments and jobs are added and updated live.

Sign in to follow issues

Merge Requests

!6Remove Entity Reference UUID dependency
Merged
🇩🇪Germany c-logemann
updated 2 months ago

Comments & Activities

Issue created by @c-logemann
Comment 2 months ago →
🇩🇪Germany c-logemann Frankfurt/M, Germany
It seems that it's easy to define our own views relatioships. But also the field definition is not as easy to replace as I thought. I will try to get this fixed by an updated hook or this will be the next and hopefully last breaking change.
Merge request !6Issue #3506678 by c-logemann: Remove Entity Reference UUID dependency → (Merged) created by c-logemann
Comment 2 months ago →
🇩🇪Germany c-logemann Frankfurt/M, Germany
I created only normal relationships. If someone needs reverse relationships we can also implement this.
A new update hook is changing the field definitions.
Pipeline finished with Skipped
2 months ago
#424348
Comment 2 months ago →
System Message

c-logemann → committed 2e10c4c8 on 1.x
Issue #3506678: Remove Entity Reference UUID dependency
Comment 2 months ago →
🇩🇪Germany c-logemann Frankfurt/M, Germany
Comment 2 months ago →
🇩🇪Germany c-logemann Frankfurt/M, Germany
> If someone needs reverse relationships we can also implement this.

Just recognized that I need this by myself. So follow up is here: ✨ Create views reverse relationships Active
Comment 2 months ago →
🇩🇪Germany c-logemann Frankfurt/M, Germany
Comment about 2 months ago →
System Message
Automatically closed - issue fixed for 2 weeks with no activity.

contrib.social Blog FAQ Discussions

Production build 0.71.5 2024