(cleanup) Membership in months or by default

[trasher]

https://bugs.galette.eu/issues/1438

• Test reminders (have to wait for tests on Take care of reminders sent last period #373)

[codecov-commenter]

Codecov Report

Attention: Patch coverage is 76.92308% with 6 lines in your changes are missing coverage. Please review.

Project coverage is 48.14%. Comparing base (0007198) to head (d92a909).

Files	Patch %	Lines
galette/lib/Galette/Entity/ContributionsTypes.php	66.66%	3 Missing ⚠️
galette/lib/Galette/Entity/Contribution.php	87.50%	2 Missing ⚠️
.../Controllers/Crud/ContributionsTypesController.php	0.00%	1 Missing ⚠️

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files

@@            Coverage Diff             @@
##             develop     #498   +/-   ##
==========================================
  Coverage      48.14%   48.14%           
+ Complexity      6321     6317    -4     
==========================================
  Files            158      158           
  Lines          23790    23790           
==========================================
  Hits           11454    11454           
  Misses         12336    12336           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[manuelh78]

Désolé, j'ai posté dans le mauvais PR:)

Salut,
Je n'ai pas eu le temps de regarder.
J'imagine qu'il y a un test qui vérifie la date d'échéance juste après l'ajout d'une contribution ?

A la suite, il faudrait ajouter un nouveau type de contribution d'une durée spécifique, par exemple 3 mois
Puis ajouter une contribution de ce type
puis contrôler que la date d'échéance a été augmentée de 3 mois

Normalement, année civile ou pas, ça ne devrait pas avoir d'impact, puisqu'en passant en mois, il n'y a plus vraiment de notion "d'année".

Manu

[trasher]

Concernant les rappels, @gagnieray posait la question suivante :

Mais dans le cas d'une extension d'adhésion courte (1 mois principalement), n'y a-t-il pas un risque de "flooder" les adhérents de rappels à peine leur adhésion enregistrée (il est normalement censé pouvoir y avoir un rappel à 30 jours puis un autre à 7 jours de la date d'échéance) 🤔

Vu qu'il n'existe aucun moyen de savoir si l'échéance est déterminée par une contribution "standard" ou non - les rappels seront toujours calculés de la même manière - donc oui, les adhérents ayant renouvelé pour un mois vont se prendre un rappel dès le départ.

Dès lors, les solutions sont :

• ne pas utiliser les rappels si on a recours aux contributions "courtes", ce qui fonctionne si l'on utilise pas du tout les cotisations "standard" (mais rien ne l'interdit),
• faire en sorte d'enregistrer quelque part que la date d'échéance est calculée depuis une contribution "courte", et ne pas inclure ces adhérents dans les rappels,
• revoir ce système différemment (là, je passe mon tour).

La première solution est la plus simple à mettre en oeuvre, puisque rien ne doit être modifié - ça marche donc parfaitement à tous les coups :D

La seconde solution requiert certainement l'ajout d'un champ dans la table des adhérents, et la modification du système des rappels pour le prendre en compte (après merge de la PR #373).
Bien que ça nécessite d'encore y passer du temps, c'est peut-être un bon compromis.

Mais cette solution ne serait pas non plus universelle : pour des contributions de 6 mois par exemple, les rappels standards pourraient être légitimement utilisés...
L'on pourrait considérer que si l'extension d'adhésion dépasse x fois le premier rappel programmé, on active les rappels quand même - ce qui complique encore les choses (dans le code, et pour comprendre le système).

Je ne vois pas d'autres solutions pour le moment, si vous avez des idées je suis preneur.

[trasher]

La problématique des rappels sera traitée dans un second temps ; je voudrai avancer sur la release de la version 1.1.0 - j'ai créé un ticket pour ça : https://bugs.galette.eu/issues/1824