diff --git a/src/UnicaenZimbra/Entity/Grant.php b/src/UnicaenZimbra/Entity/Grant.php index 6d0970d0460ed0efa8c4e9097f97b8b681066aa4..7cf6084e45e5c4eeb0a50af9cfef3b57778b1e3a 100644 --- a/src/UnicaenZimbra/Entity/Grant.php +++ b/src/UnicaenZimbra/Entity/Grant.php @@ -131,11 +131,6 @@ class Grant if (! in_array($name, $this->publicFields) && 'changed' != $name) throw new Exception('La propriété '.$name.' est inconnue ou n\'est pas modifiable'); - /* Mise à jour de l'ID */ - if ('granteeId' == $name) $this->id = $value; - /* Mise à jour du nom */ - if ('granteeName' == $name) $this->name = $value; - $this->$name = $value; if ('changed' != $name) $this->changed = true; } diff --git a/src/UnicaenZimbra/Service/Folder.php b/src/UnicaenZimbra/Service/Folder.php index 4962db0879af7d2c44882667604ab92f110499a9..a7a1a1d809ef42f6bca1543d1ddd68127910fb84 100644 --- a/src/UnicaenZimbra/Service/Folder.php +++ b/src/UnicaenZimbra/Service/Folder.php @@ -318,16 +318,13 @@ class Folder extends Service { switch( $action ){ case self::GRANT_ACTION_SENDMAIL: /* Envoi d'un mail au destinataire pour le partage */ - if ($grant->granteeType != GrantEntity::TYPE_USR){ - throw new Exception('L\'envoi de mail de notification ne fonctionne que si le partage se fait à un utilisateur'); + if (!in_array($grant->granteeType, array(GrantEntity::TYPE_USR, GrantEntity::TYPE_GRP))) { + throw new Exception('L\'envoi de mail de notification ne fonctionne que si le partage se fait avec un utilisateur (GrantEntity::TYPE_USR) ou une liste de distribution (GrantEntity::TYPE_GRP)'); } $this->sendGrantNotification($id, $grant->granteeName); break; case self::GRANT_ACTION_LINK: - /* Ajout automatique du lien vers le dossier partagé sur le destinataire */ - if ($grant->granteeType != GrantEntity::TYPE_USR){ - throw new Exception('L\'envoi de mail de notification ne fonctionne que si le partage se fait à un utilisateur'); - } + if (empty($folder)) $folder = $this->get($id,'id'); $this->createMountPoint($id, $grant->granteeName, '/'.$folder->getName().' de '.$this->getAccount()->displayName); break; }