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-383] voeg kolommen tijdstipaanbieding_stuk en tijdstipaanbieding2 toe aan views #2303

Merged
merged 1 commit into from
Nov 15, 2024

Conversation

mprins
Copy link
Member

@mprins mprins commented Nov 14, 2024

Voegt de kolommen tijdstipaanbieding_stuk en tijdstipaanbieding_stuk2 toe aan views:

  • mb_onroerendezakenmetrechthebbenden
  • mb_avg_onroerendezakenmetrechthebbenden
  • upgrade procedures Postgis / Oracle (zie hieronder )

resolve BRMO-383

uitleg

Er zijn 2 kolommen voor tijdstip aanbieding aangezien er meerdere gebaseerd op relaties kunnen zijn.

Meerdere ‘is gebaseerd op’ bij een zakelijk recht is een valide situatie.
Dit komt voor als het zakelijk recht eerst ontstaat onder opschortende voorwaarden (ontstaat dan nog niet in de BRK) en er later een stuk komt waarin deze opschortende voorwaarden in vervulling gaan.
Bij het 2e stuk ontstaat dan het zakelijk recht in de BRK, maar beide stukken worden dan als ‘is gebaseerd op’ geregistreerd.
Daarnaast lijken er gevallen te zijn met meerdere gebaseerd op stukdelen die te maken hebben met fouten uit de conversie van AKR naar Koers en/of met verkeerd herstel. Dit zal worden gecorrigeerd

Onder andere in de volgende gevallen kan een tenaamstelling gebaseerd zijn op meerdere stukdelen:

  • In sommige stukdelen worden de ‘gebaseerd op’ relaties meegenomen van de oude tenaamstellingen naar de nieuwe tenaamstellingen, omdat de oude titels nog relevant zijn. Bv. Verklaring van erfrecht.
  • Bij vermengen van tenaamstellingen kunnen meerdere ‘gebaseerd op’ relaties ontstaan. Bv. 1/2 verkregen obv stuk1 en 1/2 verkregen obv stuk2
  • Bij verenigen van percelen kunnen meerdere ‘gebaseerd op’ relaties ontstaan. Bv. perceel1 verkregen obv stuk1 en perceel2 verkregen obv stuk2
  • Bij beëindigen van een beperkt recht krijgen de tenaamstellingen van het onderliggend recht het te verwerken stuk als extra ‘is gebaseerd op’, omdat dit ook een vorm van verkrijging is. Bv. Afstand beperkt recht.

Voor een exact antwoord is contact met het Kadaster nodig.

upgrade procedure

Er is geen scripted upgrade procedure voor de aanpassingen in de materialized views mb_onroerendezakenmetrechthebbenden en mb_avg_onroerendezakenmetrechthebbenden BRMO-383.
Deze "extra" views zijn afhankelijk van "extra" views in het BAG schema dat mogelijk nog niet is aangemaakt en zijn mogelijk helemaal niet aanwezing het RSGB schema. Hieronder de stappen vermeld voor de upgrade indien alle "extra" (oa. uit 209_bag2_rsgb_views.sql) views zijn aangemaakt:

Drop de betreffende materialized views uit het RSGB schema met onderstaande statements

DROP MATERIALIZED VIEW mb_onroerendezakenmetrechthebbenden;
DROP MATERIALIZED VIEW mb_avg_onroerendezakenmetrechthebbenden;

Maak daarna de materialized views en commentaar opnieuw aan met de database specifieke code uit db/rsgb/oracle/210_bag2_brk2.0_mat_views.sql#187-591 dan wel db/rsgb/postgresql/210_bag2_brk2.0_mat_views.sql#152-444 (let op er zijn meerdere views in het bestand gedefinieerd).

Copy link

codecov bot commented Nov 14, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 38%. Comparing base (42ecd32) to head (5042c2b).
Report is 3 commits behind head on master.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff            @@
##             master   #2303   +/-   ##
========================================
- Coverage        38%     38%   -0%     
  Complexity      806     806           
========================================
  Files           371     371           
  Lines         18863   18863           
  Branches       1777    1777           
========================================
- Hits           7155    7153    -2     
- Misses        11136   11138    +2     
  Partials        572     572           

see 2 files with indirect coverage changes

@mprins mprins force-pushed the BRMO-383_stuk.tijdstipaanbieding branch from e5391ef to 5046d04 Compare November 15, 2024 09:11
@mprins mprins changed the title [BRMO-383] voeg kolommen tijdstipaanbieding en tijdstipaanbieding2 toe aan views [BRMO-383] voeg kolommen tijdstipaanbieding_stuk en tijdstipaanbieding2 toe aan views Nov 15, 2024
@mprins mprins force-pushed the BRMO-383_stuk.tijdstipaanbieding branch from 5046d04 to 86086fa Compare November 15, 2024 11:09
@mprins mprins marked this pull request as ready for review November 15, 2024 11:09
…e aan view mb_onroerendezakenmetrechthebbenden en mb_avg_onroerendezakenmetrechthebbenden
@mprins mprins force-pushed the BRMO-383_stuk.tijdstipaanbieding branch from 86086fa to 5042c2b Compare November 15, 2024 11:09
@mprins mprins merged commit b3d4003 into master Nov 15, 2024
28 of 30 checks passed
@mprins mprins deleted the BRMO-383_stuk.tijdstipaanbieding branch November 15, 2024 12:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant