Over in #2277945: Typed configuration implements TypedDataInterface improperly we changed TypedConfig to work with DataDefinition objects which means we can use it's methods in ElementInterface::getFormElement() implementations.

#1 2291721.1.patch2.53 KBalexpott
PASSED: [[SimpleTest]]: [PHP 5.4 MySQL] 73,219 pass(es). View
Members fund testing for the Drupal project. Drupal Association Learn more


alexpott’s picture

Status: Active » Needs review
2.53 KB
PASSED: [[SimpleTest]]: [PHP 5.4 MySQL] 73,219 pass(es). View
Gábor Hojtsy’s picture

Issue tags: +D8MI, +language-config

Are there other $definition['s in the config code we can get rid of? :)

alexpott’s picture

I had a look couldn't get anywhere else that the new DataDefinition objects methods can/should be used.

Gábor Hojtsy’s picture

Status: Needs review » Reviewed & tested by the community

Good with me then :)

Jose Reyero’s picture

catch’s picture

Status: Reviewed & tested by the community » Fixed

Committed/pushed to 8.x, thanks!

  • catch committed fce26fe on 8.x
    Issue #2291721 by alexpott: Use methods instead of array access in when...

Status: Fixed » Closed (fixed)

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