search

wieni / wmsingles

Singles are node types used for one-off pages that have unique content requirements
MIT License
1 stars 1 forks source link

Use currentlanguage translation #6

Closed RobinHoutevelts closed 7 years ago

RobinHoutevelts commented 7 years ago

::getSingleByBundle() and ::getSingle() should return the translation based on the current interface language.

spoit commented 7 years ago

Begrijp ik da goed als $single->getTranslation($langcode); false geeft ge null returned? Maar wat met singles zonder vertaling dan? Moet dien if ni in 2 keer gebeuren zodat ge altijd minstens de node teruggeeft?

RobinHoutevelts commented 7 years ago

Als er geen single is in de huidige taal wil ik null.

Als een single niet vertaald is mag die niet getoond worden.

Op wmcontroller is geloof ik zelfs de Frontcontroller gepatcht dat als je naar een untranslated node surft ge uitkomt op de homepage. Dus ik denk wel dat we de "be brave, force translation" overal willen doortrekken?

Als ik geen null return creëer ik de rare situatie waar ge bvb een NL breadcrumb ziet op een FR pagina ( als die single ni translated is )

RobinHoutevelts commented 7 years ago

ContentEntityBase::getTranslation()

Is niet nodig:

if ($entity->language()->getId() === $langcode) {
   return $entity
}

Dit werkt ook:

return $entity->getTranslation($langcode); // Als $entity al in langcode is, wordt die gewoon returned.