Ticket #596 (new defect)

Opened 2 years ago

Last modified 2 years ago

Verifier les traductions

Reported by: nahuel Assigned to: anonymous
Priority: low Milestone: alternc-1.0
Component: Bureau: ergonomie Version: alternc-0.9.4
Severity: minor Keywords:
Cc:

Description

Il faudrait verifier les traductions pour la 1.0 que tout soit nickel.

Change History

04/05/06 16:19:57 changed by Gigii

  • version set to 0.9.4.
  • component changed from Autres to Bureau: ergonomie.
  • severity changed from block to minor.

je confirme .......... ça laisse à désirer

05/14/06 00:41:13 changed by anarcat

j'ai passé la journée à faire le ménage dans les traductions. la plupart des chaînes sont maintenant automatiques et les chaînes manquantes peuvent être détectées plus facilement.

il reste que j'aimerais bien voir les err_err_machin disparaître. je crois que ça devrait être fait pour la 1.0.

05/14/06 00:53:13 changed by anarcat

il y a beaucoup d'appels à raise() dans le bureau, surtout dans class/. ça devrait être notre cible:

anarcat@lenny$ grep -r raise . | grep -v '\.svn' | wc    alternc/alternc/bureau
    181     398    8904

m_err->log() pourrait aussi être amélioré. Il pourrait, par exemple, détecter automatiquement quelle fonction l'appelle avec db_backtrace. Il y a 103 appels à log(), aussi localisés dans class/:

anarcat@lenny$ grep -r 'log[:space:]*(' . | grep -v '\.svn' | grep -v 'locales' | wc
    103     221    6318

Le script de correction de raise devrait:

#!/usr/bin/php

bindgettextmachin();

foreach file {
  print preg_replace($line, '/raise[:space:]\("([^,]*)",(\d+)\)/e', 'raise("\1",'.__("err_err_".\2)));
}

ou quelquechose du genre. L'idée, ça serait de remplace tout raise("module", 1) par un raise("module", __("message")) et de se débarrasser de la traduction dans la définition de raise(). Ainsi, les chaînes sont détectées correctements, et même quand le système de traduction déconne, on a des messages d'erreur lisibles.

05/14/06 00:55:12 changed by anarcat

ouh. gaffe, il y a des horreurs comme:

./class/m_dom.php:      $err->raise("dom",3+$t);

Modifier aussi le regexp pour attrapper:

./class/m_dom.php:      $err->raise("dom",1,$dom);
./class/m_mail.php:      $err->raise("mail",7,$login."@".$domain);