VNG-Realisatie / Haal-Centraal-BRK-bevragen

Project repository tbv de ontwikkeling van de Haal Centraal BRK Bevragen API
https://vng-realisatie.github.io/Haal-Centraal-BRK-bevragen/
Other
19 stars 15 forks source link

Lengte Teller en Noemer velden te kort #959

Closed Marcel-Brons closed 12 months ago

Marcel-Brons commented 1 year ago

In IMKAD zijn Teller en Noemer gedefinieerd als integers van maximaal 32 cijfers lang. In TypeBreuk zijn teller en noemer maximaal 8 cijfers lang. Het issue is dat óf de velden in TypeBreuk te kort zijn gedefinieerd óf dat in de specificatie niet is beschreven hoe wordt omgegaan met waarden die te lang zijn (afkappen, afronden of ...).

CathyDingemanse commented 1 year ago

Dank voor de melding Marcel. Er zit geen ratio van Haal Centraal achter om deze teller en noemer anders te representeren dan in het IMKAD is geregistreerd. M.a.w. er is geen sprake van een bewuste keuze of een bewerking.

@kad-hebbim @kad-pothot wij hebben even in het IMKAD gekeken. Klopt de constatering van Marcel dat het om een integer van max 32 lang zou gaan? Of is dit een notatie voor iets anders?

kad-hebbim commented 1 year ago

Deze constatering is inderdaad juist. We zullen dit aanpassen naar max. 32 cijfers.

kad-hebbim commented 1 year ago

Met Melvin overlegd en waarschijnlijk is de enige oplossing (breaking) om er een String veld van te maken met de volgende patroon: [1-9][0-9]{0,31}

CathyDingemanse commented 1 year ago

Ik vraag mij af of breuken met 32 cijfers lange tellers en noemers voorkomen. @kad-pothot hot, kun jij uitleggen hoe dit in het domein werkt en wat er dan in de werkelijkheid aan de hand als dit in de BRK voorkomt? Je kunt bij 10 cijfers al voor twee miljardste eigenaar zijn :-). Of is dit een fout in het IMKAD (bit= byte), zou dat misschien ook nog kunnen?

fsamwel commented 1 year ago

hoe kom je bij 32 cijfers? In imKad zie ik voor typebreuk dat zowel teller als noemer het type integer hebben. Daarin zit inderdaad niet het minimum (1) en maximum (99999999). Dus als we helemaal volgens imKad gaan, laten we het een integer (int32) en kan het een maximale waarde van 2^32 krijgen, dus maximaal 4.294.967.296. Dus 10 cijfers.

brienen commented 1 year ago

Oplossing: we koersen op een cijfer dat 32 karakters kan bevatten. Marcel van Kadaster toetst of dit mogelijk en wenselijk is

kad-hebbim commented 1 year ago

Even geschakeld met Thijs en het kan een integer (int32) blijven. We gaan de spec hierop aanpassen.

melsk-r commented 1 year ago

Toch nog een vraagje van mij zodat we er uiteindelijk allemaal hetzelfde onder verstaan.

Het type int32 kan maximaal het cijfer '4.294.967.296' bevatten (2^32). Een getal van 10 cijfers lang. Is dat ook wat jullie hier bedoelen of bedoelen jullie een getal van 32 cijfers lang (in feite 10^32)?

kad-hebbim commented 1 year ago

We bedoelen idd een getal van 10 cijfers lang.

brienen commented 1 year ago

De oplossing voor deze bug behelst toepassing van getallen van maximaal 10 cijfers lengte.

kad-hebbim commented 1 year ago

Validatie wordt volgt: