zoeken met oppervlakte min > max geeft geen fout

[melsk-r]

Originally created by fsamwel (https://github.com/VNG-Realisatie/BAG-Gemeentelijke-wensen-tav-BAG-Bevragingen/issues/476):

Wanneer ik een oppervlakte[min} opgeef die groter is dan oppervlakte[max] krijg ik een lege collectie, geen foutmelding.

@MelvLee wat zou jij hier liever zien? Een lege collectie (zoals nu) of een foutmelding?

[melsk-r]

This comment originally might have been created by someone else.

foutmelding is beter. Anders moet je zelf uitzoeken waarom je geen resultaat krijg

[melsk-r]

This comment originally might have been created by someone else.

In principe is het zoeken met min > max niet fout. De minimale grens is 1 en maximale grens is 999999. Als je zoekt met een min en max waarde waarbij min < max dan zoek je tussen min en max. Maar als je zoekt met een min en max waarde waarbij min > max, dan zoek je van 1 tot max en van min tot 999999. Dit is destijds bij het maken van de API specificatie ook besproken. Als min=max dan vind je dus alleen vbo's met een oppervlakte die gelijk is aan min of max. In dit geval is m.i. terecht dat er geen foutmelding komt tenzij max natuurlijk > 999999 en min < 1.

[melsk-r]

This comment originally might have been created by someone else.

Besproken met @JohanBoer: min mag toch niet groter zijn dan max. Dan rest de vraag: welke foutmelding moeten we dan geven? Er is op dit moment geen foutmelding in de foutafhandeling feature die hier past.

[melsk-r]

This comment originally might have been created by someone else.

@strijm

welke foutmelding moeten we dan geven?

Bijvoorbeeld:

{
    "type": "https://tools.ietf.org/html/rfc7231#section-6.5.1",
    "title": "Een of meerdere parameters zijn niet correct.",
    "status": 400,
    "detail": "Bad request.",
    "instance": "...",
    "code": "paramsValidation",
    "invalidParams": [
        {
            "name": "oppervlakte[min]",
            "code": "range",
            "reason": "min mag niet hoger zijn dan max."
        }
    ]
}

@MelvLee mee eens? Of heb je een ander idee voor de fouttekst?

[melsk-r]

This comment originally might have been created by someone else.

OK, dan implementeren we dat, ik heb de foutmelding toegevoegd aan de BAG foutmelding.feature, zie #493. @fsamwel en @MelvLee graag even reviewen