Changes:

  • Removed outer <p> tags from t() function to make translations cleaner.
  • Fixed some typos.
  • Enabled captcha_wrong_response_counter pluralization.
  • Generated new .pot template, got an error which I ignored: The first two watchdog() parameters should be literal strings. There should be no variables, concatenation, constants or even a t() call there.
    At watchdog('CAPTCHA',$errmsg,$errvar,WATCHDOG_ERROR) in image_captcha/image_captcha.user.inc on line 72.
  • Updated existing translations using new template.
  • Added Lithuanian translation.
CommentFileSizeAuthor
lt.po32.08 KBdomasj
captcha_translation.diff160.2 KBdomasj
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

soxofaan’s picture

Category: bug » task
Status: Needs review » Needs work

Thanks,
I already committed the code parts of you patch (http://drupal.org/cvs?commit=186986), except:

  • I did the captcha_wrong_response_counter pluralization with one format_plural() instead of a t() and a format_plural().
  • I removed all t()s from the database schema stuff in captcha.install because those strings do not have to be translatable (see Drupal core's modules)

I also regenerated the .pot template based on the new code: http://drupal.org/cvs?commit=187004

I added your Lithuanian translation: http://drupal.org/cvs?commit=187008

I did not yet update the existing (and your Lithuanian) translations to the updated .pot template.

soxofaan’s picture

Component: User interface » Translation
Status: Needs work » Fixed

Status: Fixed » Closed (fixed)

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