Fix(rgcp/lodeom) : correction des erreurs de calcul avec régularisation annuelle

[liliced]

Résoud #3265

[netlify]

✅ Deploy Preview for nostalgic-mahavira-52b682 canceled.

Name	Link
🔨 Latest commit	971139d
🔍 Latest deploy log	https://app.netlify.com/sites/nostalgic-mahavira-52b682/deploys/67913a91a5a71600085afe91

[github-actions]

🚀 La branche est déployée !

• mon-entreprise : https://3325--mon-entreprise.netlify.app
• mycompanyinfrance : https://3325-en--mon-entreprise.netlify.app
• storybook : https://3325--mon-entreprise.netlify.app/dev/storybook/ (version EN)
• API : https://3325--mon-entreprise.netlify.app/api/v1/doc/

[JalilArfaoui]

Je préfère éviter les let qui peuvent être sources d’erreur…

Je suggère ici :

const SMICMensuel = engine.evaluate({

Puis dans le if (options.rémunérationETP) { on peut  :

return SMICMensuel * prorata +
			SMICHoraire *
				(options.heuresSupplémentaires + options.heuresComplémentaires)

et après le if :

return SMICMensuel

[liliced]

Ok ! J'ai préféré inverser les return en changeant le if en if (!options.rémunérationETP) {. De cette façon, le contenu du if est plus court. C'est plus lisible selon moi mais c'est personnel.

[JalilArfaoui]

En général, il est recommandé d’éviter les conditions négatives, mais on peut mettre tout le monde à l’aise très simplement :

const pleinTemps = !options.rémunérationETP;
if (pleinTemps) {

Ça a l’air tout bête comme ça, mais en vrai à la lecture ça évite une surcharge mentale qui peut induire en erreur.

[liliced]

J'ai créé une issue #3344 pour ne pas oublier 😉

[johangirod]

Petite question de logique métier :
Au lieu de partir du principe que le salaire brut renseigné est celui du mois incomplet, ne serait-il pas mieux que celui-ci soit toujours celui inscrit dans le contrat de travail. On pourrait ainsi calculer le la rémunération pour le mois incomplet grâce aux règles de la loi, plutôt que de le laisser à la charge de l'utilisateur. En demandant par exemple, le nombre de jours d'absence, ou alors le nombre de jours travaillés et la durée du mois.

Cela permettrait d'ailleurs de calculer le prorata du PSS, qui est basé sur le nombre d'heures.

[johangirod]

On peut écrire :

salarié . mois incomplet . prorata:
  valeur: rémunération de base mois incomplet / rémunération équivalente mois complet

[liliced]

Petite question de logique métier : Au lieu de partir du principe que le salaire brut renseigné est celui du mois incomplet, ne serait-il pas mieux que celui-ci soit toujours celui inscrit dans le contrat de travail. On pourrait ainsi calculer le la rémunération pour le mois incomplet grâce aux règles de la loi, plutôt que de le laisser à la charge de l'utilisateur. En demandant par exemple, le nombre de jours d'absence, ou alors le nombre de jours travaillés et la durée du mois.

Cela permettrait d'ailleurs de calculer le prorata du PSS, qui est basé sur le nombre d'heures.

@johangirod je ne suis pas confiante dans notre capacité à calculer le bon montant de cette façon.
Ne serait-ce que pour les heures supplémentaires, l'employeur peut leur appliquer une majoration différente du minimum légal.
Et je soupçonne qu'il existe d'autres éléments de rémunération qui ne soient pas triviaux à calculer.

De plus, le simulateur va peut-être être décomissioné finalement. Il me semble donc préférable d'éviter d'y travailler davantage (hors bugs bien sûr) tant que cette question n'aura pas été tranchée.