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

Réactive le crédit d'impôt pour cotisations syndicales #1271

Merged
merged 7 commits into from
Feb 16, 2019

Conversation

claireleroy
Copy link
Contributor

@claireleroy claireleroy commented Feb 13, 2019

Contexte : Les cotisations syndicales ouvraient droit à une réduction d'impôt jusqu'à l'imposition des revenus 2011 puis à un crédit d'impôt à partir de l'imposition des revenus 2012. Le mode de calcul reste le même. Sources : Article 119 quarter C du CGI

Dans la version actuelle d'OpenFisca, le calcul des crédits d'impôt utilise la variable cotsyn (réduction d'impôt pour cotisations syndicales) dont l'attribut end était fixé au 31/12/2011. Par conséquent, le calcul des crédits d'impôt est faux : il ne prend pas en compte le crédit d'impôt pour cotisations syndicales.
Je propose de créer une nouvelle variable cotsyn2 spécifique à ce crédit d'impôt et utilisée dans le calcul de credits_impot afin de corriger cette erreur.

Lié à l'issue #1202

  • Évolution du système socio-fiscal. | Amélioration technique.
  • Périodes concernées : à partir du 01/01/2012.
  • Zones impactées : openfisca_france/model/prelevements_obligatoires/impot_revenu/credits_impot.py.
  • Détails :
    • Renomme la variable cotsyn en reduction_cotisations_syndicales
    • Crée une nouvelle variable credit_cotisations_syndicales
    • Calcule les crédits d'impôt credits_impot en utilisant cette nouvelle variable.

Ces changements :

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

@benjello
Copy link
Member

Je suggère de renommer:

  • cotsyn en cotisations_syndicales_reduction
  • cotsyn2 en cotisations_syndicales_credit

@claireleroy claireleroy requested a review from Morendil February 13, 2019 13:10
@claireleroy claireleroy changed the title Réactive le crédit d'impôt pou cotisations syndicales Réactive le crédit d'impôt pour cotisations syndicales Feb 13, 2019
@claireleroy claireleroy requested a review from fpagnoux February 13, 2019 16:41
Copy link
Contributor

@sandcha sandcha left a comment

Choose a reason for hiding this comment

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

Merci @claireleroy d'avoir signalé ce problème !
Ca semble être une régression introduite par la PR #1195

Pour renforcer le calcul, il faudrait rajouter un test. Est-ce qu’on l’ajoute ou préfères-tu le faire ?

Pour exemple, en utilisant les simulateurs sur les revenus de 2011 et 2012, on voit apparaître le crédit d’impôt sur 2012 à un montant de 66€ pour un salaire de 10 000 et une cotisation de 700 :)

@@ -584,6 +584,28 @@ def formula_2005(foyer_fiscal, period, parameters):
)


class credit_cotisations_syndicales(Variable):
Copy link
Contributor

Choose a reason for hiding this comment

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

Il semble que cotsyn ait déjà été utilisée dans les crédits. Éviter la duplication de la formule en supprimant cette nouvelle variable ?

value_type = float
entity = FoyerFiscal
label = u"cotsyn"
label = u"Réduction d'impôt pour cotisations syndicales"
definition_period = YEAR
end = '2011-12-31'
Copy link
Contributor

@sandcha sandcha Feb 13, 2019

Choose a reason for hiding this comment

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

Supprimer cette date de fin ? C’est une régression introduite par cette ligne de la PR #1195 comme on l’a confirmé avec @Morendil.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@sandcha Si j'ai bien compris tu suggères une solution plus légère en termes de ligne de code qui consisterait à ne pas créer une deuxième variable mais à supprimer la date de fin de reduction_cotisations_syndicales (ex cotsyn) ? Mais le problème c'est qu'à partir de 2012, les cotisations syndicales compteront alors double : à la fois comme réduction et crédit (puisque la nouvelle formule de reductions n'a plus de formules datées suite à la PR #1195 mais une formule unique pour toutes les périodes)

Copy link
Contributor

Choose a reason for hiding this comment

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

Oui bien vu! Je pense qu'on peut trouver une solution sans pour autant dupliquer la formule.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@Morendil @sandcha je n'ai pas trouvé de solution sans duplication .. mais si vous en voyez une je suis preneuse !

@claireleroy
Copy link
Contributor Author

@sandcha Merci pour la review!

Pour le test je veux bien que vous le fassiez si vous en avez déjà un en tête du coup (j'essaierais de me rappeler d'en faire lors de mes prochaines contributions promis !)

Pour la solution proposée, j'ai l'impression que ça ne fonctionnera pas (cf. ma réponse dans les commentaires).

@Morendil Morendil force-pushed the active-credit-cotsyn branch 3 times, most recently from 0a260f3 to 71c084c Compare February 16, 2019 13:10
@Morendil Morendil dismissed sandcha’s stale review February 16, 2019 16:09

Corrigé selon les remarques

@Morendil Morendil force-pushed the active-credit-cotsyn branch from 0fd14fe to 443946a Compare February 16, 2019 16:11
@Morendil Morendil merged commit 11b1898 into master Feb 16, 2019
@Morendil Morendil deleted the active-credit-cotsyn branch February 16, 2019 16:23
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