search

MaVoix / groupes-locaux

Un formulaire permettant aux groupes locaux #MAVOIX de formaliser leur engagement à faire campagne.
The Unlicense
4 stars 1 forks source link

Ajouter trois champs supplémentaires dans le formulaire d'édition du groupe #52

Closed ColinMaudry closed 7 years ago

ColinMaudry commented 7 years ago

Afin de permettre aux mandataires d'ajuster au mieux le montant à collecter, nous souhaitons leur donner la possibilité de fixer eux-mêmes les lignes de dépense suivantes :

Les champs correspondants avec leurs valeurs par défaut doivent être ajoutés à la table group.

À réfléchir : comment gérer la mise à jour dynamique de amount_target quand un de ces 3 champs est modifié. Sachant qu'aujourd'hui amount_target est statique et suppose les valeurs par défaut.

ColinMaudry commented 7 years ago

Merci de commiter dans develop :slightly_smiling_face:

ColinMaudry commented 7 years ago

Tant que tu y es, il faut aussi ajouter une colonne emailing_expenses, DEFAULT=0, dans la table group. (fait dans 10a1526)

oceanBigOne commented 7 years ago

@ColinMaudry, C'est censé être bon de mon côté, trois choses : 1/ J'ai conservé getAmount_target en l'etat (qui renvoi la valeur en base, et ne DOIT pas bouger selon moi), puis j'ai crée getAmount_target_withExpenses (qui renvoi Amount_target + Expenses )

2/ Pas trop combien compris les correspondances à appliquée sur les pages pages/dons/participations.php et pages/dons/page.php (notemment accountantCost et bankingCost ? ... j'ai pas touché du coup, mais possible qu'il nous manque encore un champ ? :s)

3/ le champs que tu as crées est en base pour emailing_expenses est un INT, j'ai modifié en DOUBLE

oceanBigOne commented 7 years ago

@ColinMaudry par rapport à notre discussion sur les sommes de float en JS :

<script>
var a = 2.3;
var b = 2.4;
alert(a+b);
</script>

Affiche : 4.699999999999999

Je ne constate pas le problème en PHP, mais il me semble qu'il existe aussi (à niveau inférieur). Du coup, je pense qu'on fait bien de faire des arrondis, mais j'arrive pas à être sûr qu'on ai pas de problèmes si on fait un arrondi standard (et pas arrondi au nombre supérieur ou au nombre inférieur)