Closed (fixed)
Project:
Entity Construction Kit (ECK)
Version:
8.x-1.x-dev
Component:
User interface
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
29 Dec 2016 at 20:49 UTC
Updated:
25 Apr 2019 at 06:29 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
geerlingguy commentedComment #3
lahoosascoots commentedThis should do it. I'm not 100% sure why "title" was being unset for overrides in eck_entity_bundle_field_info_alter, but I got rid of it to make this patch to work. Hopefully the maintainers have some input here.
May just need a little bit of cleanup and some tests.
Comment #4
geerlingguy commentedComment #5
legolasboThe code looks good to me besides the comment below. However I would like to see tests for this functionality.
Doesn't the field definition have a label we can use for this?
Comment #6
legolasboForgot to set to needs work.
Comment #7
lahoosascoots commentedFixed the issue brought up about the field labels. Note that I stole that original code from EckEntityTypeFormBase if you want to update it there too.
Still needs tests.
Comment #8
lahoosascoots commentedComment #9
manuel.adanChanges:
Note that to translate the overridden labels via UI the core bug #2869124: Base field overrides cannot be translated which renders some entity form labels untranslatable must be fixed.
Waiting for some feedback before adding tests.
Comment #10
pifagor commentedComment #12
pifagor commented