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 2009-10-08 13:29:36

verlapinorix
Member
Registered: 2006-01-17
Posts: 7

Récursivité documents

Bonjour

glpi 0.72.21 /Redhat 5.2

Un document à l'entité racine en récursif (sous entité=oui)
Un utilisateur d'une sous-entité (uniquement s'il n'a pas un autre profil à la racine) se voit refuser l'accès au document.

Le problème vient de l'appel à la fonction haveAccessToEntity() dans front/document.send.php qui ne comporte pas de paramètre de récursivité (valeur pas defaut=false)

ligne 63 : remplacer
if ((haveRight("document","r") && haveAccessToEntity($doc->fields['FK_entities']))
par
if ((haveRight("document","r") && haveAccessToEntity($doc->fields['FK_entities'],true))

J'ai tout bon?

Offline

#2 2009-10-08 14:18:40

remi
GLPI-DEV
From: Champagne
Registered: 2007-04-28
Posts: 7,127
Website

Re: Récursivité documents

vi

et même plus simple  :
if ($doc->can($ID, 'r'))

+


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/

Offline

#3 2009-10-08 15:20:33

verlapinorix
Member
Registered: 2006-01-17
Posts: 7

Re: Récursivité documents

Merci, je me coucherai moins bête ce soir!

Offline

#4 2009-10-08 15:57:41

MoYo
GLPI - Lead
From: Poitiers
Registered: 2004-09-13
Posts: 14,513
Website

Re: Récursivité documents

en 0.72.21 il ne doit plus y avoir de haveRight sur les documents il me semble non ?


MoYo - Julien Dombre - Association INDEPNET
Contribute to GLPI :    Support     Contribute     References     Freshmeat

Offline

#5 2009-10-08 17:25:34

verlapinorix
Member
Registered: 2006-01-17
Posts: 7

Re: Récursivité documents

grep me dit qu'il en a 17 entre document.class.php et document.send.php

Offline

#6 2009-10-08 17:59:19

MoYo
GLPI - Lead
From: Poitiers
Registered: 2004-09-13
Posts: 14,513
Website

Re: Récursivité documents

en 0.72.21 je n'en ai que 9 chez moi... mais bon passons.


MoYo - Julien Dombre - Association INDEPNET
Contribute to GLPI :    Support     Contribute     References     Freshmeat

Offline

#7 2009-10-08 18:15:01

remi
GLPI-DEV
From: Champagne
Registered: 2007-04-28
Posts: 7,127
Website

Re: Récursivité documents

Normalement c'est corrigé
https://forge.indepnet.net/repositories … i?rev=9089

+

P.S. et je confirme qu'en 0.72, tous les contrôles de droit sur les documents passe par can/check (sauf celui-là)


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/

Offline

#8 2009-10-08 21:15:39

MoYo
GLPI - Lead
From: Poitiers
Registered: 2004-09-13
Posts: 14,513
Website

Re: Récursivité documents

je ferme


MoYo - Julien Dombre - Association INDEPNET
Contribute to GLPI :    Support     Contribute     References     Freshmeat

Offline

Board footer

Powered by FluxBB