nl-digigo / visi

Beheeromgeving van de VISI open standaard.
https://nl-digigo.github.io/visi/
6 stars 4 forks source link

Prio-550: Bijlage verplicht kunnen stellen per berichttype #20

Closed JanaxLooij closed 5 years ago

JanaxLooij commented 6 years ago

Gemeld op: juni 2013 Melder: De Digitale Rotonde, ProRail en Strukton

Omschrijving Bij bijvoorbeeld document ter acceptatie moet een bijlage worden toegevoegd.

Motivatie Als er een document ter acceptatie verzonden moet worden en de bijlage wordt er niet bij gedaan weet je van te voren al dat het bericht afgekeurd gaat woorden. Er moet dus worden afgedwongen dat er een bijlage moet worden toegevoegd.

Impact

De impact van het verplicht stellen van een bijlage is laag. Dwingt af dat de berichten die een bijlage nodig hebben deze ook daadwerkelijk hebben. Kan in een aantal gevallen een voordeel voor gebruikers opleveren omdat er geen bijlagen meer vergeten worden.

Oplossing Het verplicht toevoegen van een bijlage kan niet afgedwongen worden door het verplicht aantal rijen in een tabel ook van toepassing te verklaren op appendix types.

Een kort door de bocht oplossing is om per MessageType aan te geven of een appendix verplicht is.

ENTITY MessageType; ... appendixMandatory: OPTIONAL BOOLEAN; ... END_ENTITY;

Hiermee kan in het raamwerk gedefinieerd worden dat in het geval berichttype Indiening_Termijnstaat verstuurd wordt ook een bijlage verplicht meegestuurd moet worden.

Backwards compatibility De oplossing is backwards compatible, omdat de boolean optioneel is gemaakt. Hierdoor hoeft deze optie niet gebruikt te worden.

Acties:

aanpassen _2.exp (actie: JvG, status=gedaan); reviewen van technische oplossing (actie: JvdB); uploaden _2.exp naar visi.codeplex.com (actie: JvdB); uitbreiden van testraamwerk (ober-kok) _7.xml (actie: JvG, status=gedaan); toevoegen van testscenario (actie: JvG, status=gedaan); uploaden van testraamwerk (ober-kok) _7.xml naar visi.codeplex.com (actie: JvdB);

Attachments

RE VISI TC onderwerp Bijlage en type bijlage verplicht kunnen stellen in bericht.msg _2.exp _7.xml

This work item was migrated from CodePlex

CodePlex work item ID: '1019' Assigned to: 'jeroenvanderburg' Vote count: '2'

JanaxLooij commented 6 years ago

[Arne_Bruinse@30-10-2015] Er ontstond discussie in de TC werksessie van 30-10-2015 of er nog een concrete klantvraag is voor deze issue, omdat deze wens vanuit de digitale rotonde afkomstig is.

De volgende mensen hebben prioriteit gegeven aan dit issue: 2 punten: Suzan Engelbrecht, ProRail 5 punten: Stefan Strube, Strukton 5 punten: Paul Jansen, CROW 3 punten: Jaap Holweg, Infostrait 2 punten: Peter Willems, TNO

Ik stuur een e-mail naar deze personen met de vraag om te illustreren wat de concrete probleemstelling in de praktijk is, zodat we daarmee in de volgende werksessie aan de slag kunnen of het punt kunnen annuleren.

JanaxLooij commented 6 years ago

[UnknownUser@8-12-2015]

JanaxLooij commented 6 years ago

[Arne_Bruinse@8-12-2015] bijgevoegd de mail waarin suzan engelbrecht van prorail en stefan strube van strukton gereageerd hebben, met daaronder mijn vraag. Zelf ben ik ook voorstander van deze optie, ookal hebben we dit keer geen punten aan dit punt gegeven.

JanaxLooij commented 6 years ago

[gspees@11-12-2015] Tekst uit de mail van Arne: Zouden jullie mij aan kunnen geven:

Reactie van Stefan Strube van Strukton: Het type heeft wat mij betreft betrekking op het bestandstype, zodat je inderdaad kunt sturen op archiefwaardige soorten bijlagen.

Reactie van Suzan Engelbrecht van Prorail: Lijkt mij goed dat de transactie acceptatie document gaat afdwingen dat er een bijlage toegevoegd moet worden en zonder bijlage niet verstuurd kan worden. Hiermee vergroten we de kans dat een bericht first time right ingediend wordt waardoor het proces van afhandelen sneller verloopt.

De bijlage tabel moet als verplicht veld in deze transactie staan. VISI moet daarbij kunnen reageren dat er echt een bijlage toegevoegd wordt en niet alleen tekst. Pas met bijlage kan het bericht verzonden worden. Daarnaast we willen dat er archiefwaardige bijlages toegevoegd worden misschien kunnen we dat hierin ook wel afdwingen.

JanaxLooij commented 6 years ago

[UnknownUser@11-12-2015]

JanaxLooij commented 6 years ago

[jvgeijlswijk@11-12-2015] Dit punt is op 11-12-2015 kort besproken tijdens de werksessie door Ge, Peter, Thomas, Jaap, Jeroen.

De behoefte is voor twee zaken:

  1. in een raamwerk definieren welke type bestanden (pdf, xls, otf, jpg, tif, etc.) meegestuurd mogen worden als bijlage;
  2. in een raamwerk definieren dat er een bestand meegestuurd moet (!) worden voor een bepaald berichttype.

Oplossing voor punt 1 is het uitbreiden van het raamwerk. Voor AppendixType moet het fileType gedefineerd worden. P.S. In _5.exp bestaat al een eigenschap fileType (AppendixTemplate).

Oplossing voor punt 2 is het verplicht stellen van een ComplexElementType in een AppendixType. Vervolgens kan de "min-/maxOccurs" van ComplexElementType gebruikt worden voor het definieren dat er een bestand meegestuurd moet (!) worden voor een bepaald berichttype.

JanaxLooij commented 6 years ago

[Arne_Bruinse@15-12-2015] Graag in de volgende werksessie nog deze vraag bespreken: Met deze oplossing moet er denk ik wel iets geregeld worden voor berichten/transacties waarop geen appendix type ingesteld is. Volgens mij zijn in dat geval alle appendix types kiesbaar, waaronder het appendix type waarin het toevoegen van een bijlage verplicht is. Het lijkt mij onwenselijk dat je dan op alle transacties/berichten een appendix type in moet stellen waar je niet die verplichting wilt hebben. Misschien door een of meerdere appendix types als "default" in te kunnen stellen, of door een uitzondering op te nemen dat deze regel alleen geldt als de appendix type echt gekoppeld is?

JanaxLooij commented 6 years ago

[jvgeijlswijk@19-2-2016] Een nieuw work item 4684 is aangemaakt voor type bijlage (pdf, jpg, etc.). En dit work item gaat alleen over het verplichten van een bijlage.

JanaxLooij commented 6 years ago

[UnknownUser@19-2-2016]

JanaxLooij commented 6 years ago

[UnknownUser@19-2-2016]

JanaxLooij commented 6 years ago

[UnknownUser@19-2-2016]

JanaxLooij commented 6 years ago

[UnknownUser@19-2-2016]

JanaxLooij commented 6 years ago

[jvgeijlswijk@8-4-2016] _2.exp is aangepast. Testscenario 9 is toegevoegd aan de documentatie, zie https://visi.codeplex.com/wikipage?title=Testscenario%27s en testraamwerk (ober-kok) _7.xml is uitgebreid met appendixMandatory voor berrichttype msgAanbiedingMenuKaart.

JanaxLooij commented 6 years ago

[UnknownUser@8-4-2016]

JanaxLooij commented 6 years ago

[UnknownUser@8-4-2016]

JanaxLooij commented 6 years ago

[jvgeijlswijk@8-4-2016] N.B. _2.exp en _7.xml moeten nog geupload worden naar de "source code".

JanaxLooij commented 6 years ago

[UnknownUser@8-4-2016]