Jelix locales error when using jMailer

[laurentj]

In jMailer, there are bad arguments given to jException in the SetError? method.

And then this error is generated:

[exception 200] The given locale key "jelix~errors.mail.instantiate" is invalid (for charset UTF-8, lang UTF-8)

Instead of the real error.

[laurentj]

fixed in r1381

[foxmask]

I have this error with the trunk version :/ with the exactly same error message.

[foxmask]

je suis dsol je ne peux que t'apporter la solution sous cette forme, ne disposant pas de svn d'ici pour te fournir un patch

l'erreur est ligne 178 182 de jMailer.class.php :

            if ($arg !== null) {
                throw new jException($locale, $arg, $this->lang, $this->CharSet);
            }
            else
                throw new jException($locale, array(), $this->lang, $this->CharSet);

l'ordre des arguments n'est pas en correspondance avec la signature de la methode jException qui elle attend 5 paramtres et surtout le 3 ieme est le code ; non la langue

on devrait donc avoir minima :

            if ($arg !== null) {
                throw new jException($locale, $arg, 1, $this->lang, $this->CharSet);
            }
            else
                throw new jException($locale, array(), 1, $this->lang, $this->CharSet);

car la mthode est :

public function __construct($localekey, $localeParams = array(), $code = 1, $lang = null, $charset = null) {

cordialement.

[bballizlife]

Here is a first patch. Indeed the error code was missing. I set it to 1 for both exceptions but i guess we should give specific error codes.

Can't remember how we choose error codes ;)

[laurentj]

ok for me. Error code : I choose 1 because... 1 is good no ? :-)

patch should be applied on the trunk and 1.1.x

[bballizlife]

commited in trunk and branch 1.1.x in r1416