melsk-r / HC-BAG-bevragen-issues

0 stars 0 forks source link

isNevenadres is niet optioneel #301

Open melsk-r opened 3 months ago

melsk-r commented 3 months ago

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

Het viel op dat bij AdresUitgebreid in het response geen isNevenadres meekomt als het om een hoofdadres gaat, zoals bij 1883200000036667. Ik vroeg me af waarom er voor is gekozen om bij een false geen waarde te tonen. Is het juist niet de bedoeling om hier eenduidig over te zijn? Daarmee dus altijd true of false te tonen.

melsk-r commented 3 months ago

This comment originally might have been created by someone else.

Het gaat verder dan alleen één property. Als je dit wilt, dan zou je het ook moeten doen voor de properties van AdresMogelijkOnjuist. Dat is volgens mij geen goed idee. Ik denk dat het handiger is om te controleren of een adres niet onjuist is door alleen te kijken of de mogelijkOnjuist property null is in plaats van alle AdresMogelijkOnjuist properties langslopen om dan te concluderen dat het adres niet onjuist is.

Verder zijn wij er van uit gegaan dat situaties als isNevenadres, geconstateerd, mogelijkOnjuist veel minder vaak voorkomt. Als je dan false meegeeft voor deze properties, dan zou je de payload alleen maar vullen met veel niet relevante data.