#1875992: Add EntityFormDisplay objects for entity forms went in moving widget settings from instances to a new object, so this should be removed from the configuration schema.

CommentFileSizeAuthor
#6 2013891-6.patch2.99 KBswentel
#6 interdiff.txt829 bytesswentel
#2 2013891-2.patch2.18 KBswentel
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

swentel’s picture

swentel CreditAttribution: swentel commented
Issue tags: +Field API

Tagging

swentel’s picture

swentel CreditAttribution: swentel commented
Title: Remove widget config schema from text module » Remove widget config schema from text module and options module
Status: Active » Needs review
FileSize
2013891-2.patch2.18 KB

options contained them too ..

aspilicious’s picture

aspilicious CreditAttribution: aspilicious commented
Status: Needs review » Reviewed & tested by the community

Makes sense

swentel’s picture

swentel CreditAttribution: swentel commented
Status: Reviewed & tested by the community » Needs work

Nope, widget needs to be removed.

swentel’s picture

swentel CreditAttribution: swentel commented
Status: Needs work » Reviewed & tested by the community

URG NEVERMIND

swentel’s picture

swentel CreditAttribution: swentel commented
FileSize
interdiff.txt829 bytes
2013891-6.patch2.99 KB

Actually, missed a little bit from the original field schema.

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
CreditAttribution: alexpott commented

So this actually makes me realise we don't have any schema for entity.display.%.%.% as far as I can see... actually all the config entites created by the entity module are lacking schema :) needs an issue.... however is out-of-scope for this one :)

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
CreditAttribution: alexpott commented
Title: Remove widget config schema from text module and options module » Remove widget config schema from text, options and field modules
Status: Reviewed & tested by the community » Fixed

Committed 22a09e4 and pushed to 8.x. Thanks!

Automatically closed -- issue fixed for 2 weeks with no activity.