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

Fix AL when loan is fully reimbursed #875

Merged
merged 3 commits into from
Jan 19, 2018
Merged

Conversation

guillett
Copy link
Member

  • Évolution du système socio-fiscal.
  • Périodes concernées : toutes.
  • Zones impactées : prestations/aides_logement.py
  • Détails :
  • Garantie que l'aide au logement pour un foyer primo-accédant est nulle si le prêt est déjà remboursé.

@guillett
Copy link
Member Author

Fixes #869

@guillett guillett force-pushed the fix_al_primo_accedant branch from a60ab2a to a304ad1 Compare January 19, 2018 08:49
@@ -46,3 +46,28 @@
age: 40
output_variables:
aides_logement_primo_accedant: 0

- name: AL Primo Accédant / Prêt remboursé
output_variables:
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 que mettre output_variables:à la fin du test serait plus en phase avec l'organisation 'traditionnelle' des test ?

@@ -910,7 +910,7 @@ def formula_2007_07(famille, period, parameters):
K = famille('aides_logement_primo_accedant_k', period)
Lo = famille('aides_logement_primo_accedant_loyer_minimal', period)

return K * max_(0, (L + C - Lo))
return (L > 0) * K * max_(0, (L + C - Lo))
Copy link
Contributor

Choose a reason for hiding this comment

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

D'après ce que je comprends, les APL primo-accédant sont liés à la souscription d'un prêt d’accession sociale (PAS). Le remboursement du PAS (anticipé ou non) induit la perte des ces aides. Si c'est la bonne interprétation, une autre condition à ajouter serait l'obtention d'un PAS.

Copy link
Member Author

Choose a reason for hiding this comment

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

Merci pour l'information mais c'est un sujet différent du problème que l'on souhaite résoudre avec cette modification.

Il faudrait en effet regarder ça de plus près si des différences nous sont remontées par des usagers.

Anna-Livia
Anna-Livia previously approved these changes Jan 19, 2018
@Anna-Livia Anna-Livia dismissed their stale review January 19, 2018 09:08

Some questions

@guillett guillett merged commit 46f9f50 into master Jan 19, 2018
@guillett guillett deleted the fix_al_primo_accedant branch January 19, 2018 11:09
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.

3 participants