Skip to content

Commit 0a26fec

Browse files
authored
Merge pull request #353 from nl-portal/feature/fix-kvk-adres-optional-fields
something went wrong if straatnaam or huisnummer is not available in …
2 parents 8223bf2 + 550053e commit 0a26fec

File tree

2 files changed

+57
-49
lines changed

2 files changed

+57
-49
lines changed

haalcentraal/haalcentraal-hr/src/main/kotlin/nl/nlportal/haalcentraal/hr/domain/Adres.kt

+3-3
Original file line numberDiff line numberDiff line change
@@ -19,10 +19,10 @@ data class Adres(
1919
val type: String,
2020
val indAfgeschermd: String,
2121
val volledigAdres: String,
22-
val straatnaam: String,
22+
val straatnaam: String?,
2323
val postcode: String,
24-
val postbusnummer: Int,
24+
val postbusnummer: Int?,
2525
val plaats: String,
2626
val land: String,
27-
val huisnummer: Int,
27+
val huisnummer: Int?,
2828
)

haalcentraal/haalcentraal-hr/src/test/resources/data/get-maatschappelijke-activiteiten.json

+54-46
Original file line numberDiff line numberDiff line change
@@ -2,105 +2,113 @@
22
"kvkNummer": "90012768",
33
"indNonMailing": "Ja",
44
"naam": "Test bedrijf",
5-
"formeleRegistratiedatum": "20170710",
5+
"formeleRegistratiedatum": "20161124",
66
"materieleRegistratie": {
7-
"datumAanvang": "20170108"
7+
"datumAanvang": "20161124"
88
},
99
"totaalWerkzamePersonen": 1,
1010
"handelsnamen": [
1111
{
12-
"naam": "Test EMZ Dagobert",
12+
"naam": "Test bedrijf",
1313
"volgorde": 0
1414
},
1515
{
16-
"naam": "Tweede handelsnaam 1MZ",
16+
"naam": "Rappórt Frans van den Broek",
1717
"volgorde": 1
18-
},
19-
{
20-
"naam": "Derde handelsnaam 1MZ",
21-
"volgorde": 2
22-
},
23-
{
24-
"naam": "Vierde handelsnaam 1MZ",
25-
"volgorde": 3
2618
}
2719
],
2820
"sbiActiviteiten": [
2921
{
30-
"sbiCode": "1011",
31-
"sbiOmschrijving": "Slachterijen (geen pluimvee-)",
22+
"sbiCode": "70222",
23+
"sbiOmschrijving": "Advisering op het gebied van management en bedrijfsvoering (geen public relations en organisatie- adviesbureaus)",
3224
"indHoofdactiviteit": "Ja"
3325
},
3426
{
35-
"sbiCode": "1012",
36-
"sbiOmschrijving": "Pluimveeslachterijen",
27+
"sbiCode": "69202",
28+
"sbiOmschrijving": "Accountants-administratieconsulenten",
3729
"indHoofdactiviteit": "Nee"
3830
},
3931
{
40-
"sbiCode": "1013",
41-
"sbiOmschrijving": "Vleesverwerking (niet tot maaltijden)",
32+
"sbiCode": "8211",
33+
"sbiOmschrijving": "Brede administratieve dienstverlening",
4234
"indHoofdactiviteit": "Nee"
4335
}
4436
],
45-
"_links": {
46-
"self": {
47-
"href": "https://api.kvk.nl/test/api/v1/basisprofielen/69599084"
37+
"links": [
38+
{
39+
"rel": "self",
40+
"href": "https://api.kvk.nl/api/v1/basisprofielen/90012768"
4841
},
49-
"vestigingen": {
50-
"href": "https://api.kvk.nl/test/api/v1/basisprofielen/69599084/vestigingen"
42+
{
43+
"rel": "vestigingen",
44+
"href": "https://api.kvk.nl/api/v1/basisprofielen/90012768/vestigingen"
5145
}
52-
},
46+
],
5347
"_embedded": {
5448
"hoofdvestiging": {
5549
"vestigingsnummer": "000038509504",
5650
"kvkNummer": "69599084",
57-
"formeleRegistratiedatum": "20200828",
51+
"formeleRegistratiedatum": "20161124",
5852
"materieleRegistratie": {
59-
"datumAanvang": "20150828"
53+
"datumAanvang": "20161124"
6054
},
61-
"eersteHandelsnaam": "Test EMZ Dagobert",
55+
"eersteHandelsnaam": "Test bedrijf",
6256
"indHoofdvestiging": "Ja",
6357
"indCommercieleVestiging": "Ja",
6458
"totaalWerkzamePersonen": 1,
6559
"adressen": [
60+
{
61+
"type": "correspondentieadres",
62+
"indAfgeschermd": "Nee",
63+
"volledigAdres": "Postbus 1000 2260BA LEIDSCHENDAM",
64+
"postcode": "2260BA",
65+
"postbusnummer": 1000,
66+
"plaats": "LEIDSCHENDAM",
67+
"land": "Nederland"
68+
},
6669
{
6770
"type": "bezoekadres",
6871
"indAfgeschermd": "Nee",
69-
"volledigAdres": "Abebe Bikilalaan 17 M 1034WL Amsterdam",
70-
"straatnaam": "Abebe Bikilalaan",
71-
"huisnummer": 17,
72-
"huisletter": "M",
73-
"postcode": "1034WL",
74-
"plaats": "Amsterdam",
72+
"volledigAdres": "Genemuidenstraat 208 2545NZ 's-Gravenhage",
73+
"straatnaam": "Genemuidenstraat",
74+
"huisnummer": 208,
75+
"postcode": "2545NZ",
76+
"plaats": "'s-Gravenhage",
7577
"land": "Nederland"
7678
}
7779
],
78-
"_links": {
79-
"self": {
80+
"links": [
81+
{
82+
"rel": "self",
8083
"href": "https://api.kvk.nl/test/api/v1/basisprofielen/69599084/hoofdvestiging"
8184
},
82-
"vestigingen": {
85+
{
86+
"rel": "vestigingen",
8387
"href": "https://api.kvk.nl/test/api/v1/basisprofielen/69599084/vestigingen"
8488
},
85-
"basisprofiel": {
89+
{
90+
"rel": "basisprofiel",
8691
"href": "https://api.kvk.nl/test/api/v1/basisprofielen/69599084"
8792
},
88-
"vestigingsprofiel": {
93+
{
94+
"rel": "vestigingsprofiel",
8995
"href": "https://api.kvk.nl/test/api/v1/vestigingsprofielen/000038509504"
9096
}
91-
}
97+
]
9298
},
9399
"eigenaar": {
94100
"rechtsvorm": "Eenmanszaak",
95101
"uitgebreideRechtsvorm": "Eenmanszaak",
96-
"_links": {
97-
"self": {
98-
"href": "https://api.kvk.nl/test/api/v1/basisprofielen/69599084/eigenaar"
102+
"links": [
103+
{
104+
"rel": "self",
105+
"href": "https://api.kvk.nl/api/v1/basisprofielen/69599084/eigenaar"
99106
},
100-
"basisprofiel": {
101-
"href": "https://api.kvk.nl/test/api/v1/basisprofielen/69599084"
107+
{
108+
"rel": "basisprofiel",
109+
"href": "https://api.kvk.nl/api/v1/basisprofielen/69599084"
102110
}
103-
}
111+
]
104112
}
105113
}
106-
}
114+
}

0 commit comments

Comments
 (0)