-
Notifications
You must be signed in to change notification settings - Fork 1
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
[Tech] Ajout de la validation métier des cas d'usage côté serveur #2014
base: main
Are you sure you want to change the base?
Conversation
dade474
to
aff2b85
Compare
|
||
private fun validateMission(mission: MissionEntity) { | ||
if (mission.startDateTimeUtc.isAfter(mission.endDateTimeUtc)) { | ||
throw BackendUsageException( |
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 faut faire comme ça pour que ca remonte bien cpoté front :
throw BackendUsageException(code = BackendUsageErrorCode.UNVALID_PROPERTY, message = "La date de fin doit être postérieure à la date de début ")
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 la même chose ce que j'ai fait ici, c'est le meme ordre dans le constructeur, est-ce que tu veux que je change ?
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.
je me suis trompée c'est data
et pas message
qu'il faut mettre, comme dit sur Mattermost
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.
comme on s'est dit, est-ce que le code ne suffit pas pour interpréter l'erreur ?
backend/src/main/kotlin/fr/gouv/cacem/monitorenv/domain/validators/mission/MissionValidator.kt
Outdated
Show resolved
Hide resolved
backend/src/main/kotlin/fr/gouv/cacem/monitorenv/domain/validators/mission/MissionValidator.kt
Show resolved
Hide resolved
backend/src/main/kotlin/fr/gouv/cacem/monitorenv/domain/validators/mission/MissionValidator.kt
Outdated
Show resolved
Hide resolved
backend/src/main/kotlin/fr/gouv/cacem/monitorenv/domain/validators/mission/MissionValidator.kt
Outdated
Show resolved
Hide resolved
...d/src/main/kotlin/fr/gouv/cacem/monitorenv/domain/validators/reporting/ReportingValidator.kt
Show resolved
Hide resolved
...n/kotlin/fr/gouv/cacem/monitorenv/domain/validators/vigilance_area/VigilanceAreaValidator.kt
Show resolved
Hide resolved
backend/src/main/kotlin/fr/gouv/cacem/monitorenv/domain/validators/mission/MissionValidator.kt
Show resolved
Hide resolved
@@ -18,9 +18,9 @@ data class CreateOrUpdateReportingDataInput( | |||
val targetDetails: List<TargetDetailsEntity>? = listOf(), | |||
val geom: Geometry? = null, |
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.
geom
ne peut être null
70fec30
to
e18691b
Compare
e18691b
to
ef1630f
Compare
Quality Gate passedIssues Measures |
Règle de gestion métier
Ces règles ne prennent pas en compte les données dites « obligatoires » qui sont validé par le typage de l’objet
Mission en cours
Date de début supérieure à la date de fin
Minimum 1 unité de contrôle
Type de la mission
Trigrammes optionnels « ouvert par » et « complété par » mais minimum 3 caractères
Contrôle
Surveillance
Règle supplémentaire pour une Mission terminée
Signalement
Zone de vigilance uniquement pour publication (non-brouillon)
Règle supplémentaire pour une zone de vigilance « en tout temps »
Related Pull Requests & Issues