Bon... Il reste encore un petit bug avec la nouvelle fonction d'initialisation de Ghost Tracker : Lorsque la table #_user_lastvisit_check est contient déjà des éléments, on a une erreur Sql "duplicate key". Ce qui est quelque part normal si on tente de réinjecter l'id d'un user déjà présent.

Ce qui l'est moins, normal, c'est que ce cas de figure ne devrait pas se produire...

[Bugtracker] ticket 1226

Je corrige et publie dès que possible.

Bonsoir Eric ;

avec le plugin Event Cats, je crée un utilisateur en utilisant la fonction native de Piwigo register_user. Celle-ci active un trigger capturé par nbc UserAdvManager avec la fonction UserAdvManager_Adduser (dans le main.inc.php), laquelle utilise $_POST['password'].

J'ai eu une notice lors du test de la création d'utilisateurs, car initialement le champ dédié au mot de passe avait l'attribut name = "ec_in_up_psd_txt". Evidemment, cela est résolu en modifiant l'attribut par name = "password".

Vaut-il mieux considérer qu'il y a une sorte de standard de nommage des attributs, ou bien la fonction UserAdvManager_Adduser devrait plutôt tester si $_POST['password'] existe ?

Eric a écrit:

Salut Luc.

En fait de standard, je pense qu'il est bon d'utiliser les attributs générés par le Core de Piwigo.  (..)

C'est clair.

En fait après coup, après avoir écrit mon message, je me suis dit qu'il faut que la création d'un utilisateur se rapproche le plus possible de la création "native" par Piwigo, afin d'offrir une compatibilité maximum, au cas par exemple où d'autres plugins encore interviendraient dans les processus discutés.

Dans mon plugin, je crois que je fais le test isset($_POST['password']) (...)

Pas dans la fonction UserAdvManager_Adduser du main.inc.php en tous cas ;-) !

Je tâcherai de faire les vérifications et les tests qui s'imposent dans la soirée et je te tiens au courant.

Merci beaucoup ! En fait, je pense avoir finalement la réponse à ma question. Mais ça me paraissait intéressant de soulever le sujet.



D'une façon générale il faudra que je fasse attention à la compatibilité d'Event Cats avec nbc UserAdvManager.

Dernière modification par LucMorizur (2009-11-04 13:58:27)

Eric a écrit:

LucMorizur a écrit:

Eric a écrit:

Salut Luc (...)


Après vérification, c'est exact. Je ne contrôle pas $_POST['password'] dans UserAdvManager_Adduser pour la simple et bonne (?) raison que cela modifierait le fonctionnement natif de l'inscription à la galerie.

En effet, si je mets ce test en place à l'appel de cette fonction, le mot de passe deviendrait de facto obligatoire pour tout nouvel inscrit. Même si l'admin n'a pas paramétré le fonctionnement comme tel.

Tu crois ? Je ne vois pas tout-à-fait les choses comme ça, sachant que dans UserAdvManager_Adduser, $_POST['password'] est utilisé dans l'instruction suivante :

SendMail2User(1, $register_user['id'], $register_user['username'], $_POST['password'], $register_user['email'], true);

donc a priori pour envoyer un courriel au nouvel utilsateur qui vient juste de s'inscrire, je pense pour lui dire "votre identifiant est... ; votre mot de passe est...", pourquoi ne pas plutôt modifier légèrement cette instruction de la façon suivante :

$psd = (isset($_POST['password'])) ? $_POST['password'] : '';
SendMail2User(1, $register_user['id'], $register_user['username'], $psd, $register_user['email'], true);

de façon à envoyer une chaîne vide si $_POST['password'] n'est pas utilisé ?

Eric a écrit:

D'ailleurs, le code natif de Piwigo, que ce soit dans register.php ou function_user.inc.php (qui initie le trigger register_user), ne fait pas de contrôle sur l'entrée du mot de passe. Sauf le fait qu'il doivent correspondre à ce qui est saisie dans le champ de confirmation.

Effectivement.

Eric a écrit:

J'avoue ne pas connaitre le principe de fonctionnement de ton plugin Event Cats mais, dans ce contexte, je ne vois pas ce qui poserait pb entre nos deux plugins. Je vais installer Event Cats sur ma galerie de dev pour voir. Peut-être pourrais-je trouver une autre approche qui nous conviendrait à tous deux ?

A vrai dire pour le moment il me paraissait plus logique que Event Cats s'adapte à UserAdvManager que le contraire :-) . En tous cas tel que je l'imagine normalement Event Cats ne devrait pas être gêné par UserAdvManager. Sauf peut-être lorsque l'admin utilise la fonctionnalité de "groupe temporaire" de UserAdvManager, et encore. Mais c'est le principal point que je dois tester.

En tous cas pour le moment je ne vois pas d'adaptation particulière dont Event Cats aurait besoin.

Chaque fois que je livre sur SVN, j'essaie de faire en sorte que ce soit une version qui ne plante pas, même si évidemment toutes les fonctionnalités ne sont pas encore présentes. Tu peux donc essayer la dernière livraison, mais évidemment ce n'est pas complet. N'hésite pas à faire des commentaires, bien sûr ;-) !

Bonjour

Je viens d'installer Piwigo et je reste en admiration devant ce tres beau travail d'equipe .
Toutefois je rencontre un petit probleme avec le plugin nbc UserAdvManager : Sans lui, je peux mettre ou enlever le mail obligatoire à l'inscription, mais je dois gerer les validations à la main .
Dès que j'installe le plugin, le mail ne devient plus obligatoire .

Y'a t'il une solution, ou j'ai oblié quelque chose ?

Merci .

Ps : piwigo 2.05

Christophe

[Bugtracker] ticket 1229 corrigé dans la version 2.12.4 du plugin !

Bonjour le Forum

Il doit y avoir un truc bizarre car le Plugin n'est plus dans la liste : Spéciales -> Plugins ! ! ! ? ? ?

Ou je ne suis pas bien réveillée ! !

Non je confirme. Avec la dernière version.

Edit : Il est revenu après une nouvelle installation (C'est bête, les tables on étaient effacées)

Dernière modification par Patricia (2009-11-05 10:13:48)