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

aktenummer wordt gebruikt ondanks indicatie onjuist #129

Closed
fsamwel opened this issue Oct 10, 2024 · 2 comments · Fixed by #133
Closed

aktenummer wordt gebruikt ondanks indicatie onjuist #129

fsamwel opened this issue Oct 10, 2024 · 2 comments · Fixed by #133
Assignees
Labels
bug Something isn't working gezag brp api gezag

Comments

@fsamwel
Copy link
Contributor

fsamwel commented Oct 10, 2024

Zie #97 features/gezag-minderjarige/3.1-recente-gebeurtenis-adoptie.feature:150

Hier is eerst adoptie (aktenummer met Q op positie 3) geregistreerd en is datum ingang familierechtelijke betrekking ná de ingangsdatum gezagsverhouding.
Maar deze adoptie is gecorrigeerd, dus op die persoon-rij zit onjuist_ind met een waarde. Die moet je dan negeren.

stapel_nr volg_nr persoon_type burger_service_nr geslachts_naam akte_nr familie_betrek_start_datum onjuist_ind
0 1 1 000000012 Ingrid 1AQ0100 20201011 O
0 1 0 000000012 Ingrid 1AR0100 20201011

Elke rij in de tabel waarbij onjuist_ind is ongelijk aan NULL moet worden genegeerd. Dit geldt voor elke soort gegeven in elke tabel. Alleen voor de inschrijving (tabel lo3_pl) komt dit niet voor en bij een volg_nr==0 komt dit ook niet voor. Dus bij elke query waarbij volg_nr groter kan zijn dan 0 moet je een WHERE clause opnemen onjuist_ind IS NOT NULL

@fsamwel fsamwel added the bug Something isn't working label Oct 10, 2024
@fsamwel fsamwel moved this to Sprint backlog in Scrum Board BRP bevragen Oct 10, 2024
@fsamwel fsamwel added the gezag brp api gezag label Oct 10, 2024
@fsamwel fsamwel changed the title indicatie onjuist op aktenummer adoptie wordt toch gebruikt aktenummer wordt gebruikt ondanks indicatie onjuist Oct 10, 2024
@Patrick-4488 Patrick-4488 linked a pull request Oct 14, 2024 that will close this issue
@github-project-automation github-project-automation bot moved this from In progress to Done in Scrum Board BRP bevragen Oct 14, 2024
@Patrick-4488 Patrick-4488 moved this from Done to Review in Scrum Board BRP bevragen Oct 14, 2024
@FrozenSync
Copy link
Contributor

Ter verduidelijking: de beschrijving bevat wat onduidelijkheid. De laatste zin "... moet je een WHERE clause opnemen onjuist_ind IS NOT NULL" klopt niet met de rest van de beschrijving. Dit moet onjuist_ind IS NULL zijn. De PR is gemaakt op basis van deze aanname.

@fsamwel kan je verifiëren of dit klopt?

@fsamwel
Copy link
Contributor Author

fsamwel commented Oct 15, 2024

Dit moet inderdaad onjuist_ind IS NULL zijn.

@FrozenSync FrozenSync removed their assignment Oct 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working gezag brp api gezag
Projects
Development

Successfully merging a pull request may close this issue.

3 participants