Developers of Sympa

[Olivier Salaun <address@concealed>]

John Douglass wrote:
> 
> I noticed a logic flaw when attempting to do the "alternate e-mail" in
> the preferences section.
> 
> Note the - and the +

>     ## Alt email is the same as main email address
> -    unless (if ($in{'new_alternative_email'} eq $param->{'user'}{'email'}) 
> {
> +    if ($in{'new_alternative_email'} eq $param->{'user'}{'email'}) {
>         &error_message('incorrect_email', {'email' => 
> $in{'new_alternative_email'}});
>         &do_log('notice', "do_record_email:incorrect email 
> %s",$in{'new_alternative_email'});
>         return 'pref';
>     }

You're absolutely right, we've applied your patch to CVS.

We were reported problems with alternative_email management that were due
to this bug.

Thanks.

--
Olivier Salaün
Comité Réseau des Universités