Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

BAG: INSERT statements voor adresseerbaarobjectnevenadres leiden tot fout #251

Closed
MartijnVdS opened this issue May 9, 2018 · 1 comment

Comments

@MartijnVdS
Copy link

Bij het verwerken van de dagelijkse BAG-mutaties, zagen we een fout met betrekking tot "adresseerbaarobjectnevenadres". Dit blijkt te komen doordat de geldige waardes voor ligplaatsstatus, standplaatsstatus en verblijfsobjectstatus in enums gedefinieerd zijn, en de software een lege string probeerde te inserten.

In bagattribuut.py (BAGrelatieAttribuu -> maakInsertSQL) worden extra attributen van de verschillende BAG-objecten toegevoegd aan de INSERT SQL:

for attr in self._extraAttributes:
    if self._parent.heeftAttribuut(attr):
        inhoud.append(self._parent.attribuut(attr).waardeSQL())
    else:
        inhoud.append('')

Omdat de "parent" van de "nevenadres"-relatie in "adresseerbaarobjectnevenadres" maar 1 van de 3 velden heeft (ligplaatsstatus, standplaatsstatus, verblijfsobjectstatus), probeert de huidige code hier een INSERT te doen met een lege string als waarde voor de andere 2.

Door in plaats van de lege string, None aan inhoud te appenden, werkt alles weer naar behoren.

"maakCopySQL" maakt van alle lege strings NULLs, en daarom treedt het probleem daar niet op.

@MartijnVdS MartijnVdS changed the title INSERTs op adresseerbaarobjectnevenadres werken niet BAG: INSERT statements voor adresseerbaarobjectnevenadres leiden tot fout May 9, 2018
@justb4
Copy link
Contributor

justb4 commented Aug 7, 2024

Ik sluit deze. Heeft betrekking op deprecated BAG v1 code. Voor BAG v2 moet mutatie-support sowieso nog ontwikkeld. Zie issue #349 en PR #352.

@justb4 justb4 closed this as completed Aug 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants