Hi,

I've attached the french translation of the latest version. Do you want to check it ? I've avoided committing it you gave me the go.

CommentFileSizeAuthor
#4 fr.po2.92 KBfgm
#2 fr.po3.03 KBfgm
fr.po3.03 KBfgm
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

salvis’s picture

salvis
he/his
CreditAttribution: salvis commented
Category: bug » feature
Status: Needs review » Needs work

Thank you!

This looks good, except for a few of the translations lost the trailing period. OTOH, the package name in the acl.info file must not have a trailing period.

Reviewing your file I was surprised to see the translations of the strings from hook_schema() in acl.install. I should have removed the calls to t() according to #332123: Remove t() from all schema descriptions. This means that the bulk of your translations aren't needed.

I'm sorry about making you do unnecessary work — I've updated the acl.install and acl.pot files as well as the existing translations in the -dev version (give it up to 12h to be repackaged).

fgm’s picture

fgm
Primary language French
Location Paris, France
CreditAttribution: fgm commented
Status: Needs work » Needs review
FileSize
fr.po3.03 KB

Here is the updated fr.po.

salvis’s picture

salvis
he/his
CreditAttribution: salvis commented
Status: Needs review » Needs work

The only change between the two versions is one removed space. Did you upload the wrong file?

fgm’s picture

fgm
Primary language French
Location Paris, France
CreditAttribution: fgm commented
Status: Needs work » Needs review
FileSize
fr.po2.92 KB

Ugh.... apparently, I had re-downloaded 6.1.2 instead of 6.1.dev :-(

Redone from 6.1.dev

salvis’s picture

salvis
he/his
CreditAttribution: salvis commented

Please add the period in

#: acl.admin.inc:90
msgid "Invalid user specified."
msgstr "Utilisateur spécifié invalide"

.

Should

#: acl.admin.inc:100
msgid "Current users"
msgstr "Utilisateurs existants"

be translated by "Utilisateurs actuels"? It's used in the list of current users of an ACL.

After that, feel free to commit. Thanks!

fgm’s picture

fgm
Primary language French
Location Paris, France
CreditAttribution: fgm commented
Status: Needs review » Fixed

Done.

I noticed de.po and sv.po still have the install strings, though.

salvis’s picture

salvis
he/his
CreditAttribution: salvis commented

Thanks!

Yes, the next time someone touches those files, the install strings should be dropped.

Status: Fixed » Closed (fixed)

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