Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix AGS duplication #682

Merged
merged 3 commits into from
Feb 16, 2017
Merged

Fix AGS duplication #682

merged 3 commits into from
Feb 16, 2017

Conversation

laem
Copy link
Contributor

@laem laem commented Feb 15, 2017

Le taux était dupliqué sous deux noms différents, chomfg et ags, dans les fichiers cotsoc.xml et prelevements-sociaux.xml...

@benjello
Copy link
Member

Attention @laem, je ne suis pas sûr que l'on utilise ce qui se trouve dans prélèvement sociaux pour l'instant ...

@laem
Copy link
Contributor Author

laem commented Feb 15, 2017

OK. Dans ce cas là je mets à jour les deux en les gardant ?
(je voulais exploiter la machine pour faire les tests, il fait 30° chez nous 😁 )

@benjello
Copy link
Member

benjello commented Feb 15, 2017

Je pense que c'est plus raisonnable en attendant le grand soir du vrai nettoyage ;-)

laem added 3 commits February 15, 2017 16:07
Was already fixed in prelevements-sociaux, but this file is not used
@laem laem force-pushed the fix-ags-duplication branch from 8aacd58 to be6a17d Compare February 15, 2017 15:12
@laem laem requested a review from benjello February 15, 2017 15:33
@laem laem changed the title [WIP] Fix AGS duplication Fix AGS duplication Feb 15, 2017
@laem
Copy link
Contributor Author

laem commented Feb 16, 2017

@benjello GTM ?

@benjello
Copy link
Member

GTM @laem

@laem laem merged commit c0bea16 into master Feb 16, 2017
@laem laem deleted the fix-ags-duplication branch February 16, 2017 09:35
@pndsagora
Copy link

@laem pouvez-vous décrire plus en détail le problème présent sur la 10.0.1 et corrigé sur la 13.1.4 s'il vous plaît ?

@laem
Copy link
Contributor Author

laem commented Feb 16, 2017

@pndsagora La formule AGS, qui est une cotisation de la feuille de paie, fait référence au paramètre chomfg. Dans la correction 10.0.1, j'ai modifié la valeur du paramètre ags, alors que ce paramètre ags n'est lié à aucune formule.
Dans 13.1.4, j'ai modifié le bon paramètre.

@fpagnoux
Copy link
Member

@pndsagora on est dans le domaine des cotisations salariales, donc a priori pas d'impact sur les prestations sociales.

Par ailleurs, vous pouvez suivre #683 sur les discussions à venir sur le Changelog.

@pndsagora
Copy link

@laem & @fpagnoux , merci pour ce retour. (à la fois sur le sujet AGS et sur le contenu du changelog).
@fpagnoux, comment s'assurer que le "à priori pas d'impact" est bien aucun impact de façon certaine ?

@fpagnoux
Copy link
Member

Sur le sujet du changelog, n'hésitez pas à participer aux discussions qui ont/auront lieu sur l'issue, vous êtes les premiers concernés :).

@fpagnoux, comment s'assurer que le "à priori pas d'impact" est bien aucun impact de façon certaine ?

Question théorique difficile. Vu le périmètre du PNDS et celui de cette PR, je peux vous garantir à au moins 99% que cette PR n'aura aucun impact pour vous. Je n'ai en revanche pas d'outil me permettant de vous donner une preuve formelle de ce fait.

Ce genre d'analyses pourrait être dans le scope du package openfisca-parsers, qui permet d'analyser la législation codée pour en tirer des informations méta. On pourrait vérifier qu'il n'y a pas de dépendance entre le paramètre modifié ici et les variables calculées par le PNDS, mais aujourd'hui ces outils ne sont pas opérationnels (dis moi si je me trompe @cbenz )

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants