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

[BO - Edition] Recalcul Suroccupation #2188

Merged
merged 1 commit into from
Feb 1, 2024

Conversation

hmeneuvrier
Copy link
Collaborator

@hmeneuvrier hmeneuvrier commented Feb 1, 2024

Ticket

#2175

Description

Quand on changeait la valeur d'allocataire dans Situation du foyer, le tag Suroccupation n'était pas mis à jour, car les valeurs setLogementSocialAllocationCaisse et setLogementSocialAllocation de situationFoyer n'étaient pas mises à jour

Changements apportés

  • Mise à jour du SignalementManager

Pré-requis

Tests

  • Faire un signalement avec une toute petite superficie et un grand nombre de pièce (ou vice-versa) pour avoir le tag Suroccupation soit si on est allocataire, soit si on le l'est pas, mais pas dans tous les cas
  • Editer la valeur allocataire du signalement et vérifier que le tag suroccupation se met à jour comme il faut

@hmeneuvrier hmeneuvrier linked an issue Feb 1, 2024 that may be closed by this pull request
Copy link

sonarcloud bot commented Feb 1, 2024

Quality Gate Passed Quality Gate passed

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@emilschn
Copy link
Collaborator

emilschn commented Feb 1, 2024

Je ne pense pas que ce soit lié à ça, mais j'ai voulu éditer la situation du foyer pour changer la valeur d'allocataire, et j'ai eu cette erreur :
App\Entity\SignalementQualification::setDernierBailAt(): Argument #1 ($dernierBailAt) must be of type ?DateTimeImmutable, DateTime given, called in /app/src/Factory/SignalementQualificationFactory.php on line 59

[ edit : transféré dans ticket #2189 ]

Copy link
Collaborator

@emilschn emilschn left a comment

Choose a reason for hiding this comment

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

OK lecture et test

Copy link
Collaborator

@sfinx13 sfinx13 left a comment

Choose a reason for hiding this comment

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

Lecture et test avec récupération de la PR #2191
Je merge les deux

@sfinx13 sfinx13 merged commit 4098f40 into develop Feb 1, 2024
3 checks passed
@sfinx13 sfinx13 deleted the bugfix/2175-bo-edition-recalcul-suroccupation branch February 13, 2024 21: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.

[BO - Edition] Recalcul Suroccupation
3 participants