Project activity #35727
closed
Validatie implementeren bij het zetten van een NVB resultaat
Added by Diego Mirandola 2 months ago.
Updated 2 days ago.
Description
Als de gebruiker een NVB resultaat wil zetten, dan moet er gecheckt worden dat de zaak bepaalde eisen voldoet.
Bijvoorbeeld:
- als afleidingswijze: "eigenschap" moet er gecheckt worden dat de bijbehorende eigenschap ingevuld is
- ...
Dit moet twee keer gecheckt worden:
- in de app, in dialog 'afhandelen', als er een NVB resultaat wordt geselecteerd; als de check faalt, dan 'ok' is disabled en een melding met voldoende informatie komt tevoorschijn
- in de backend, in de implementatie van 'zaakClose' action; we doen dit omdat de validatie in openzaak gebeurt niet bij het zetten van de resultaat maar bij het zetten van de eindstatus, dus we willen voorkomen dat de resultaat wordt gezet zonder eindstatus; als de eerste punt correct geïmplementeerd is dan is deze overbodig, maar het is correct dat een api altijd zijn eigen validatie implementeert.
We gaan dit zo implementeren:
- backend service die de validatie logica implementeert
- webscript die deze backend service gebruikt, de frontend moet deze gebruiken om check 1 uit te voeren
- 'zaakClose' action executer moet deze service gebruiken om check 2 uit te voeren
- Assignee set to Diego Mirandola
- Status changed from Backlog to In Progress
Eerste opzet is klaar in tezza-services branch feature/nvb-validate-resultaat/#35727. Dit bevat:
Voortgang in de logica voor de volgende afleidingswijzen:
- eigenschap: klaar
- hoofdzaak: klaar maar ik snap niet hoe ik deze kan testen
- termijn: klaar (doet eigenlijk niets)
- zaakobject: objecttype wordt gecheckt, ik snap niet hoe ik de datumKenmerk op het zaakobject kan definiëren
- Target version deleted (
Tezza Backlog)
#35727#note-8 is klaar in TS branch feature/nvb-validate-resultaat/#35727
NVB validaties voor de volgende afleidingswijzen zijn geïmplementeerd:
- eigenschap
- hoofdzaak
- termijn
- zaakobject, maar: voorlopig wordt er alleen gecheckt dat een zaakobject bestaan met de aangegeven objecttype en dat deze een geldig url heeft, er is nog geen check op de datumKenmerk
De volgende wijzigingen in tezza app zijn klaar in contezza-apps branch feature/tezza/nvb-validate-resultaat/#35727.
In zaak afhandelen dialoog, als er een nvb resultaattype wordt geselecteerd, wordt webscript #35727#note-9 gebruikt om de combinatie zaak + resultaattype te valideren.
Als de zaak alle eisen voldoet, dan de huidige melding is zichtbaar:
* De vernietigingstermijn gaat lopen als gerelateerde belangen zijn vervallen.
Anders, wordt de volgende error getoond:
Het is niet mogelijk om dit resultaattype te selecteren want de volgende bron ontbreekt of is ongeldig: <invalidResource>
waar de <invalidResource> komt vanuit de webscript response.
Tests zijn klaar in contezza-integration-tests branch feature/nvb-validate-resultaat/#35727.
De volgende flow wordt getest en via api en via ui:
- zaak aanmaken
- combinatie zaak + resultaattype met afleidingswijze=eigenschap valideren, error verwacht
- zaakeigenschap toevoegen
- combinatie zaak + resultaattype met afleidingswijze=eigenschap valideren, succes verwacht
Let op: de tests zijn gebaseerd op catalogus NVB die aangemaakt is op dev-tezza. Ik heb die catalogus lokaal geïmporteerd om te kunnen ontwikkelen en testen.
- Status changed from In Progress to Ready in Dev
- Assignee changed from Diego Mirandola to Rick de Rooij
- Status changed from Ready in Dev to Test approved
- % Done changed from 0 to 100
Akkoord, zit in TS 3.6.4.
- Status changed from Test approved to Resolved
- Status changed from Resolved to In Progress
- Assignee changed from Rick de Rooij to Diego Mirandola
- Status changed from In Progress to Ready in Dev
- Assignee changed from Diego Mirandola to Maaike Bommerson
De backend is nu op dev.
Graag testen in branch feature/tezza/nvb-validate-resultaat/#35727, tezza app gekoppeld met dev-tezza.
Het afhandelen van zaken van zaaktypen in catalogus NVB moet zo werken:
- Afleidingswijze brondatum Eigenschap: toont nvb error en laat niet afhandelen totdat de zaakeigenschap 'geboortedatum leerling' wordt toegevoegd, daarna toont nvb info en laat afhandelen
- Afleidingswijze brondatum Hoofdzaak: toont nvb error en laat niet afhandelen
- Afleidingswijze brondatum Termijn: toont nvb info en laat afhandelen
- Afleidingswijze brondatum Zaakobject: toont nvb error en laat niet afhandelen
Nvb error en info zijn degene in #35727#note-10.
- Status changed from Ready in Dev to Test approved
- Assignee changed from Maaike Bommerson to Diego Mirandola
Getest, kan mee met de release
- Status changed from Test approved to Resolved
Backend is in TS 3.6.4.
Frontend is in contezza-apps 4.6.3.
Also available in: Atom
PDF