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

[BRMO-196] NHR RSGB eenmanszaak fix #1520

Merged
merged 5 commits into from
Sep 27, 2022
Merged

[BRMO-196] NHR RSGB eenmanszaak fix #1520

merged 5 commits into from
Sep 27, 2022

Conversation

puckipedia
Copy link
Contributor

@puckipedia puckipedia commented Sep 14, 2022

Fixt alle heeftAlsEigenaar die niet een rechtspersoon is (bijv. natuurlijkPersoon, buitenlandseVennootschap, samenwerkingsverband), samen met fixes voor de tests (om te controleren dat er altijd een fk_4pes_sc_identif aanwezig is)

resolve BRMO-196

@mprins mprins added this to the 2.3.2 milestone Sep 15, 2022
@mprins mprins changed the title NHR RSGB eenmanszaak fix [BRMO-196] NHR RSGB eenmanszaak fix Sep 15, 2022
Copy link
Member

@mprins mprins left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Voor de testcase "/mantis10752/52019667.xml" wordt een "ingeschreven natuurlijk persoon" aangemaakt zonder dat er enige attributen worden gevuld; dat is onjuist, deze natuurlijk persoon is niet ingeschreven (in de GBA); deze record moet er dus niet zijn.
bovendien is de geslachtsaanduiding in tabel nat_prs onjuist (moet "m" zijn, is "O"); in de tabel subject ontbreken de (buitenlandse) adresgegevens van deze persoon.

(de #2 testcase is gelijk aan de #1 testcase)

Een nieuwe testcase zou goed zijn geweest... vooraf door de anonimizer op https://github.com/B3Partners/bericht-anonimizer halen om persoonsgegevens te verhaspelen

@mprins
Copy link
Member

mprins commented Sep 20, 2022

Er moet ook een "snelle update" komen die de bestaande data problemen oplost.
Zie https://github.com/B3Partners/brmo/blob/master/brmo-loader/src/main/java/nl/b3p/brmo/loader/updates/UpdateProcess.java

@mprins mprins enabled auto-merge (squash) September 27, 2022 14:10
@mprins
Copy link
Member

mprins commented Sep 27, 2022

@puckipedia graag merge conflict oplossen

@mprins mprins merged commit 38a0128 into master Sep 27, 2022
@mprins mprins deleted the nhr-rsgb-fix branch September 27, 2022 15:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants