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

Corrige doubles comptes revenus capital dans rsa asi aspa #1077

Merged

Conversation

bfabre01
Copy link

  • Correction d'un crash
  • Périodes concernées : toutes
  • Zones impactées:
    • openfisca_france\model\prestations\minima_sociaux\rsa.py
    • openfisca_france\model\prestations\minima_sociaux\asi_aspa.py
    • openfisca_france\model\revenus\capital\financier.py
  • Détails :
    • Supprime les doubles comptes de certains revenus du capital dans les bases ressources du RSA et de l'ASI-ASPA
    • Exemple : les revenus de f2ee étaient injectés deux fois via rsa_base_ressources_patrimoine_individu et via les variables revenus_capitaux_prelevement_bareme et revenus_capitaux_prelevement_liberatoire

Ces changements (effacez les lignes ne correspondant pas à votre cas) :

  • Corrigent ou améliorent un calcul déjà existant.

@bfabre01 bfabre01 requested a review from guillett August 16, 2018 15:11
@bfabre01 bfabre01 force-pushed the corrige-doubles-comptes-revenus-capital-dans-rsa-asi-aspa branch from ccc6978 to a0167b8 Compare August 17, 2018 11:29
@guillett guillett force-pushed the corrige-doubles-comptes-revenus-capital-dans-rsa-asi-aspa branch from 121bbae to 1d2230d Compare August 17, 2018 14:16
Copy link
Member

@guillett guillett left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Merci pour cette PR ! J'ai apporté des modifications au CHANGELOG. Je merge quand les tests sont passés.

@guillett guillett merged commit a2ed41e into master Aug 17, 2018
@guillett guillett deleted the corrige-doubles-comptes-revenus-capital-dans-rsa-asi-aspa branch August 17, 2018 14:30
@bfabre01
Copy link
Author

@guillett : merci pour le merge :-) !

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.

2 participants