Our usual return value for not found stuff is NULL, so ComplexDataInterface::getPropertyDefinition() and ContentEntityInterface::getFieldDefinition() should follow that behavior.

Postponed on #2002134: Move TypedData metadata introspection from data objects to definition objects to avoid conflicts.

CommentFileSizeAuthor
#1 return-null-2202401-1.patch1.97 KBBerdir
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Berdir’s picture

Berdir
Primary language German
Location Switzerland
CreditAttribution: Berdir commented
Status: Postponed » Needs review
FileSize
return-null-2202401-1.patch1.97 KB

should be as easy as this...

fago’s picture

fago
Primary language German
Location Vienna
CreditAttribution: fago commented
Status: Needs review » Reviewed & tested by the community

It is!

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
CreditAttribution: alexpott commented
Status: Reviewed & tested by the community » Fixed

Committed ea8bca5 and pushed to 8.x. Thanks!

  • Commit ea8bca5 on 8.x by alexpott: 
    Issue #2202401 by Berdir: Make getPropertyDefinition() return NULL...

Status: Fixed » Closed (fixed)

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