Corriger la note sur les caractères autorisés (ie. - n'est pas permis)

[anonymous]

bonjour,

lorsque l'on crée un membre, on saisit son login. Les seuls caractères autorisés pour le login sont a-z 0-9 et -. Or si on met un login du nom : ----- , il prend pas en compte la validité du login.

soit on accepte un login avec -----

soit on dit pourquoi on refuse : "un caractére alphanumérique obligatoire"

c'est pas génant en soi mais une erreur et une erreur !!!

++

[anonymous]

bonjour,

il rejoint celui-ci #427

à voir donc

++

[mlutfy]

J'ai corrigé ceci dans le hg de Koumbit:

    // Vérification de la conformité du login + doit commencer par lettre ou chiffre (pas tiret)
    $login=strtolower($login);
    if (!ereg("^[a-z0-9][a-z0-9-]*$",$login)) { //$
      $err->raise("admin",10);
      return false;
    }

i.e. le regexp vérifie d'abord pour [a-z0-9], puis [a-z0-9-]. Notre qu'un login d'une lettre ou chiffre serait accepté. Je peux committer direct dans le trunk?

Je serais tenté à imposer un login qui commence par une lettre, et que le login ait au moins 3 caractères. Ça affecterait uniquement les nouveaux logins, donc pas d'impact sur les installations existantes.

Qu'en pensez-vous?

mathieu (bgm sur freenode)

[mlutfy]

Petit détail: le fichier à modifier est class/m_admin.php :-)

[anarcat]

(In [2256]) remove '-' from the list of authorized characters, it's too buggy for production.

Closes: #708