Bonjour,
Pardonnez-moi, j'ai un peu de mal à lire cette ligne avec toutes
ces négations et je ne comprends pas votre ajout :
if ( !haveRight("show_ticket","1")
&& $job->fields["author"]!=$_SESSION["glpiID"]
&& $job->fields["assign"]!=$_SESSION["glpiID"]
&& ( !haveRight("show_group_ticket",1)
|| !in_array($job->fields["FK_group"],$_SESSION["glpigroups"])
)
)
Ce que je traduis par :
Si ( l'utilisateur ne peut pas voir tous les tickets
et l'utilisateur n'est pas l'auteur
et l'utilisateur n'a pas reçu la gestion du ticket
et ( l'utilisateur ne peut pas voir tous les tickets
ou l'utilisateur ne fait pas partie du groupe du ticket
)
)
alors print " "
sinon print "<a href..."
Il me semble que votre ajout est déjà contenu sur la première
ligne et, puisque les conditions sont séparées par des ET, cette
condition est redondante (et du coup une "fausse correction").
En écrivant le test à l'envers, on peut essayer d'y voir plus clair
(notez que je permute aussi, le "alors" et le "sinon") :
Si ( l'utilisateur peut voir tous les tickets
ou l'utilisateur est l'auteur
ou l'utilisateur a reçu la gestion du ticket
ou ( l'utilisateur peut voir tous les tickets
et l'utilisateur fait partie du groupe du ticket
)
)
alors print "<a href..."
sinon print " "
Ma modification mettait les 4 conditions au même niveau :
Si ( l'utilisateur ne peut pas voir tous les tickets
et l'utilisateur n'est pas l'auteur
et l'utilisateur n'a pas reçu la gestion du ticket
et l'utilisateur ne fait pas partie du groupe du ticket
)
alors print " "
sinon print "<a href..."
Ce qui, à l'envers s'écrit :
Si ( l'utilisateur peut voir tous les tickets
ou l'utilisateur est l'auteur
ou l'utilisateur a reçu la gestion du ticket
ou l'utilisateur fait partie du groupe du ticket
)
alors print "<a href..."
sinon print " "
Quel bug vouliez-vous corriger ? Pouvez-vous m'éclairer ?
Désolé de vous faire perdre du temps sur ce détail.
A bientôt,
--
Jean-Christophe Boggio -o)
[EMAIL PROTECTED] /\\
Independant Consultant and Developer _\_V
_______________________________________________
Glpi-dev mailing list
[email protected]
https://mail.gna.org/listinfo/glpi-dev
