Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Notification formcreator pour un formulaire en attente de validation #871

Closed
lolokai opened this issue Dec 29, 2017 · 21 comments
Closed

Notification formcreator pour un formulaire en attente de validation #871

lolokai opened this issue Dec 29, 2017 · 21 comments
Labels
Milestone

Comments

@lolokai
Copy link

lolokai commented Dec 29, 2017

Bonjour,

J'ai créé un formulaire en précisant un groupe comme valideur.
J'ai également créé une notification avec comme évènement "Formulaire en attente de validation" et comme destinataire "Valideur".
Néanmoins, lorsque le formulaire est créé aucune notification n'est envoyé aux membres du groupe (fichier mail.log non renseigné). Cependant, les personnes faisant partie de ce groupe ont bien le droit de valider ou non le formulaire lorsqu'elles sont connectés sur GLPI.
Alors que si je met un utilisateur simple comme valideur, celui-ci reçoit bien les notifications.

Version de FormCreator : 2.6.0
Version de GLPI : 9.2.1

@btry btry added this to the 2.7.0 milestone Dec 29, 2017
@btry
Copy link
Collaborator

btry commented Dec 29, 2017

Bonjour

A première vue, je pense que la notification échoue à convertir un groupe en une liste d'utilsaiteurs.

@lolokai
Copy link
Author

lolokai commented Dec 29, 2017

Petite précision : sur la version précédente de formcreator cela fonctionnait parfaitement bien.

@btry
Copy link
Collaborator

btry commented Dec 29, 2017

Pouvez-vous préciser la version exacte ?

@lolokai
Copy link
Author

lolokai commented Dec 29, 2017

Oui il s'agissait de la version 2.5.2

@lolokai
Copy link
Author

lolokai commented Dec 29, 2017

Avec GLPI 9.1.6

@btry
Copy link
Collaborator

btry commented Dec 29, 2017

Merci. Je regarderai après la release de la 2.6.1. Cela fait trop longtemps que je repousse sa release.

Vérifiez les logs SQL et PHP au cas où il y aurait une erreur en lien avec le bug.

@btry btry added the bug label Dec 29, 2017
@lolokai
Copy link
Author

lolokai commented Jan 2, 2018

Pour info : Pas d'informations ni dans les fichiers SQL / ni PHP

Bonne année ;)

@hiscorebob
Copy link

Bonjour,

J'ai exactement le même comportement décrit par lolokai. Je confirme cette regression introduit par la version 2.6.0
Cordialement,

@btry
Copy link
Collaborator

btry commented Jan 9, 2018

Bonjour

Quelle est al dernière version qui a fonctionné correctement pour vous ?

Ca peut m'aider à identifier le code fautif.

@lolokai
Copy link
Author

lolokai commented Jan 9, 2018

Comme indiqué : la version 2.5.2 fonctionnait bien

@btry
Copy link
Collaborator

btry commented Jan 9, 2018

Je pense que ce correctif fera l'affaire.

Pouvez-vous me faire un retour ?

@hiscorebob
Copy link

Le correctif fonctionne. Les notifications des formulaires en attente sont bien envoyés maintenant.

@btry
Copy link
Collaborator

btry commented Jan 9, 2018

Merci pour le retour. La correction sera disponble à la prochaine release.

@lolokai
Copy link
Author

lolokai commented Jan 9, 2018

@hiscorebob : tu utilises la version 2.6.0 ou la 2.6.1 ? Sur la 2.6.1 cela ne semble pas fonctionner

@hiscorebob
Copy link

J'ai fait la modification sur la 2.6.0

@lolokai
Copy link
Author

lolokai commented Jan 9, 2018

@btry tu peux jeter un oeil ?

@btry
Copy link
Collaborator

btry commented Jan 9, 2018

J'ai fait le correctif à partir de la version de développement de ce jour. C'est la version 2.6.1 + un premier patch pour la prochaine version.

Le patch doit fonctionner aussi bien pour 2.6.0 que 2.6.1, et en faisant une session de debug, l'erreur m'a paru bien claire, sans aucun doute possible.

@lolokai si vous avez les connaissances suffisantes en PHP, je vous invite à provoquer l'envoi de mails à un groupe et capturer le contenu de la variable $query dans la méthode concernée par le patch.

En la jouant avec un outil comme phpmyadmin vous pourrez voir le résultat de la requête. Une requête fautive va avoir une clause WHERE groups_id = 0.

@lolokai
Copy link
Author

lolokai commented Jan 9, 2018

Ok je jète un oeil et je vous tiens au courant.

Merci ;)

@btry
Copy link
Collaborator

btry commented Jan 9, 2018

Voilà un exemple de requête sans le patch

SELECT DISTINCT `glpi_users`.`id` AS users_id,
 `glpi_users`.`language` AS LANGUAGE
FROM `glpi_groups_users`
INNER JOIN `glpi_users` 
ON (`glpi_groups_users`.`users_id` = `glpi_users`.`id`)
INNER JOIN `glpi_profiles_users` 
ON (`glpi_profiles_users`.`users_id` = `glpi_users`.`id` AND (`glpi_profiles_users`.`entities_id` = '0'))
INNER JOIN `glpi_groups` 
ON (`glpi_groups_users`.`groups_id` = `glpi_groups`.`id`)
WHERE `glpi_groups_users`.`groups_id` = '0' AND `glpi_groups`.`is_notify`

@lolokai
Copy link
Author

lolokai commented Jan 9, 2018

Après revérification je confirme que ça fonctionne bien.

Sujet à clore donc ;)

@btry
Copy link
Collaborator

btry commented Jan 9, 2018

Parfait; merci pour le retour.

@btry btry closed this as completed Jan 9, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants