Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Matthijs created an issue. See original summary.

Matthijs’s picture

drunken monkey’s picture

Thanks for reporting, and for the patch!
However, I'd say we should then also add it further down, in getPartialItemIds()? See (and please test) the attached patch.

In any case, how would one even go about creating an entity with langcode zxx? Is that actually supported (through the UI)? Is it an actual use case?
But I'm not against adding this, it's a small enough change and shouldn't impact most users at all.

borisson_’s picture

Status: Needs review » Reviewed & tested by the community

We've added this in at least one project trough the UI, so it's a thing that some people do. Looks very small and should have no impact.

drunken monkey’s picture

Status: Reviewed & tested by the community » Fixed

Ah, OK, thanks a lot for the input!
In that case, even better of course to fix it. Committed.
Thanks again, everyone!

Status: Fixed » Closed (fixed)

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