You are not logged in.

Announcement

 Téléchargez la dernière version stable de GLPI      -     Et vous, que pouvez vous faire pour le projet GLPI ? :  Contribuer
 Download last stable version of GLPI                      -     What can you do for GLPI ? :  Contribute

#1 2014-09-24 09:12:49

LaDenrée
HELPER
Registered: 2012-11-19
Posts: 3,193

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

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


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.5 (behaviours+DashBoard+fields+pdf)hebergé sur serveur mutualisé.

Offline

#2 2014-10-03 11:09:35

yllen
GLPI-DEV
From: Sillery (51)
Registered: 2008-01-14
Posts: 13,144

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)

Offline

#3 2014-10-03 11:35:53

LaDenrée
HELPER
Registered: 2012-11-19
Posts: 3,193

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.5 (behaviours+DashBoard+fields+pdf)hebergé sur serveur mutualisé.

Offline

#4 2014-10-03 13:15:54

yllen
GLPI-DEV
From: Sillery (51)
Registered: 2008-01-14
Posts: 13,144

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)

Offline

#5 2016-01-07 11:30:44

GUERIN Olivier
Guest
Registered: 2012-01-20
Posts: 146

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,

Offline

#6 2016-01-15 20:21:39

yllen
GLPI-DEV
From: Sillery (51)
Registered: 2008-01-14
Posts: 13,144

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)

Offline

Board footer

Powered by FluxBB