TweedeKamerDerStaten-Generaal / OpenDataPortaal

GitHub van het officiële Open Data Portaal van de Tweede Kamer der Staten-Generaal.
https://opendata.tweedekamer.nl
49 stars 3 forks source link

Zittende Kamerleden met een spatie in hun roepnaam of achternaam in het Persoon object #152

Open berthubert opened 1 month ago

berthubert commented 1 month ago

De volgende Persoon UUIDs hebben een spatie aan het einde van hun roepnaam: 5c9533b7-a733-480e-bb38-711e96129684 e3f71155-7306-459e-82f8-0b93695b7f7c c8a96d5e-c255-455d-b239-0e8754f7341f 5c9533b7-a733-480e-bb38-711e96129684

Dit geldt ook nog voor twee oud kamerleden: 5c9533b7-a733-480e-bb38-711e96129684 f31cabe4-e1f4-46db-88a9-b46ba44dded4

De bemoeilijkt het matchen vanuit verslagen. Allevier hebben overigens een dubbele voornaam, zoals Diederik Thomas

berthubert commented 1 month ago

En 16d4d7c2-1bf0-46db-b1ff-edf72eb2e4d6 heeft een spatie in zijn achternaam aan het einde.

monty241 commented 1 month ago

Terzijde: in de driver die wij gebruiken worden spaties (whitespace) aan begin en einde naam automatisch verwijderd. Dit vermijdt semantische issues zoals verschil PostgreSQL en Oracle. Dit gaat nog iets verder dan varchar2; "" wordt bijvoorbeeld null.

Op de nationale politieke index hebben we dit probleem niet gehad.

OpenDataPortaal commented 1 month ago

Wij gaan dit doorgeven aan de beheerders van het bronsysteem.

berthubert commented 1 month ago

Dank u!

OpenDataPortaal commented 1 week ago

Dit issue is inmiddels opgelost!

Zoals te zien met onderstaande query komt het nu niet meer voor dat er spaties voorkomen in namen van Personen. https://gegevensmagazijn.tweedekamer.nl/OData/v4/2.0/persoon?$filter=endswith(roepnaam,%20%27%20%27)%20or%20endswith(voornamen,%20%27%20%27)%20or%20endswith(Achternaam,%20%27%20%27)&$count=true

berthubert commented 1 week ago

Super! Kan ik mijn workarounds weghalen. Overigens zijn we er nog net niet helemaal:

sqlite> select * from persoon where tussenvoegsel like '% ';
|                  id                  | skiptoken |      bijgewerkt      |           updated            |   functie    | initialen | tussenvoegsel |   achternaam   |      voornamen       | roepnaam | geboortedatum | geboorteplaats | geboorteland | overlijdensdatum | overlijdensplaats | geslacht |  titels   |                          enclosure                           | contentLength | contentType | woonplaats | land | nummer |
|--------------------------------------|-----------|----------------------|------------------------------|--------------|-----------|---------------|----------------|----------------------|----------|---------------|----------------|--------------|------------------|-------------------|----------|-----------|--------------------------------------------------------------|---------------|-------------|------------|------|--------|
| 5434f6c8-7f54-4e5d-8fb9-c1047122fcd2 | 16661587  | 2023-08-29T11:09:47Z | 2023-08-29T11:21:24.1266464Z | Oud Kamerlid | E.        | van           | Egerschot      | Esther               | Eske     | 1977-05-03    | Delfzijl       | Nederland    |                  |                   | vrouw    | mr.       | https://gegevensmagazijn.tweedekamer.nl/SyncFeed/2.0/Resourc | 21648         | image/jpeg  | Den Haag   | NL   | 2423   |
|                                      |           |                      |                              |              |           |               |                |                      |          |               |                |              |                  |                   |          |           | es/5434f6c8-7f54-4e5d-8fb9-c1047122fcd2                      |               |             |            |      |        |
| 5dfe97b2-fffb-4586-a999-1cbcfde80674 | 16694854  | 2023-08-29T11:10:32Z | 2023-08-29T14:00:11.6425785Z | Oud Kamerlid | T.J.H.    | van den       | Nieuwenhuijzen | Tom Jacobus Henricus | Tom      | 1982-07-18    | Weert          | Nederland    |                  |                   | man      | drs. ing. | https://gegevensmagazijn.tweedekamer.nl/SyncFeed/2.0/Resourc | 48004         | image/jpeg  | Utrecht    | NL   | 46117  |
|                                      |           |                      |                              |              |           |               |                |                      |          |               |                |              |                  |                   |          |           | es/5dfe97b2-fffb-4586-a999-1cbcfde80674                      |               |             |            |      |        |

Maar dank!

OpenDataPortaal commented 1 week ago

Oei, deze hebben wij ook even over het hoofd gezien. Deze gevallen zijn ook nog doorgegeven aan onze beheerders!