dachcom-digital/pimcore-seo

Respect Pimcore fallback languages

valentin- opened this issue · 4 comments

Q A
Bug report? no
Feature request? no
BC Break report? no
RFC? no

Hi, I'm currently working on a project with different regions/languages and fallback languages, f.e:

  • en
  • de-at -> fallback to de
  • de-de -> fallback to de

The fallback languages are useful if the same news object is used in different regions (Austria, Germany). Therefore only the title for de will be set (in the data object), and the value for de-at/de-de will be empty.

It would be handy if this feature also applies to the seo settings. Currently the value for de_AT/de_DE must be set, de won't get used as fallback.

image

What do you think about this feature?

Thanks in advance.

You're right, this should be possible!

@valentin- I just created a PR for that.

But I'm not sure if we should implement a flag to enable this feature, since this could be treated as a breaking change (new output).

Something like this maybe?

seo:
  meta_data_integrator:
    allow_locale_fallback: true # default false

We've just discussed this internally and have come to the conclusion that it does not need an additional configuration flag. This should be the standard behavior.

Fixed with #47