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

Ipp 13 : Update des crédits/réductions d'impôt restants #919

Merged
merged 9 commits into from
Apr 6, 2018

Conversation

claireleroy
Copy link
Contributor

Les modifications de cette PR sont comparées à la branche ipp-update-calcul-ip-net (#918)

  • Evolution du système socio-fiscal
  • Périodes concernées : toutes
  • openfisca_france/model/prelevements_obligatoires/impot_revenu/credits_impot.py
  • openfisca_france/model/prelevements_obligatoires/impot_revenu/reductions_impot.py
    - openfisca_france/model/revenus/autres.py
  • Détails :
    • Les avantages fiscaux pour mécénat d'entreprises (depuis 2003) et pour acquisition de biens culturels (depuis 2002) sont définies comme des réductions d'impôt et non plus des crédits d'impôt
    • Mise à jour de 'creimp' pour 2014-2016, et correction d'une petite erreur pour 2012

Ces changements :

  • Ajoutent une fonctionnalité (par exemple ajout d'une variable).
  • Corrigent ou améliorent un calcul déjà existant.

'''
f7us = foyer_fiscal('f7us', period)

return f7us
Copy link
Member

Choose a reason for hiding this comment

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

Bizarre que cette variable soi réduite à une case ?
Pas de plafond rien ?
Par ailleurs, il faut utiliser cerfa-field pour indiquer la case en question

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oui comme pas mal d'autres réductions, il faut seulement reporter la réduction d'impôt finale sur la déclaration 2042 mais dans un autre formulaire (ici le formulaire no 2069-RCI-SD) les gens font eux même le calcul. C'est documenté page 232 de cette brochures pratique par exemple.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Je n'ai pas compris la remarque sur le cerfa_field .. ?

Copy link
Member

Choose a reason for hiding this comment

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

Les cases de l'impôt ont des champs cerfa_field comme ici

return (f2ab + f2ck + f8ta + f8tb + f8tc + f8te - f8tf + f8tg + f8th + f8to - f8tp + f8tl + f8ts + f8tz + f8uw +
f8uz + f8vm + f8wa + f8wb + f8wc + f8wd + f8we + f8wr + f8wt + f8wu)

# TODO : add tax credit 8VM and 8VL (2016)
Copy link
Contributor

Choose a reason for hiding this comment

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

Les commentaires peuvent être en français :)

@@ -22,6 +22,7 @@ def formula_2002_01_01(foyer_fiscal, period, parameters):
'''
Renvoie la somme des réductions d'impôt à intégrer pour l'année 2002
'''
accult = foyer_fiscal('accult', period)
Copy link
Contributor

Choose a reason for hiding this comment

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

est-ce qu'on pourrait trouver un nom de variable plus explicite ? acquisition_culture par exemple ?



class f8vm(Variable):
cerfa_field = {QUIFOY['vous']: u"8VM",
Copy link
Contributor

Choose a reason for hiding this comment

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

les QUIFOY était pour moi des éléments de legacy. @benjello tu confirmes ?

Copy link
Member

Choose a reason for hiding this comment

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

C'est peut-être encore utilisé marginalement.
Il faut/faudrait faire une recherche sur tous les cerfa_field pour voir si elle est maintenue de façon homogène ou pas. Sinon on peut amender la notation. Mais cela ne vaut pas le coup sur cette PR si on ne le fait pas sur tout le code.
Mon avis: enquête + nouveau ticket

@claireleroy claireleroy force-pushed the ipp-update-calcul-ipnet branch from 0573fa1 to 28fc100 Compare April 6, 2018 16:26
@Anna-Livia Anna-Livia changed the base branch from ipp-update-calcul-ipnet to master April 6, 2018 17:04
@claireleroy claireleroy force-pushed the ipp-update-remaining-tax-credit branch from 6c7103b to 6adefd9 Compare April 6, 2018 17:31
@claireleroy claireleroy merged commit 70855e1 into master Apr 6, 2018
@claireleroy claireleroy deleted the ipp-update-remaining-tax-credit branch July 13, 2018 13:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants