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

Elimineer oude manier van json en yaml generatie #557

Open
melsk-r opened this issue Nov 7, 2024 · 4 comments
Open

Elimineer oude manier van json en yaml generatie #557

melsk-r opened this issue Nov 7, 2024 · 4 comments

Comments

@melsk-r
Copy link
Contributor

melsk-r commented Nov 7, 2024

Op dit moment worden er op 2 verschillende manieren json en yaml files gegenereerd o.b.v. BSM modellen.

  • De eerste manier genereert deze bestanden m.b.v. een XSLT stylesheet.
  • De tweede manier genereert een XML mapping bestand voor JSON welke vervolgens met een Java library wordt omgezet naar json syntax dat vervolgens met een andere Java library naar yaml wordt omgezet.

De tweede manier heeft zich inmiddels bewezen en de eerste manier kan daarom worden uitgefaseerd. De bestanden die met de tweede manier worden gegenereerd kunnen daardoor voortaan in de 'yaml' folder worden geplaatst.

@melsk-r
Copy link
Contributor Author

melsk-r commented Nov 7, 2024

Dit is inmiddels uitgewerkt in de branch 'RobertMelskens-Remove-old-version-of-yaml-creation'. Ik heb hiervoor o.a. in 'src/main/java/nl/imvertor/YamlCompiler/YamlCompiler.java' voor de VNG en voor de VNG volgende organisaties aanpassingen aangebracht. Het is ook door mij succesvol getest en de branch kan dus in principe gemerged worden met de 'Development' branch.

Echter in dat java bestand staat ook nog de specifiek voor het kadaster vervaardigde functie generateKadaster(). Die moet natuurlijk geen nadelige gevolgen ondervinden van mijn wijzigingen. Het is mij alleen niet duidelijk of deze functie nog wel in gebruik is. Het zou kunnen van niet want het lijkt er op dat deze functie nergens meer wordt aangeroepen. Dat moet gecheckt worden.

  • Zo ja, kan iemand dan testen of de in deze branch aangebrachte wijzigingen nadelige gevolgen hebben voor deze functie?
  • Zo nee, mag de functie dan uit het java bestand worden verwijderd?

@ArjanLoeffen
Copy link
Contributor

ArjanLoeffen commented Nov 8, 2024

Ik denk dat we dit even samen moeten doen. En @ThiesMesdag vragen mee te kijken/denken.

@melsk-r
Copy link
Contributor Author

melsk-r commented Nov 8, 2024

Wordt de functie dan nog ergens gebruikt?

@ThiesMesdag
Copy link
Contributor

Ik denk het niet, maar ik zal dat navragen. Hoe kunnen wij die nieuwe manier gebruiken? Of is dat de Json generatie op basis van de MIM-xml?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants