Imvertor / Imvertor-Maven

Mavenized version of Imvertor
http://www.imvertor.org
9 stars 12 forks source link

Elimineer oude manier van json en yaml generatie #557

Open melsk-r opened 2 weeks ago

melsk-r commented 2 weeks ago

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

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 commented 2 weeks ago

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.

ArjanLoeffen commented 2 weeks ago

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

melsk-r commented 2 weeks ago

Wordt de functie dan nog ergens gebruikt?

ThiesMesdag commented 4 days ago

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?