developer.jelix.org is not used any more and exists only for history. Post new tickets on the Github account.
developer.jelix.org n'est plus utilisée, et existe uniquement pour son historique. Postez les nouveaux tickets sur le compte github.

Opened 13 years ago

Closed 13 years ago

Last modified 13 years ago

#221 closed bug (fixed)

utilisation du principe de comptes actifs/inactifs natif dans jAuth

Reported by: chris Owned by:
Priority: low Milestone: Jelix 1.0beta3
Component: jelix Version: 1.0 beta2.1
Severity: normal Keywords:
Cc: Blocked By:
Blocking: Documentation needed:
Hosting Provider: Php version:

Description

Alors que je suivais l'exemple de http://jelix.org/articles/manuel/events qui correspondait exactement à mon besoin, je me suis aperçu que tout est déjà en natif dans jAuth.

Voir lib/jelix/auth/jAuth.class.php, lib/jelix-modules/jauth/classes/jauth.listener.php et lib/jelix-modules/jauth/events.xml.

Pourtant, bien qu'ayant rajouté le champ actif (à 0 pour mon user) dans la dao, l'identification n'était pas interdite.

A la lecture un peu plus approfondie du code, je constate que si dans jAuth.class.php, je remplace le test $rep['canlogin'] === false par $rep['canlogin'] == false, alors ça fonctionne. Par contre, je n'ai pas trouvé pourquoi ce n'est pas false qui est reçu, mais juste un 0.

Change History (3)

comment:1 Changed 13 years ago by chris

Je crois que j'ai trouvé. Évidemment, il faut laisser le test complet ===, mais modifier :

$ok = $ok & ($user->password != );

en :

$ok = $ok && ($user->password != );

dans jauth.listener.php

comment:2 Changed 13 years ago by laurentj

  • Milestone set to Jelix 1.0beta3
  • Priority changed from normal to low
  • Resolution set to fixed
  • Status changed from new to closed

comment:3 Changed 13 years ago by laurentj

correction appliquée dans le trunk. merci :-)

Note: See TracTickets for help on using tickets.