Pourquoi vous catche le checkright ? Vous pouvez juste laisser lnexception
se propager non ?
Le 20 avr. 2013 14:31, "Matthieu Guffroy" <notifications@github.com> a
écrit :

> In class/ServiceBase.class.php:
>
> > @@ -226,7 +261,27 @@ public function userAutocomplete($queryString) {
> >          return $return;
> >      }
> >
> > -    public function getServices() {
> > -        return Services::getServices();
> > +    /**
> > +    * Renvoie la liste des services pour lesquels l'utilisateur et l'application courante
> > +    * ont des droits pour y accéder
> > +    */
> > +    public function getEnabledServices() {
> > +        $services = Services::getServices();
> > +        $result = array();
> > +        foreach($services as $service) {
> > +            $this->service_name = $service;
> > +            try {
> > +                $this->checkRight();
> > +                $result[] = $service;
> > +            } catch(\Exception $e) { /* no right for this service */ }
>
> Si le service est bugué (et il l'est si le checkRight dessus ne marche
> pas) je préfère ne pas le renvoyer au client plutot que de lui afficher un
> bouton (dans le cas de scoobydoo) vers un module qui ne marchera pas...
>
> Mais effectivement, après on peut mettre un double catch, un pour le
> checkRight "refus d'accés normal" et un pour les Exception plus global qui
> la log, pour nous avertir qu'il y'a un problème quelque part.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/payutc/server/pull/137/files#r3884888>
> .
>