It looks like some strings of the user interface cannot be translated, especially Tasks, Notes, Tickets etc. on the main dashboard.

CommentFileSizeAuthor
#7 storm.theme_.inc_.patch4.3 KBMagnity
#3 storm_dashboard_translate.patch414 bytesjurgenhaas
#2 Hardcopy.PNG45.7 KBFrankT
#2 storm_dashboard.de_.po2.81 KBFrankT

Comments

jurgenhaas’s picture

jurgenhaas
he/him
Primary language German
Location Gottmadingen
commented
Status: Active » Postponed (maintainer needs more info)

Not sure this is correct. I checked the code and all occurances are called through t(). Can you please be a bit more specific?

FrankT’s picture

FrankT commented
StatusFileSize
new storm_dashboard.de_.po2.81 KB
new Hardcopy.PNG45.7 KB

Attached is an hardcopy with the untranslatable strings marked. I also attached the german translation I created (and I'm using of course). You may commit the translation to CVS, by the way.

jurgenhaas’s picture

jurgenhaas
he/him
Primary language German
Location Gottmadingen
commented
Status: Postponed (maintainer needs more info) » Needs review
StatusFileSize
new storm_dashboard_translate.patch414 bytes

Oh sorry, you're right. Those strings a passed through variables and that's why the search didn't result in any matches. Please used attached patched and let us know if that works for you.

Don't think we need separate translation files for Storm Dashboard. All that's needed should be found in Storm already, I guess.

Magnity’s picture

Magnity commented

@FrankT,
There is already a de.po file in Storm - so perhaps you could patch against that if required? As @jurgenhaas says, the strings will probably be the same in both.

FrankT’s picture

FrankT commented

I applied the patch supplied in #3 successfully - but it did not help. Could you check the patch?

jurgenhaas’s picture

jurgenhaas
he/him
Primary language German
Location Gottmadingen
commented
Project: Dynamic Dashboard for Storm (Project Management) » Storm
Version: 6.x-1.0 » 6.x-1.27
Component: User interface » Storm.module

Looks like this is one for the core Storm module. The relevant strings come from theme_storm_link() in storm.theme.inc

I have assigned this issue to the Storm project.

Magnity’s picture

Magnity commented
StatusFileSize
new storm.theme_.inc_.patch4.3 KB

OK - I see areas that might be causing this.

Does the attached patch work?

Magnity’s picture

Magnity commented
Status: Needs review » Fixed

Confident that the patch in #7 will have fixed this issue.

Committed to D6 and D7 branches.

FrankT’s picture

FrankT commented

I can confirm that the latest patch solved the problem (I did not roll back the first patch supplied).

Status: Fixed » Closed (fixed)

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