Tried to run Commons in different languages.

The 2 Commons variables are currently not translatable:

commons_anonymous_welcome_title
commons_anonymous_welcome_body

CommentFileSizeAuthor
#10 variable_translation.png77.75 KBdevin carlson
#10 translation_interface.png85.13 KBdevin carlson
#10 declare-variables-1917922-10.patch1.57 KBdevin carlson
#4 1917922.commons_site_homepage.translate_default_variable_values.patch1.13 KBjpontani

Comments

slowflyer’s picture

slowflyer commented

Currently the variables seemed not to be used at all...

The text:
Welcome to our community
Share your thoughts, find answers to your questions.

Is hardcoded in a panel pane.

japerry’s picture

japerry
Location KOMK
commented
Status: Active » Postponed

Currently commons only supports english, and we bypass the language selector upon install. For multilingual support, we would have to do a more indepth audit of the whole installer, which has lots of hardcoded english content.

slowflyer’s picture

slowflyer commented

I think that's not an issue of the installer...
The installer can stay in english and produce an english version ...

But if yout start making Commons multilingual after installation, it should be easier to do...

f.e. writing the welcome text to a node instead of a non translatable panel pane, would make it more easy.

jpontani’s picture

jpontani commented
Status: Postponed » Needs review
StatusFileSize
new 1917922.commons_site_homepage.translate_default_variable_values.patch1.13 KB

Attached a patch that wraps the default values for variable_get in t().

slowflyer’s picture

slowflyer commented

The patch #4 seems not to solve the issue ...

  • t() function should be outside of variable_get() otherwise it doesnt help if the variables are defined
  • the variables are not used at the moment (#2)
jpontani’s picture

jpontani commented

The variables are used in the panels config, and are set upon installation.

ezra-g’s picture

ezra-g commented
Component: Code » Installer
ezra-g’s picture

ezra-g commented
Status: Needs review » Reviewed & tested by the community

As I revisit this issue it seems the ultimate solution would be for the homepage text to be a piece of featurized content with proper translation.

However, t() seems like an acceptable stopgap given our future plans to provide featurized content, such as demo text.

ezra-g’s picture

ezra-g commented
Title: Localize Commons variables » Localize Commons homepage text
devin carlson’s picture

devin carlson commented
Assigned: Unassigned » devin carlson
Status: Reviewed & tested by the community » Needs review
StatusFileSize
new declare-variables-1917922-10.patch1.57 KB
new translation_interface.png85.13 KB
new variable_translation.png77.75 KB

Variable translation can be taken care of by declaring variables to the Variable module (a dependency of the Internationalization module).

The attached patch declares all of the Commons Site Homepage variables.

variable_translation.png
translation_interface.png

devin carlson’s picture

devin carlson commented
Version: » 7.x-3.2
Status: Needs review » Reviewed & tested by the community

Tested #10 by setting up a simple English + French Commons site and attempting to translate the homepage text using variable translation.

devin carlson’s picture

devin carlson commented
Status: Reviewed & tested by the community » Fixed

Committed #10 to Commons Site Homepage 7.x-3.x.

http://drupalcode.org/project/commons_site_homepage.git/commit/d5233f1

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.