Forum GLPI-Project

fiscal

Member

Registered: 2006-02-07

Posts: 4

Notifications multiples vers un destinataire pour un meme evenement

Bonjour

Nous sommes en test avec glpi 0.78.1 sous linux Redhat EL5.
Pour des besoins internes nous avons créé un nouveau modele de notification : Enquete de satisfaction.
Nous avons aussi créé une nouvelle notification : Envoi Enquete de satisfaction, déclenchée par l'evenement : Ticket closed par exemple.
Le but est que le demandeur reçoive , en plus du ticket de fermeture , un autre mail l'invitant à repondre à une enquete.

Le probleme selon nous  se situe dans le fichier  glpi/inc/notificationevent.class.php au niveau de la fonction raiseEvent, pour laquelle il est impossible d'envoyer plusieurs notifications à un même destinataire.

Il est compréhensible de ne pas envoyer  plusieurs fois la meme notification à un destinataire donné (s'il a plusieurs rôles) ,mais nous ne voyons pas d'incovenient à ce qu'une personne recoive plusieurs notifications différentes.
Voici ce que nous proposons : 

diff -u  notificationevent.class.php.orig  notificationevent.class.php
--- notificationevent.class.php.orig    2011-01-25 16:45:22.000000000 +0100
+++ notificationevent.class.php 2011-01-25 17:22:05.000000000 +0100
@@ -65,14 +65,14 @@
 
       //If notifications are enabled in GLPI's configuration
       if ($CFG_GLPI["use_mailing"]) {
-         $email_processed = array();
-         $email_notprocessed = array();
 
          $notificationtarget = NotificationTarget::getInstance($item,$event,$options);
          $entity = $notificationtarget->getEntity();
          //Foreach notification
          foreach (Notification::getNotificationsByEventAndType($event, $item->getType(),
                                                                $entity) as $data) {
+            $email_processed = array();
+            $email_notprocessed = array(); 
             $targets = getAllDatasFromTable('glpi_notificationtargets',
                                             'notifications_id='.$data['id']);
 
@@ -117,13 +117,13 @@
                   }
                }
             }
+            unset($email_processed);
+            unset($email_notprocessed);
          }
       }
-      unset($email_processed);
-      unset($email_notprocessed);
       $template = null;
       return true;
    }
 
 }

remi

GLPI-DEV

From: Champagne

Registered: 2007-04-28

Posts: 7,127

Website

Re: Notifications multiples vers un destinataire pour un meme evenement

En version 0.80 (en cours de dév), l’enquête de satisfaction possède sa propre notification.

Ne pas envoyer 2 notifications, pour un même événement est volontaire.

+

---

Dév. Fedora 29 - PHP 5.6/7.0/7.1/7.2/7.3/7.4 - MariaDB 10.3 - GLPI master
Certifié ITILv3 - RPM pour Fedora, RHEL et CentOS sur https://blog.remirepo.net/

delbaz

Member

Registered: 2011-02-02

Posts: 1

Re: Notifications multiples vers un destinataire pour un meme evenement

Ne pas envoyer 2 notifications, pour un même événement est volontaire.

Bonjour,

pourquoi ?

Je suis assez d'accord avec fiscal.

Il est compréhensible de ne pas envoyer  plusieurs fois la même notification à un destinataire donné (s'il a plusieurs rôles) ,mais nous ne voyons pas d'inconvenient à ce qu'une personne reçoive plusieurs notifications différentes.

Pourquoi ne pas pouvoir envoyer 2 notifications de natures différentes sur un même évènement ?
Pourquoi ne pas laisser le gestionnaire faire ce qu'il veut ?

On peut préférer ne pas tout mettre dans le même mail