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-374|BRMO-231] Opruimen BRK 1 transformatie xsl en aanliggende code ⛓️‍💥 #2317

Merged
merged 8 commits into from
Nov 28, 2024

Conversation

mprins
Copy link
Member

@mprins mprins commented Nov 26, 2024

De mogelijkheid om BRK Levering 1 data te laden of te verwerken wordt hiermee verwijderd.

  • Verwijder BRKDirectoryScanner en BRKScannerProces automatisch proces uit code
  • Verwijder de Geavanceerde Functies voor BRK 1. Verwijdered:
    • "Repareren BRK mutaties met status STAGING_NOK"
    • "Exporteren BRK mutaties"
    • "Herhaal transformatie BRK verwijderberichten, oplossen achtergebleven 'kad_onrrnd_zk' records"
    • "Vul de 'herstelde bestandsnaam' van BRK laadprocessen"
    • "Verwijderen van BRK berichten met status ARCHIVE"
    • "Opschonen en archiveren van BRK berichten met status RSGB_OK, ouder dan 3 maanden"
  • Verwijder BR_BRK bericht soort
  • Verwijder snelle updates:
    • "Toevoegen BSN aan ingeschreven natuurlijk persoon"
    • "Toevoegen RSIN aan ingeschreven niet-natuurlijk persoon"
    • "Bijwerken van omschrijving, datum en ref_id in brondocument"
    • "Bijwerken van onvolledig adres"
    • "Bijwerken van rechthebbende VVE op zakelijk recht"
    • "Bijwerken van GBA Niet Ingezetene 'clazz' in personen tabellen"
    • "Bijwerken subject adres comfort data"
    • "Bijwerken ingangsdatum_recht zakelijk recht"
  • Verwijder niet langer gebruikte XSLT stylesheets. Opruimen van:
    • update-brondocument.xsl
    • update-bsn.xsl
    • update-comfort-adres.xsl
    • update-incompleetadres.xsl
    • update-niet_ingezetene-clazz.xsl
    • update-rsin.xsl
    • update-zak_recht-begindatum.xsl
    • update-zak_recht-vve.xsl
  • Verwijder alle verwijzingen naar BRK 1 uit de code
  • Verwijder brk opties uit alle configuraties
  • Update Automatische Processen documentatie
  • Update Geavanceerde functies documentatie
  • Update Snelle updates documentatie
  • Upgrade scripts om BRKScannerProces en BRK v1 GDS2OphaalProces op te ruimen
  • Update release notes: https://github.com/B3Partners/brmo/wiki/Upgrade-v4.0.0-naar-v5.0.0

dit is de laatste stap in BRMO-231

Copy link

codecov bot commented Nov 26, 2024

Codecov Report

Attention: Patch coverage is 12.50000% with 42 lines in your changes missing coverage. Please review.

Project coverage is 40%. Comparing base (5deeca2) to head (67aaa7c).
Report is 24 commits behind head on master.

Files with missing lines Patch % Lines
...brmo/service/scanner/AbstractExecutableProces.java 0% 15 Missing ⚠️
...p/brmo/service/stripes/OphaalConfigActionBean.java 0% 15 Missing ⚠️
...nl/b3p/brmo/service/scanner/GDS2OphalenProces.java 0% 10 Missing ⚠️
...er/src/main/java/nl/b3p/brmo/loader/RsgbProxy.java 72% 1 Missing and 1 partial ⚠️
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff            @@
##             master   #2317    +/-   ##
=========================================
+ Coverage        38%     40%    +2%     
+ Complexity      806     805     -1     
=========================================
  Files           371     367     -4     
  Lines         18878   18159   -719     
  Branches       1777    1689    -88     
=========================================
- Hits           7162    7158     -4     
+ Misses        11143   10430   -713     
+ Partials        573     571     -2     
Files with missing lines Coverage Δ
...rc/main/java/nl/b3p/brmo/loader/BrmoFramework.java 27% <ø> (ø)
...src/main/java/nl/b3p/brmo/loader/StagingProxy.java 58% <100%> (+1%) ⬆️
...brmo/service/proxy/BerichtEndpointFileServlet.java 66% <ø> (ø)
...rmo/service/proxy/BerichtEndpointProxyServlet.java 0% <ø> (ø)
.../nl/b3p/brmo/service/scanner/ProcesExecutable.java 0% <ø> (ø)
...o/service/stripes/AdvancedFunctionsActionBean.java 0% <ø> (ø)
...nl/b3p/brmo/service/stripes/UpdatesActionBean.java 0% <ø> (ø)
...er/src/main/java/nl/b3p/brmo/loader/RsgbProxy.java 70% <72%> (+2%) ⬆️
...nl/b3p/brmo/service/scanner/GDS2OphalenProces.java 0% <0%> (ø)
...brmo/service/scanner/AbstractExecutableProces.java 7% <0%> (+1%) ⬆️
... and 1 more

@mprins mprins changed the title WIP [BRMO-374] Opruimen BRK 1 transformatie xsl en aanliggende code [BRMO-374] Opruimen BRK 1 transformatie xsl en aanliggende code Nov 26, 2024
Copy link
Member Author

@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.

is blocked by #2318

@mprins mprins marked this pull request as ready for review November 26, 2024 22:24
@mprins mprins force-pushed the BRMO-374_opruimen_brk1_verwerking branch from 88dbad0 to 8c79038 Compare November 28, 2024 07:44
Verwijder "Repareren BRK mutaties met status STAGING_NOK", "Exporteren BRK mutaties", "Herhaal transformatie BRK verwijderberichten, oplossen achtergebleven 'kad_onrrnd_zk' records", "Vul de 'herstelde bestandsnaam' van BRK laadprocessen", "Verwijderen van BRK berichten met status ARCHIVE" uit `AdvancedFunctionsActionBean`
- Verwijder geavanceerde functie "Opschonen en archiveren van BRK berichten met status RSGB_OK, ouder dan 3 maanden"
- Verwijder snelle updates: "Toevoegen BSN aan ingeschreven natuurlijk persoon", "Toevoegen RSIN aan ingeschreven niet-natuurlijk persoon", "Bijwerken van omschrijving, datum en ref_id in brondocument", "Bijwerken van onvolledig adres", "Bijwerken van rechthebbende VVE op zakelijk recht", "Bijwerken van GBA Niet Ingezetene 'clazz' in personen tabellen", "Bijwerken subject adres comfort data", "Bijwerken ingangsdatum_recht zakelijk recht"
Opruimen van:

- update-brondocument.xsl
- update-bsn.xsl
- update-comfort-adres.xsl
- update-incompleetadres.xsl
- update-niet_ingezetene-clazz.xsl
- update-rsin.xsl
- update-zak_recht-begindatum.xsl
- update-zak_recht-vve.xsl
@mprins mprins force-pushed the BRMO-374_opruimen_brk1_verwerking branch from 8c79038 to 67aaa7c Compare November 28, 2024 07:46
@mprins mprins requested a review from Copilot November 28, 2024 08:20

Choose a reason for hiding this comment

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

Copilot reviewed 27 out of 42 changed files in this pull request and generated no suggestions.

Files not reviewed (15)
  • brmo-loader/src/main/resources/xsl/update-brondocument.xsl: Language not supported
  • brmo-loader/src/main/resources/xsl/update-bsn.xsl: Language not supported
  • brmo-loader/src/main/resources/xsl/update-comfort-adres.xsl: Language not supported
  • brmo-loader/src/main/resources/xsl/update-incompleetadres.xsl: Language not supported
  • brmo-loader/src/main/resources/xsl/update-niet_ingezetene-clazz.xsl: Language not supported
  • brmo-loader/src/main/resources/xsl/update-rsin.xsl: Language not supported
  • brmo-loader/src/main/resources/xsl/update-zak_recht-begindatum.xsl: Language not supported
  • brmo-loader/src/main/resources/xsl/update-zak_recht-vve.xsl: Language not supported
  • brmo-persistence/src/main/resources/META-INF/persistence.xml: Language not supported
  • brmo-persistence/src/test/resources/META-INF/persistence.xml: Language not supported
  • brmo-proxyservice/src/main/webapp/WEB-INF/web.xml: Language not supported
  • brmo-loader/src/main/java/nl/b3p/brmo/loader/entity/BrkBericht.java: Evaluated as low risk
  • brmo-loader/src/main/java/nl/b3p/brmo/loader/xml/BrkSnapshotXMLReader.java: Evaluated as low risk
  • brmo-persistence/src/main/java/nl/b3p/brmo/persistence/staging/BRKScannerProces.java: Evaluated as low risk
  • brmo-loader/src/main/java/nl/b3p/brmo/loader/BrmoFramework.java: Evaluated as low risk
@mprins mprins changed the title [BRMO-374] Opruimen BRK 1 transformatie xsl en aanliggende code [BRMO-374|BRMO-231] Opruimen BRK 1 transformatie xsl en aanliggende code Nov 28, 2024
@mprins mprins merged commit 56243f9 into master Nov 28, 2024
26 of 29 checks passed
@mprins mprins deleted the BRMO-374_opruimen_brk1_verwerking branch November 28, 2024 09:04
@mprins mprins added the breaking vergt extra aandacht bij upgrade label Dec 10, 2024
@mprins mprins changed the title [BRMO-374|BRMO-231] Opruimen BRK 1 transformatie xsl en aanliggende code ⛓️‍💥 [BRMO-374|BRMO-231] Opruimen BRK 1 transformatie xsl en aanliggende code ⛓️‍💥 Dec 10, 2024
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