Skip to content

[BUGFIX] subfields#1226

Merged
pstokkink merged 1 commit into
masterfrom
pieter/bugfix-subfields
May 4, 2026
Merged

[BUGFIX] subfields#1226
pstokkink merged 1 commit into
masterfrom
pieter/bugfix-subfields

Conversation

@pstokkink
Copy link
Copy Markdown
Contributor

field.name gebruikt alleen de parent prefix als die parent geen relatie is. Dit zorgde ervoor dat als:

  1. een veld een relatie is met subvelden
  2. en de id van het subveld niet overeenkomt met het id van een van de velden in de tabel
    er een 500 error kwam omdat het ongeprefixte veld werd opgevraagd. In het specifieke geval van meetbouten ging dat fout omdat er een aantal velden met subvelden was met een volgnummer, terwijl meetbouten zelf dat veld niet hebben.

Heb geen zeer specifieke test ervoor geschreven, omdat het uitbreiden van de bestaande test al laat zien dat subvelden goed doorkomen.

field.name only prefixes parent_field name when it is not a relation
@pstokkink pstokkink merged commit ccef293 into master May 4, 2026
7 checks passed
@pstokkink pstokkink deleted the pieter/bugfix-subfields branch May 4, 2026 09:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants