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

Splitsen Monitoringresultaat in: MonitoringresultaatMetOnderbouwing en MonitoringresultaatMetWaarde #167

Closed
wilkoquak opened this issue Mar 3, 2022 · 3 comments
Assignees
Labels
9 enhancement New feature or request SDIMG-59 Komt voort uit SDIMG-59 (#150) verwerkt in v2.0 verwerkt in versie 2.0 van IMG

Comments

@wilkoquak
Copy link
Member

wilkoquak commented Mar 3, 2022

Aanleiding

Dit wijzigingsverzoek komt voort uit issue #150 om het aantal validatieregels in het model te minimaliseren. In dit geval gaat het om de klasse Monitoringresultaat waarbij het attribuut kwalitatieveOnderbouwing en de regels die erbij horen redundant zijn.

In de klasse Monitoringresultaat zit een optioneel attribuut 'kwalitatieveOnderbouwing' dat als het de waarde 'True' heeft aangeeft dat de documentverwijzing over een kwalitatieve onderbouwing is. Dit kun je ook al zien aan het feit of het attribuut monitoringwaarde is ingevuld. Immers, alleen als hier geen waarde staat is een kwalitatieve onderbouwing nodig.

Zie diagram:
image

Wijzigingsverzoek

  • Verwijder het attribuut kwalitatieveOnderbouwing; hiermee verdwijnen op de regels voor dit attribuut.
  • Voeg in de documentatie toe dat indien de monitoringswaarde niet is ingevuld de verwijzing naar het document logischerwijs een kwalitatieve onderbouwing is en anders niet.

Het resulterende diagram ziet er dan als volgt uit:
image

Impactanalyse

  • Het voorstel heeft geen impact op de inhoud van het model.
  • Het voorstel resulteert in een niet-backwards compatibele wijzigings van het xml-schema
  • Allen betrokken partijen zullen de schema wijziging merken
@wilkoquak wilkoquak added the SDIMG-59 Komt voort uit SDIMG-59 (#150) label Mar 3, 2022
@PB-GNM PB-GNM added the 9 enhancement New feature or request label Mar 4, 2022
@Geonovum Geonovum deleted a comment from wilkoquak May 23, 2022
@wilkoquak wilkoquak self-assigned this Jun 2, 2022
@PB-GNM
Copy link
Collaborator

PB-GNM commented Jun 7, 2022

Discussie van adviesgroep 30-5-2022:
Het is belangrijk om altijd de onderbouwing toe te voegen aan het IMG. Kunnen we dat nog wel doen als de waarde ontbreekt?
Ja, dat kan nog steeds. De onderbouwing wordt altijd in een apart veld toegevoegd aan het IMG. Het attribuut geeft enkel informatie over het type monitoringsresultaat.

Advies van adviesgroep 30-5-2022: wijziging doorvoeren.

@PB-GNM
Copy link
Collaborator

PB-GNM commented Jun 7, 2022

De wijziging is doorgevoerd in versie 2.0 van IMG

@PB-GNM PB-GNM added the verwerkt in v2.0 verwerkt in versie 2.0 van IMG label Jun 7, 2022
@wilkoquak
Copy link
Member Author

Met publicatie van release def-im-img-20220705 kan dit issue worden afgesloten.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
9 enhancement New feature or request SDIMG-59 Komt voort uit SDIMG-59 (#150) verwerkt in v2.0 verwerkt in versie 2.0 van IMG
Projects
None yet
Development

No branches or pull requests

2 participants