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

Onduidelijkheid over bedoeling feature foutmeldingen #494

Open
NicoleKortoomsBAG opened this issue Nov 4, 2021 · 3 comments
Open

Onduidelijkheid over bedoeling feature foutmeldingen #494

NicoleKortoomsBAG opened this issue Nov 4, 2021 · 3 comments

Comments

@NicoleKortoomsBAG
Copy link
Contributor

@JohanBoer , @MelvLee ,
Op regel 157 van de foutafhandeling feature staat een validatie voor als er 1 getal wordt opgegeven bij een locatie of bbox parameter.
De reason wordt dan: "Waarde is geen geldige geometrie."

Echter, er zit ook een validatie minItems/maxItems op die velden. Je krijgt dus ook de validatiemelding: "Array bevat minder dan 2 items." bij een locatie parameter, en "Array bevat minder dan 4 items." bij een bbox parameter.
Is dit inderdaad de bedoeling?

voorbeeld:
"invalidParams": [
{
"name": "locatie",
"code": "minItems",
"reason": "Array bevat minder dan 2 items."
},
{
"name": "locatie",
"code": "geometry",
"reason": "Waarde is geen geldig geometrie."
}
]

@MelvLee
Copy link
Collaborator

MelvLee commented Nov 8, 2021

Omdat locatie en bbox complexe parameters zijn, kan het voorkomen dat er niet aan meerdere validatie regels wordt voldaan:

  • teveel of te weinig items
  • één of meerdere items voldoet niet aan de type
  • één of meerdere items voldoen niet aan de opgegeven crs

Ik denk dat het daarom niet handig is om voor een geometrie parameter voor elke validatie regel een aparte code of een generieke code te gebruiken. In beide gevallen wordt de consumer code namelijk complexer. Bij verschillende codes moet logica worden geïmplementeerd om de codes te kunnen afhandelen en bij generieke codes moet logica worden geïmplementeerd om de code binnen context te kunnen interpreteren. minItems heeft een andere betekenings voor locatie en bbox dan voor bijv. de pandIdentificaties parameter.

@NicoleKortoomsBAG
Copy link
Contributor Author

@MelvLee Ik lees hierboven de overwegingen die meespelen om de vraag te beantwoorden, maar niet echt een uitkomst, dus ik weet nog niet zo goed wat nu is gewenst.

@MelvLee
Copy link
Collaborator

MelvLee commented Nov 8, 2021

@NicoleKortoomsBAG, één fout als de waarde niet valide is. Dus, zoals het is gespecificeerd in de foutafhandeling.feature. Voor crs geldt een uitzondering omdat de waarde valide kan zijn, maar dat toch niet is omdat het niet overeenkomt met de opgegeven crs. Verder heeft geometry hogere 'precendence' dan geometryMismatch. Ik zal hiervoor een scenario opnemen in de foutafhandeling.feature file

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Haal-Centraal-BAG
  
Backlog
Development

No branches or pull requests

2 participants