-
Notifications
You must be signed in to change notification settings - Fork 10
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
Comments
Omdat locatie en bbox complexe parameters zijn, kan het voorkomen dat er niet aan meerdere validatie regels wordt voldaan:
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. |
@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. |
@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 |
@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."
}
]
The text was updated successfully, but these errors were encountered: