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

reduction_ss_condition_revenus est elle une réduction ? #1204

Closed
adrienpacifico opened this issue Nov 6, 2018 · 7 comments
Closed

reduction_ss_condition_revenus est elle une réduction ? #1204

adrienpacifico opened this issue Nov 6, 2018 · 7 comments

Comments

@adrienpacifico
Copy link
Contributor

adrienpacifico commented Nov 6, 2018

Une question qui concerne peut être principalement @benjello et les autres économistes.

L'impôt sur le revenu fonctionne dans cet ordre là pour l'instant:

ir_plaf_qf --> ip_net (Impôt net avant réductions) -->
iaidrdi (Impôt après imputation des réductions d'impôt).

reduction_ss_condition_revenus introduit en 2016 est à priori une réduction, pourquoi l'avoir mis dans la formule d'ip_net et non pas dans iaidrdi ?

Ps: Je pense qu'il est possible de factoriser et d'améliorer la lisibilité des réductions d'impôt, là c'est assez ardu à lire.

@benjello
Copy link
Member

benjello commented Nov 6, 2018

Bonne question @adrienpacifico. Faut demander à la personne qui l'a mis en oeuvre.
Je ne serais pas contre la modification que tu proposes si elle correspond à la feuille de calcul de la DFIP.
Et en passant, il faudrait aussi renommer ir_plaf_qf, ip_net et iaidrdi avec des noms longs afin d'éviter ces confusions.

@claireleroy
Copy link
Contributor

@adrienpacifico @benjello C'est moi qui ais mis en place cette réduction :)

Malgré son nom, cette réduction d'impôt sous condition de revenus n'est pas franchement similaire aux autres réductions d'impôt. Elle s'apparente plus à une décote bis. Dans la feuille de calcul de la DGFiP elle intervient bien juste après l'application de la décote dans la partie 7 "Diminution de l'impôt" et non pas dans la partie 8 "Déduisez vos réductions".

@benjello
Copy link
Member

benjello commented Nov 9, 2018 via email

@Morendil
Copy link
Contributor

Morendil commented Nov 9, 2018

@benjello @adrienpacifico oui sur le refactoring des réductions j'avais très envie d'inscrire ça dans nos priorités, ça vous dit de programmer un atelier à ce sujet ?

@benjello
Copy link
Member

benjello commented Nov 9, 2018

Je vais être un peu chargé pour les prochains mois et je ne me vois pas investir autant de temps si tôt.
Désolé. Mais peut-être que @claireleroy est moins chargée ;-)

@Morendil
Copy link
Contributor

Morendil commented Nov 16, 2018

Pas de souci. Je pense qu'on peut simplifier la formule de ip_net qui est inutilement dupliquée, dans la mesure où la formule de reduction_ss_condition_revenusest effectivement datée. Je pousse une branche en ce sens.

Pour les renommages ça me semble important qu'on en discute à plusieurs, plutôt que de les faire de façon unilatérale ou dans des PR liées à d'autres sujets, car ce seront nécessairement des "breaking changes". Il me semble qu'ils nécessitent plus de consensus et de formalisme que des modifications d'implémentation de formules. Au minimum une issue de type RFC pour proposer un jeu de renommages cohérents sur un sujet donné, qui permettrait de recenser les consommateurs des variables concernées, de s'assurer que ça ne casse pas une appli qui en dépend, et d'apporter un peu plus de cohérence au passage.

Qu'en pensez-vous ?

@benjello
Copy link
Member

OK. Je ne suis pas contre un commentaire succint au bout de la ligne contenant reduction_ss_condition_revenus

Merci @Morendil !

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

No branches or pull requests

4 participants