-
Notifications
You must be signed in to change notification settings - Fork 101
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 l'application de l'abattement sur dividendes #1410
Conversation
6abc4cc
to
5e9a080
Compare
d76866a
to
e0713c5
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Petite suggestion de nommage.
openfisca_france/model/prelevements_obligatoires/impot_revenu/ir.py
Outdated
Show resolved
Hide resolved
@claireleroy : faut juste rebaser et update version et changelog et c'est bon pour moi. |
95ee9dc
to
4488a29
Compare
@benjello je ne sais jamais à quel niveau il faut updater le numéro de version. À ton avis, est-ce que cette PR est considérée comme un changement mineur ? |
@claireleroy : oui changement mineur car tu ne crées pas de nouvelle variable et tu corriges une erreur qui va affecter les résultats. |
@maukoquiroga @sandcha : un petit coup d'oeil avant que je merge ? |
rbg: 194000.0 | ||
rfr: 198000.0 | ||
rni: 194000.0 | ||
- name: abat_dividendes_mixtes |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@claireleroy Si tu peux ajouter une ligne, et enrichir la description avec la description de la pull request cela serait top !
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Par exemple :
- name: abat_dividendes_mixtes | |
- name: Impôt sur le revenu avant 2013 - Déclaration mixte : barème progressif et prélèvement forfaitaire libératoire |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
merci pour ton retour ! j'ai rajouté un nom plus explicite !
@@ -0,0 +1,79 @@ | |||
- name: abat_dividendes_only_bareme |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Par exemple :
- name: abat_dividendes_only_bareme | |
- name: Impôt sur le revenu avant 2013 - Déclaration avec option barème progressif et abattement de 40% |
3da33f3
to
fc96bc8
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM !
Contexte
Avant 2013, il existait deux systèmes d'imposition des dividendes à l'impôt sur le revenu : au barème progressif ou au prélèvement forfaitaire libératoire (PFL). Dans la déclaration de revenus, un contribuable doit déclarer ses dividendes en case 2DA si il a opté pour le PFL et en case 2DC si il a opté pour le barème. En pratique, il est possible de choisir une option "mixte", c'est-à-dire d'imposer une partie de ses dividendes à un des régimes d'imposition et une autre partie à l'autre régime. Cependant, si un contribuable opte pour une option "mixte", il perd le bénéfice de l'abattement de 40% applicable aux dividendes imposés au barème.
Bug constaté
Dans Openfisca, on simule bien l'abattement de 40% sur les dividendes pour les contribuables au barème mais on fait bénéficier de cet abattement également aux contribuables qui sont au régime "mixte" d'imposition, ce qui est incorrect.
Ce diagnostique est effectué en lançant 2 tests (voir script
tests/calculateur_impots/yaml/abat_dividendes.yaml
). Les résultats attendus sont obtenus à partir du simulateur en ligne de l'impôt de la DGFiP :=> RNI = 1AJ (200000) - abat frais pro plafonné (12000) + 2DC (10000) - 40% * 2DC (4000) = 194000; IR = 67942
=> RNI = 1AJ (200000) - abat frais pro plafonné (12000) + 2DC (10000) = 198000; IR = 69742
Solution proposée
model/prelevements_obligatoires/impot_revenu/ir.py
.Ces changements :