Normalizer can't detect a field type extending entity reference

Open on Drupal.org →

Created on 9 April 2024, about 1 year ago

Problem/Motivation

A custom field type, extending entity reference is not correctly detected in RdfSyncNormalizer because the is_a() check parameters are inverted.
For such a custom field, which might have other custom columns (e.g., a description additional column), we have to strengthen the check to make sure it only applies to the `target_id` column

Proposed resolution

Fix the is_a() check
Check also for $columnName === 'target_id'

Remaining tasks

None.

User interface changes

N/A

API changes

N/A

Data model changes

N/A

🐛 Bug report

Status

Closed: duplicate

Version

1.0

Component

Code

Created by

🇷🇴Romania claudiu.cristea Arad 🇷🇴

Live updates comments and jobs are added and updated live.

Sign in to follow issues

Merge Requests

!13Normalizer can't detect a field type extending entity reference
Open
🇷🇴Romania claudiu.cristea
updated about 1 year ago

Comments & Activities

Issue created by @claudiu.cristea
Merge request !13Fix ER check → (Open) created by claudiu.cristea
Comment about 1 year ago →
🇷🇴Romania claudiu.cristea Arad 🇷🇴
Status changed to Needs review about 1 year ago3:45pm 9 April 2024
Comment about 1 year ago →
🇷🇴Romania claudiu.cristea Arad 🇷🇴
Status changed to Closed: duplicate about 1 year ago4:55pm 9 April 2024
Comment about 1 year ago →
🇷🇴Romania claudiu.cristea Arad 🇷🇴
Moved the changes to 🐛 Wrong normalization of multiple entity references Needs review . Closing as duplicate.

contrib.social Blog FAQ Discussions

Production build 0.71.5 2024