search

afup / web

The current AFUP's website
38 stars 64 forks source link

Impossible de telecharger/recevoir par mail les factures #1058

Closed pocky closed 2 years ago

pocky commented 3 years ago

Bonjour l'équipe,

J'ai renouvelé la cotisation de Vanoix cette après-midi. J'ai ensuite voulu recevoir par mail la facture pour la comptabilité du mois prochain et j'ai deux 403 lors du téléchargement et l'envoi de facture par mail.

C'est tout pour cette issue :heart:

stakovicz commented 3 years ago

D'après mes premières recherche, c'est que tu n'as pas le rôle ROLE_COMPANY_MANAGER. Ça bloque dans la fonction isCurrentUserAllowedToReadInvoice()

stakovicz commented 3 years ago

Je vois une solution ça serait de modifier la fonction verifierDroitManagerPersonneMorale() pour l'autoriser quand il n'y a qu'une personne physique liée à la société.

stakovicz commented 3 years ago

@agallou tu en pense quoi ?

agallou commented 2 years ago

Le souci vient effectivement du rôle ROLE_COMPANY_MANAGER qui manquait à l'époque.

Après une discussion sur slack on avait corrigé le problème le 20 avril 2021 en ajoutant ce rôle, mais en se distant qu'il y avait tout de même un souci : sans ce rôle ces lignes de cotisation n'auraient pas dues être affichées (a noter que j'aurais du reporter les informations dans l'issue).

Pour moi le problème se règle plus en n'affichant les cotisations de la personne morale (attention, on peux avoir à la fois des cotisations de personnes morales et physique, il y a des cas à la marge comme celui-ci qui arrivent) que si on a le role ROLE_COMPANY_MANAGER (comme ça on aurait les mêmes conditions que pour le téléchargement/envoi par mail).

stakovicz commented 2 years ago

OK mais dans le cas de @pocky, comment il peut télécharger les factures pour sa société ?

agallou commented 2 years ago

@stakovicz dans son cas vu qu'on avait ajouté le rôle ROLE_COMPANY_MANAGER il arrive maintenant à télécharger les factures. Du coup il n'y a plus ce problème.

stakovicz commented 2 years ago

OK dons je vais simplement afficher les liens en fonction de ce rôle.

Je prépare ça.