Some missing translations [#1199908]

Dear,

I've translated the Scheduler to PT-BR and I found some missing translations:

scheduler.module line 103 -> $output = '

' . t('Adjust the settings for the scheduler module.') . '

'; (not on .po file)

scheduler.module -> line 112 -> I change the line from . '

' to . '

'.t("or").'

' (now catching the translation)

scheduler.module lines 195, 204, 237 -> put the "t" variable to make the translations. Need to include these options at .po file

I attach the new patch here.

Regards

Comment	File	Size	Author
#8	1199908_8.scheduler.add_t_for_translations.d6.patch	1.58 KB	jonathan1055
#1	missing_translations-1199908-1.patch	1.75 KB	jaydub
	scheduler.patch	2.53 KB	pmichelazzo

Comments

Comment #1

jaydub commented 24 September 2011 at 01:12

Version:	master	» 7.x-1.x-dev
Status:	Active	» Needs review

Status	File	Size
new	missing_translations-1199908-1.patch	1.75 KB

Re-rolled the patch as a git patch and updated version for drupal 7 (can be backported to drupal 6 later if translation is missing there).

Comment #2

jonathan1055 commented 24 September 2011 at 08:56

Status:

Needs review

» Reviewed & tested by the community

Patch looks good and applies as expected, but I have not done any translations. Maybe pmichelazzo could use it and confirm?

Comment #3

pmichelazzo

Portuguese, Brazil

Brussels

commented 11 November 2011 at 05:43

Wow, sorry about my delay.

I test the patch and works fine. Great job!

Comment #4

jonathan1055 commented 12 April 2012 at 19:15

This patch will need a re-roll if #1526458: Lightweight Cron help page could use some improvements gets committed first.

Comment #5

rickmanelius commented 16 April 2013 at 01:38

Status:

Reviewed & tested by the community

» Needs work

Hi Everyone. This doesn't apply cleanly against the current dev branch because a few items have changed. If you do a quick scan and submit a new patch, I'll make sure it gets committed.