Skip to content

Commit

Permalink
IBX-6260: Fixed preview voter taking language priority into considera…
Browse files Browse the repository at this point in the history
…tion (#865)
  • Loading branch information
ViniTou authored Aug 21, 2023
1 parent 3df4f90 commit 4523e6b
Showing 1 changed file with 6 additions and 19 deletions.
25 changes: 6 additions & 19 deletions src/lib/Siteaccess/AbstractSiteaccessPreviewVoter.php
Original file line number Diff line number Diff line change
Expand Up @@ -32,38 +32,25 @@ public function __construct(
*/
public function vote(SiteaccessPreviewVoterContext $context): bool
{
$siteaccess = $context->getSiteaccess();
$siteAccess = $context->getSiteaccess();
$location = $context->getLocation();
$languageCode = $context->getLanguageCode();
$contentLanguages = $context->getVersionInfo()->languageCodes;

if (empty(array_intersect($this->getRootLocationIds($siteaccess), $location->path))) {
if (empty(array_intersect($this->getRootLocationIds($siteAccess), $location->path))) {
return false;
}

if (!$this->validateRepositoryMatch($siteaccess)) {
if (!$this->validateRepositoryMatch($siteAccess)) {
return false;
}

$siteaccessLanguages = $this->configResolver->getParameter(
$siteAccessLanguages = $this->configResolver->getParameter(
'languages',
null,
$siteaccess
$siteAccess
);

if (!in_array($languageCode, $siteaccessLanguages, true)) {
return false;
}

$primarySiteaccessLanguage = reset($siteaccessLanguages);
if (
$languageCode !== $primarySiteaccessLanguage
&& in_array($primarySiteaccessLanguage, $contentLanguages)
) {
return false;
}

return true;
return in_array($languageCode, $siteAccessLanguages, true);
}

protected function validateRepositoryMatch(string $siteaccess): bool
Expand Down

0 comments on commit 4523e6b

Please sign in to comment.