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 - signalement] Création d'un désordre par erreur si superficie non définie #2208

Merged
merged 1 commit into from
Feb 5, 2024

Conversation

sfinx13
Copy link
Collaborator

@sfinx13 sfinx13 commented Feb 2, 2024

Ticket

#2207

Description

Sur un signalement fait pas un tiers, la superficie n'est pas obligatoire. Si c'est une pièce unique, on a le désordre suivant par erreur
"La superficie du logement < 9m"
et du coup, les scores et qualifs qui vont avec.
image

Dans le calcul des désordres liés à la composition du logement, vérifier que la superficie est bien définie.
DesordreCompositionLogementLoader->load() ligne 51

Changements apportés

  • Ajout d'une condition avant d'ajouter le désordre

Pré-requis

Tests

  • Créer un signalement tiers sans superficie avec pour nombre de pièce égale à 1 et vérifier que le désordre La superficie du logement < 9m n'existe pas
  • Editer le signalement en mettant une superficie et un nombre de pièce égale à 1, vérifier la présence du désordre

@sfinx13 sfinx13 changed the base branch from main to develop February 2, 2024 16:34
Copy link

sonarcloud bot commented Feb 2, 2024

Quality Gate Failed Quality Gate failed

Failed conditions

6.5% Duplication on New Code (required ≤ 3%)
B Reliability Rating on New Code (required ≥ A)

See analysis details on SonarCloud

idea Catch issues before they fail your Quality Gate with our IDE extension SonarLint SonarLint

Copy link
Collaborator

@hmeneuvrier hmeneuvrier left a comment

Choose a reason for hiding this comment

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

Relecture et test OK

@hmeneuvrier hmeneuvrier merged commit 83c229c into develop Feb 5, 2024
2 of 3 checks passed
@sfinx13 sfinx13 deleted the bugfix/2207-check-superficie-desordre branch February 13, 2024 21:05
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.

None yet

2 participants