Display machine name should match render array element

Created on 29 January 2025, 6 days ago

Problem/Motivation

Currently the display machine name in commerce_pvt_entity_extra_field_info() is commerce_pvt_field while the the render array element in commerce_pvt_commerce_product_view() is commerce_pvt.

This mismatch means the position of the field can't be set in the UI.

Steps to reproduce

Install Commerce, Commerce PVT etc. and add a product with variations
Install Field Layout (we're also using Field Group → )
In "Manage display" for the product, select a layout, (optionally add a field group) and position the Commerce PVT field in a column/region

The Commerce PVT field always appears outside the selected layout.

Proposed resolution

Set the display machine name and component name to the same as the render array element.

Remaining tasks

User interface changes

API changes

Data model changes

🐛 Bug report

Status

Active

Version

2.0

Component

Code

Created by

🇦🇺Australia imclean Tasmania

Live updates comments and jobs are added and updated live.

Merge Requests

!4Display machine name should match render array element
Merged
🇦🇺Australia imclean
updated 6 days ago

Comments & Activities

Issue created by @imclean
Merge request !4Set the display machine name to match the render array key. → (Merged) created by imclean
Comment 6 days ago →
🇦🇺Australia imclean Tasmania
Comment 6 days ago →
🇮🇹Italy eglaw Cittadella
Good call !
Merged, released 2.0.0-beta8.

contrib.social Blog FAQ Discussions

Production build 0.71.5 2024