Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Set ACTIVE_LOCALE depending on the current page #39

Closed
wants to merge 1 commit into from

Conversation

mlocati
Copy link
Contributor

@mlocati mlocati commented Sep 19, 2013

THIS IS JUST A THINK-ABOUT PULL REQUEST

Let's implement the controller->warmUp method that sets the ACTIVE_LOCALE to the current page locale before the locale-dependent defines are defined.

Related core pull request: concretecms/concretecms#1375

Let's implement the controller->warmUp method that sets the
ACTIVE_LOCALE to the current page locale before the locale-dependent
defines are defined
@mlocati
Copy link
Contributor Author

mlocati commented Sep 20, 2013

An alternative to this pull request could be concretecms/concretecms#1381

@Remo
Copy link
Contributor

Remo commented Jul 1, 2015

@mlocati as far as I'm aware we only need to move the startup function to on_page_view and all existing i18n problems should be solved. It might be possible that one of our legacy add-on fixes another problem. Let me know if we still need this, going to close it till then.

@Remo Remo closed this Jul 1, 2015
@mlocati mlocati deleted the fix-localization-defines branch July 2, 2015 06:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants