Make sure to add hreflang on localized pages.

[Nyholm]

This introduces a twig function hreflang that will print <link rel="alternate" hreflang="es" href="http://es.example.com/" /> on all pages that are localized.

I do also remove calls to deprecated functions.

[stof]

replacing deprecated calls should be sent in a separate PR rather than being mixed with a new feature. Replacing deprecated calls probably requires less discussions than a new feature, and mixing both makes it harder to review the new feature

[stof]

RequestStack is not supported on the Symfony 2.3 LTS

[Nyholm]

Sorry, you are correct. Should I get the request from the TwigEnvironment instead?

[acasademont]

Hi @Nyholm as @stof suggested would be better to only have a PR with the new feature. Can you rebase with master?

[Nyholm]

Thank you @acasademont for taking time to review this. I have now rebased the code.

What do you think about using the twig environment instead of injecting the RequestStack?

[XWB]

@Nyholm You need to fix the tests as well.

[Nyholm]

I've updated this PR. It supports SF2.3 and the tests are fixed.
@acasademont, is this ready to be merged?

[XWB]

@Nyholm Have you also checked the i18n_locales routing option? Imagine the following setup:

jms_i18n_routing:
    locales: [nl, en, fr]

not_french:
    path: /not-french
    options: { i18n_locales: [nl, en] }

This should ignore the fr locale and only generate <link rel="alternate" hreflang="nl" href="http://nl.example.com/" /> and ```.

[XWB]

You also need to add twig to composer.json.

[Nyholm]

Thank you @XWB for your feedback. I had not considered the scenario where you do not use all locales.

I have rewritten the PR according to your comments.

[picks44]

Hi @Nyholm @stof @XWB, how can we use this function? I see nothing in the doc... Also, is there a possibility to get all locale used in the config? SO we can use this in a controller to create a language switcher for instance, without having to duplicate all language entries...

[picks44]

Up ?