Issue #392 has been updated by Stéphane Baud.
Merci pour votre retour rapide. Bon, on va communiquer en Français, pourquoi se compliquer la life après tout ? Je comprends votre réponse, toutefois, c'est un peu inacceptable du point de vue de la pratique : si le choix est entre générer un warning (ce qui signifie en clair s'écarter des bonnes pratiques de coding) et passer en dur un paramêtre qui de toute façon correspond à la réalité (mode "manager" si target = Active Directory) alors le choix me semble clair non ? De toute façon, ce paramêtre n'a de sens que si AD = false, cela ne sert à rien - si ce n'est à compliquer votre code - de considérer que l'utilisateur puisse le configurer sur autre chose. En clair, soit AD = true auquel cas ce param est toujours considéré égal à "manager", soit AD = false auquel cas vous le prenez en compte. Cordialement, Stéphane Baud ---------------------------------------- Bug #392: self-service-password 0.7 (latest version) http://tools.lsc-project.org/issues/392 Author: Stéphane Baud Status: New Priority: Normal Assigned to: Category: Self Service Password Target version: self-service-password-? The change_password() call in resetbyquestions.php, is missing the last argument ($who_change_password) which triggers the following warning when configured for use with Active Directory : PHP Warning: Missing argument 9 for change_password(), called in /application/pwdreset/test/pages/resetbyquestions.php on line 158 and defined in /application/pwdreset/test/lib/functions.inc.php on line 195, referer: http://pwdreset-tst/index.php?action=resetbyquestions Simply adding the missing argument (tested locally) fixes the problem. -- You have received this notification because you have either subscribed to it, or are involved in it. To change your notification preferences, please click here: http://tools.lsc-project.org/my/account
_______________________________________________ ltb-dev mailing list [email protected] http://lists.ltb-project.org/listinfo/ltb-dev
