1 (edited by LaDenrée 2014-09-24 16:15:02)

Topic: Probleme : édition des suivis Y/N sans effet

la classe ticketfollowup ne gère pas l'option "editer suivis (auteur)" correctement :
la fonction canUpdateItem ne gère pas le cas "non"
proposition de correction :(en gras)

je propose d'ajouter dans la fonction CanUpdateItem le code en gras ci dessous :

/**
    * Is the current user have right to update the current followup ?
    *
    * @return boolean
   **/
   function canUpdateItem() {

      if ($this->fields["users_id"]!=Session::getLoginUserID()
          && !Session::haveRight('update_followups',1)) {
         return false;
      }

      $ticket = new Ticket();
      if (!$ticket->can($this->getField('tickets_id'),'r')) {
         return false;
      }

      if ($this->fields["users_id"]===Session::getLoginUserID()
          && Session::haveRight('update_own_followups',1)) {
            return true;

      }
// return false If users can not update his followup nor all the followup;
     if ($this->fields["users_id"]===Session::getLoginUserID()
          && ! Session::haveRight('update_own_followups',1)
          && ! Session::haveRight('update_followups',1)) {
          return false;

      }
      // Only the technician
      return (Session::haveRight("update_followups","1")
              || $ticket->isUser(CommonITILObject::ASSIGN, Session::getLoginUserID())
              || (isset($_SESSION["glpigroups"])
                  && $ticket->haveAGroup(CommonITILObject::ASSIGN, $_SESSION['glpigroups'])));
   }

edit : le problème existait déjà en 0.83 et probablement même avant

Trouver la panne avant de réparer...
*GLPI 0.90.5+fusion90+1.0+behaviours0.90+reports+fields+appliances+pdf+badges+webservices PHP7.0 Mariadb10
*GLPI 9.1.3 (behaviours+DashBoard+fields+pdf)hebergé sur serveur mutualisé.

2

Re: Probleme : édition des suivis Y/N sans effet

Editer les suivis (Auteur) concernant surtout le demandeur du ticket.
Comme tu le vois dans le code, on considère qu'un technicien assigné au ticket ou membre du groupe assigné a le droit de modifier un suivi.

CentOS 6.5 - PHP 5.6 - PHP 7 - MySQL 5.6  + APC + memcached
GLPI 0.90.5 SVN - GLPI 9.1.3 SVN
Certifiée ITIL (ITV2F, ITILF, ITILOSA)

3

Re: Probleme : édition des suivis Y/N sans effet

j'ouvre la discussion avec ma vision (si j'ouvre la discussion c'est quelle est discutable ) :
Les suivis ont vocation à être notifiés au demandeur/observateur.
si on modifie un suivi, on n'a plus la trace de ce qui a été envoyé.
exemple (simplifié mais vécu) :
suivi N°1 : "on passera demain changer votre imprimante";notification au demandeur.
edit du suivi N°1 : " on passera la semaine prochaine" ; (pas de notification paramétrée pour modif suivi).

le surlendemain le demandeur appelle " je vous ai attendu hier ! ", réponse du centre d'appel : non on a planifié la semaine prochaine "  et embrouille avant qu'on ne comprenne ce qui s'est passé avec l'historique.--> "excusez nous"

voilà un des exemple qui fait que j'aimerais pourvoir bloquer complètement les suivis.
si je veux autoriser les tech, je peux autoriser le profil tech à modifier tous les suivis :  comme ils ne voient que leurs tickets ou les tickets de leur groupe, la possibilité existe(si je veux).

voilà ma vision du truc.

Trouver la panne avant de réparer...
*GLPI 0.90.5+fusion90+1.0+behaviours0.90+reports+fields+appliances+pdf+badges+webservices PHP7.0 Mariadb10
*GLPI 9.1.3 (behaviours+DashBoard+fields+pdf)hebergé sur serveur mutualisé.

4

Re: Probleme : édition des suivis Y/N sans effet

Effectivement, les points de vue sont discutables c'est ce qui fait avancer les projets...
Pour cela, j'ai ouvert un ticket sur la forge afin que tout le monde puisse voir ce sujet et donner son opinion : https://forge.indepnet.net/issues/5073

CentOS 6.5 - PHP 5.6 - PHP 7 - MySQL 5.6  + APC + memcached
GLPI 0.90.5 SVN - GLPI 9.1.3 SVN
Certifiée ITIL (ITV2F, ITILF, ITILOSA)

5

Re: Probleme : édition des suivis Y/N sans effet

Bonjour,

JE souhaitais savoir si ce sujet avait été tranché avec la version 0.90?

Merci,

6

Re: Probleme : édition des suivis Y/N sans effet

Non, personne n'a donné son avis.
Je viens de récréer cette demande sur la nouvelle forge : https://github.com/glpi-project/glpi/issues/379
n'hésitez pas à donner votre opinion

CentOS 6.5 - PHP 5.6 - PHP 7 - MySQL 5.6  + APC + memcached
GLPI 0.90.5 SVN - GLPI 9.1.3 SVN
Certifiée ITIL (ITV2F, ITILF, ITILOSA)