[FAU-420] Add new "German language skills for international students" filter and allow sorting by it

[amiut]

Please check if the PR fulfills these requirements

• The commit message follows our guidelines
• Tests for the changes have been added (for bug fixes/features)
• Docs have been added/updated (for bug fixes/features)

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Feature

What is the current behavior? (You can also link to an open issue here)
https://inpsyde.atlassian.net/browse/FAU-420
No filter for required German language skills for degree programs

What is the new behavior (if this is a feature change)?

• Added new "German language skills for international students" filter
• Allow sorting by "German language skills for international students" (only if filter is applied)

Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)
No

Other information:
Related PR: RRZE-Webteam/FAU-Studium-Common#5

Screenshots of new filter / sorting column, Maybe we can discuss changing the layout slightly so it looks better.

[o-samaras]

Code-wise LGTM. Please fix the stylelint issues before merging this and also update the common package once it's merged so that Psalm is happy as well :) Thanks

[zhyian]

It looks good and functions well too! As Orestis mentioned, it would be good if the linter's issues were addressed. Thank you for your work on this.

[shvlv]

I think if we need to update the actual meta key here -

FAU-Studium-Embed/src/Infrastructure/Search/FilterablePostsMetaUpdater.php

Lines 72 to 114 in 5bbcc52

	/**
	* @psalm-param LanguageCodes $languageCode
	*/
	private function updateForLanguage(string $languageCode, DegreeProgramViewRaw $rawView): void
	{
	update_post_meta(
	$rawView->id()->asInt(),
	DegreeProgram::DEGREE . '_' . $languageCode,
	$rawView->degree()->name()->asString($languageCode)
	);
	if ($rawView->start()->offsetExists(0)) {
	update_post_meta(
	$rawView->id()->asInt(),
	DegreeProgram::START . '_' . $languageCode,
	$rawView->start()->offsetGet(0)->asString($languageCode),
	);
	}
	if ($rawView->location()->offsetExists(0)) {
	update_post_meta(
	$rawView->id()->asInt(),
	DegreeProgram::LOCATION . '_' . $languageCode,
	$rawView->location()->offsetGet(0)->asString($languageCode)
	);
	}
	$admissionRequirement = AdmissionRequirementsTranslated::fromAdmissionRequirements(
	$rawView->admissionRequirements(),
	$languageCode,
	)
	->mainLink();
	if (!$admissionRequirement instanceof AdmissionRequirementTranslated) {
	return;
	}
	update_post_meta(
	$rawView->id()->asInt(),
	DegreeProgram::ADMISSION_REQUIREMENTS . '_' . $languageCode,
	$admissionRequirement->name()
	);
	}

to make possible sorting by that meta key.

[amiut]

Nice catch 👌
It's definitely needed for consuming sites

[amiut]

Code-wise LGTM. Please fix the stylelint issues before merging this and also update the common package once it's merged so that Psalm is happy as well :) Thanks

Thanks done, there were some un-related issues, I added those to the baseline file so we can revisit them later.