Currently the "Not applicable" language is not supported. Patch forthcoming...

CommentFileSizeAuthor
#3 2883475-3--language_not_applicable.patch1.12 KBdrunken monkey
#3 2883475-3--language_not_applicable--interdiff.txt751 bytesdrunken monkey
#2 search_api-not_applicable_langcode_support-2883475-2-D8.patch694 bytesmatthijs

Comments

Matthijs created an issue. See original summary.

matthijs’s picture

matthijs
Primary language Dutch
commented
Status: Active » Needs review
StatusFileSize
new search_api-not_applicable_langcode_support-2883475-2-D8.patch694 bytes
drunken monkey’s picture

drunken monkey
he/him
Primary language German
Location Vienna, Austria
commented
Component: General code » Plugins
StatusFileSize
new 2883475-3--language_not_applicable--interdiff.txt751 bytes
new 2883475-3--language_not_applicable.patch1.12 KB

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

borisson_
Primary language Dutch
Location Mechelen, 🇧🇪
commented
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

drunken monkey
he/him
Primary language German
Location Vienna, Austria
commented
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.