-
Notifications
You must be signed in to change notification settings - Fork 9
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
[Eure-et-Loir] Ajoute les prestations de compensation de handicap #72
Conversation
@CecileIAEGI Pour la PCH à domicile, le descriptif en en-tête de cette PR correspond parfaitement bien à ce qui a été résumé dans le tableau Eure-et-Loir mais comprend quelques incohérences avec le RDAS. En particulier :
Les 2 et 3 peuvent être complétées ultérieurement. La 1 me semble plus source d'erreur. |
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.
👍 Code très clair
Demande de changement pour le même bug que pour les aides ménagères (correctif proposé).
Et, dans un premier temps, je serais en faveur d'une intervention sur deux incohérences tableau Eure-et-Loir/RDAS : à domicile, Complément AEEH au lieu d'AEEH + en établissement, ajout de PCH domicile antérieure avant.
@CecileIAEGI, j'ai conscience que cela équivaut à une incohérence de spécifications pour ce livrable ; voir le moment le plus opportun aussi pour toi pour leur vérification/traitement.
condition_handicap = individu('handicap', period) | ||
condition_hebergement = individu.famille('place_hebergement', period) | ||
|
||
return condition_residence * condition_nationalite * condition_handicap * condition_hebergement |
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.
De même que pour la PCH à domicile, la PCH en établissement hériterait des incohérences (1, 2 et 3 évoquées) si je saisis bien cette phrase du RDAS (page 88-en-haut/84-en-bas) :
➕ Une condition supplémentaire d'attribution qui me semble être "avoir bénéficié de la PCH à domicile" :
Ajouter cette dernière condition ? Qu'en dis-tu ? 🤔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.
ça me parait compliqué à rajouter car comme condition car pour moi c'est le binome (ne sort jamais de l'établissement + ne bénéficiait pas avant de la PCH) qui s'applique.
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.
Ceci dit, si on ne fait pas référence à la PCH à domicile comme prérequis à la PCH en établissement, il me semble qu'il manque à minima les mêmes conditions de non cumul dans la formule de la PCH en établissement.
Vérifier le tout avec le métier pour une prochaine PR ?
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.
Oui en effet, je pense que tu as raison pour les règles de non cumul avec l'aeeh et l'actp pour la PCH à domicile
Je vais les rajouter.
Malheureusement je ne peux pas vraiment contacter le métier parce que officiellement ma mission avec le CD28 est terminée. Donc je me abse sur ce qui est inscrit dans le RDAS et lorsque la suite de la mission commencera (si elle commence) je pourrais demander au métier si on a encore besoin.
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 @CecileIAEGI 🙂
Je joins deux propositions de changement non bloquantes.
Par ailleurs, comme indiqué en commentaire, il me semble qu'il manque a minima les règles de non cumul pour la PCH en établissement ou son lien avec la PCH à domicile pour une prochaine PR selon ce que dira le métier !
En tout cas, bon pour merge après rebase et versionnement !
#condition_aides_aeeh = not_(individu.famille('aeeh', period) > 0) | ||
condition_aides_aeeh = not_(individu.famille('beneficiaire_complement_aeeh', period)) | ||
condition_aides_apa = not_(individu('apa_domicile', period) > 0) | ||
condition_aides_actp = not_(individu('beneficiaire_actp', period)) * not_(individu('beneficiaire_acfp',period)) |
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.
condition_aides_actp
n'est-il pas ambigu quand il représente aussi bien l'ATCP (allocation compensatrice pour tierce personne) et l'ACFP (allocation compensatrice frais professionnels) ? Renommer ?
Pour le contexte, il semble que la PCH remplace l'ACTP et l'ACFP depuis 2006 sans les interrompre pour les bénéficiaires qui l'avaient déjà d'après cet article (loi 2005-102).
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.
les gens qui lisent le code vont voir que c'est actp + acfp.
Est-ce réellement nécessaire de renommer sachant que acfp va avec le temps disparaitre complètement ?
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.
C'est que la formule restera dans le temps.
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.
si tu préfères je peux faire une petite mise à jour et mettre conditions_aides_actp_acfp ?
55b8612
to
ebe1058
Compare
Le lien entre la PCH à domicile et la PCH en établissement n'a pas été ajouté à ce stade afin de représenter les deux aides au niveau du simulateur. À compléter ultérieurement au besoin. |
Ajout des aides complètes PCH à domicile et PCH en établissement
- Renommage fichiers et class - Ajout condition_residence - Modification des opérateurs
Ajout pch_etablisement.yml pour le remplacer
- Ajout de conditions de nationalité refugie + apatride - Ajout de la condition de résidence - Ajout des tests associés à ces ajouts de conditions
- Non cumul avec le complément aeeh ajouté à la place de non cumul avec l'aeeh - Ajout de la condition non bénéficiaire de l'acfp comme décrit dans le RDAS - Correction des conditions et suppression de variables intermédiaires inutiles
- Suppression de variables intermédiaires inutiles - Ajout des règles de non cumul pour la PCH à domicile - Mise à joru des tests associés
Rebase en cours. |
83ad7f9
to
5fc3088
Compare
Ajout des aides PCH de l'Eure-et-Loir
Les conditions d'attributions des aides PCH sont les suivantes :
PCH à domicile :
PCH en établissement :