-
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
Fix de net a brut #786
Fix de net a brut #786
Conversation
d61cad5
to
faac639
Compare
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.
Remarques mineures.
@@ -13,7 +13,7 @@ def calculate_net_from(salaire_de_base, simulation, period, requested_variable_n | |||
|
|||
# We're not wanting to calculate salaire_de_base again, but instead manually set it as an input variable | |||
# To avoid possible conflicts, remove its function | |||
simulation.holder_by_name['salaire_de_base'].formula.function = None | |||
simulation.get_holder('salaire_de_base').formula.function = None |
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 fonction étant dépréciée, la remplacer par simulation.get_variable_entity('salaire_de_base').get_holder('salaire_de_base')
?
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.
👌
|
||
* Changement mineur | ||
* Détails | ||
- Répare la réforme `de_net_a_brut`. |
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.
Préciser la version à partir de laquelle elle a été cassée ?
53f3fe7
to
7561281
Compare
de_net_a_brut
.