Document that NULL is an allowed return value for EntityRepositoryInterface::getTranslationFromContext()

Is this a docs-only change, or do we want to have a followup adding a return typehint in a major release?

We could follow the steps along the lines of using the deprecation process for constructor and interface method signature changes → , although I haven't tested whether it would work for return typehints.

@Murz as the author of this ticket could you answer #7

Yes, this is a docs-only change to return the extended type hint, that explicitly describes that returning NULL is allowed and in which cases. Because except for the docs, returning NULL is possible in the EntityRepositoryInterface::getTranslationFromContext(), but the docs say that this is not allowed.

So, with this change, we can move forward with the language fallback logic to return NULL, if no fallback translations are allowed (in separate issues).

Opened https://www.drupal.org/project/drupal/issues/3388290 📌 language fallback logic to return NULL Active as the follow up, will need more info.

scratch that see we have 🐛 EntityRepository::getTranslationFromContext() function forcibly adds default entity language to fallback candidates Needs review

Committed to 11.x, 10.2.x, and 10.1.x as a patch-safe API documentation improvement. Thanks!

xjm → closed merge request !4662

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