-
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
Ajoute l'exclusion des cas particuliers dans le calcul de la base ressource patrimoine #975
Conversation
Bonjour et merci @frtomas, aurais-tu des éléments législatifs concernant ces modifications ? |
famille_percoit_aah = famille.any(membres_famille_percoit_aah) | ||
famille_percoit_aeeh = famille('aeeh', period) > 0 | ||
|
||
return not_(est_locataire_foyer) * not_(famille_percoit_aah) * not_(famille_percoit_aeeh) *\ |
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.
Il me semle plus judicieux d'utiliser plus de parenthèses plutôt que \
famille_percoit_aeeh = famille('aeeh', period) > 0 | ||
|
||
return not_(est_locataire_foyer) * not_(famille_percoit_aah) * not_(famille_percoit_aeeh) *\ | ||
(patrimoine > 30000) * ( |
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.
La valeur 30000 est un paramètre de la législation qui doit être renseigné comme tel et ne pas être inscrit en dur.
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 de ces retours, les deux corrections sont faites.
Bonjour @guillett , la seule référence qui m'a été fournit est cette note explicative, qui est effectivement officielle, mais qui n'a peut être pas de valeur juridique Cela suffit-il ? Si oui, je l'ajouterai en référence une fois que j'aurais fini de traiter le reste des retours. |
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.
Encore merci. J'ai fait quelques remarques supplémentaire.
Pour les références, je me dit qu'un commentaire reprenant le texte serait bien en identifiant la source comme tu veux.
@@ -0,0 +1,6 @@ | |||
description: Seuil de l'estimation de la valeur du patrimoine au delà duquel le patrimoine est valorisé | |||
reference: https://www.legifrance.gouv.fr/affichTexte.do?cidTexte=JORFTEXT000033243725&categorieLien=id |
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.
Le lien suivant est plus stable (cf. http://www.eli.fr/fr/constructionURI.html)
https://www.legifrance.gouv.fr/eli/decret/2016/10/12/LHAL1606833D/jo/texte
CHANGELOG.md
Outdated
* Évolution du système socio-fiscal. | ||
* Périodes concernées : à partir du 01/10/2016. | ||
* Zones impactées : | ||
- `prestations/` |
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.
prestations/aides_logement
famille_percoit_aah = famille.any(membres_famille_percoit_aah) | ||
famille_percoit_aeeh = famille('aeeh', period) > 0 | ||
|
||
seuil_valorisation = parameters(period).prestations.aides_logement.patrimoine.seuil_valorisation |
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.
👍
Ok @guillett j'ai fait les ajustements nécessaires et j'ai ajouté la référence. |
…trimoine dans les ressources pour le calcul de l'AL et les tests correspondants
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 beaucoup @frtomas pour les différentes modifications !
J'ai remplacé ton commit de merge par un rebase.
J'attends le passage des tests et je merge.
Merci @guillett , j'avais oublié que depuis github c’était un merge et non un rebase qui était fait. |
prestations/
Ces changements :