Tasks et notifications android 

Si on renvoie une exception c'est qu'on sait pas gérer la tâche, et qu'on
ne saura pas le faire avant la prochaine mise en prod donc autant la
détruire.

Il y a un duplicata de code tout beurk parcequ'il n'y a pas de
try/catch/finally en php5.4 (seulement en 5.5). À la limite je peux
declarer une fonction anonyme dans doOne et l'utiliser dans le catch et à
la fin.
Le 8 juin 2014 11:32, "Thomas Recouvreux" <notifications@github.com> a
écrit :

> In src/Payutc/Bom/Task.php:
>
> > +        $query = $qb->execute();
> > +
> > +        if ($query->rowCount() == 0) {
> > +            return false;
> > +        }
> > +
> > +        try {
> > +            $data = $query->fetch();
> > +            if ($data['tas_type'] == 'notification') {
> > +                Notification::send($data['tas_user'], $data['tas_message']);
> > +            } else {
> > +                throw new NotImplement("Task type : " . $data['tas_type']);
> > +            }
> > +        } catch (\Exception $e) {
> > +            $qdelete = $conn->createQueryBuilder()
> > +                ->delete('t_task_tas')
>
> Pourquoi la tache est détruite ? Il faudrait qu'elle soit retry dans le
> futur. Code dupliqué avec les lignes suivantes
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/payutc/server/pull/356/files#r13523461>.
>