Re: [Glpi-dev] Proposition de correction CF pb d'att ribution des entités en fonction des groups

2007-06-16 Thread Walid Nouh
Ah MoYo a été plus rapide que moi ;)

Julien Dombre wrote:
 J'ai effectué une modification qui devrait tenir compte de toutes les 
 cas négatifs dans un cas multivalué.

 Merci du retour.

 Julien


 [EMAIL PROTECTED] a écrit :
   
 Proposition de patch

 a valider
   
   
 

 ___
 Glpi-dev mailing list
 Glpi-dev@gna.org
 https://mail.gna.org/listinfo/glpi-dev

 *
 Le contenu de ce courriel et ses eventuelles pièces jointes sont
 confidentiels. Ils s'adressent exclusivement à la personne destinataire.
 Si cet envoi ne vous est pas destiné, ou si vous l'avez reçu par erreur,
 et afin de ne pas violer le secret des correspondances, vous ne devez pas
 le transmettre à d'autres personnes ni le reproduire. Merci de le renvoyer
 à l'émetteur et de le détruire.

 Attention : L'Organisme de l'émetteur du message ne pourra être tenu 
 responsable de l'altération
 du présent courriel. Il appartient au destinataire de vérifier que les
 messages et pièces jointes reçus ne contiennent pas de virus.
 Les opinions contenues dans ce courriel et ses éventuelles pièces
 jointes sont celles de l'émetteur. Elles ne reflètent pas la position de 
 l'Organisme
 sauf s'il en est disposé autrement dans le présent courriel.
 **


   


___
Glpi-dev mailing list
Glpi-dev@gna.org
https://mail.gna.org/listinfo/glpi-dev


[Glpi-dev] Proposition de correction CF pb d'att ribution des entités en fonction des groups

2007-06-15 Thread mcjo79
Proposition de patch

a valider

fichier inc/rulesengine.class.php

691,692c691,703
   $res |=
matchRules($tmp,$criteria-fields[condition],$criteria-fields[pattern]);

---
   switch
($criteria-fields[condition]) {
   case PATTERN_IS_NOT :
   if
(matchRules($tmp,$criteria-fields[condition],$criteria-fields[pattern])) {
   $res =
1;
   } else {
   $res =
0;
   break 2;
   }
   break ;
   default :
   $res |=
matchRules($tmp,$criteria-fields[condition],$criteria-fields[pattern]);
   break ;
   }

Mais à mon avis, il faut aussi revoir les autres cas de type IS_NOT

Et peut-etre changer l'initialisation de
$res à true ou false en fonction des cas
ainsi que son affectation à soit

$res |=
matchRules($tmp,$criteria-fields[condition],$criteria-fields[pattern]);

soit

$res =
matchRules($tmp,$criteria-fields[condition],$criteria-fields[pattern]);


Rappelle des faits

J'ai deux pool d'utilisateur

A simple utilisateur membre du group societeA

et un groupe de responsable de service
membre du group societeA et responsable

Je crée deux règles non récursives
règle 1 :
Si group == societeA et group != responsable
alors
fixé l'entité à A
profile 'simpleUtilisateur'

règle 2 :

Si group == societeA et group == responsable
alors
fixé l'entité à A
profile 'responsable'


et mes simple utilisateur ont bien uniquement le profile simpleUtilisateur, mais
les responsables on les doubles profiles.
Le seul moyen que j'ai trouvé poru corriger ce problème est d'exclure
explicitement les responsables automatiquement avec une règle sur le (LDAP)uid
!= lulu, (LDAP)uid != fred etc...

Pourriez-vous m'indiquer quel fichier modifier pour que la gestion des règles
dynamiques vérifie bien tous les groupes utilisateurs


___
Glpi-dev mailing list
Glpi-dev@gna.org
https://mail.gna.org/listinfo/glpi-dev


Re: [Glpi-dev] Proposition de correction CF pb d'att ribution des entités en fonction des groups

2007-06-15 Thread Julien Dombre
J'ai effectué une modification qui devrait tenir compte de toutes les 
cas négatifs dans un cas multivalué.

Merci du retour.

Julien


[EMAIL PROTECTED] a écrit :
 Proposition de patch

 a valider
   
   

___
Glpi-dev mailing list
Glpi-dev@gna.org
https://mail.gna.org/listinfo/glpi-dev