Imvertor / Imvertor-Maven

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

BRO: Referentielijst wordt niet gevonden op GitHub #548

Closed jacobvosimpronotion closed 1 month ago

jacobvosimpronotion commented 1 month ago

De werking van de tag 'Locatie' bij een Referentielijst i.c.m. de tag 'Afkorting' bij een Basismodel is wat betreft het opnemen van die referentielijst in een catalogus anders dan verwacht.

O.b.v. informatie in https://github.com/Imvertor/Imvertor-Maven/issues/540 is die verwachting als volgt:

Dus gegeven:

wordt als bestand verwacht: https://raw.githubusercontent.com/BROprogramma/SLD/gh-pages/lists/Aanpak.xml

Dit bestand is aanwezig.

Maar: bij het genereren van de catalogus (met Imvertor 3.0) verschijnt als foutmelding voor element 'Model::Aanpak = Approach': "Codelist content location not specified xsl/OfficeCompiler/Imvert2modeldoc-lists.xsl / xsl/OfficeCompiler/Imvert2modeldoc-lists.xsl".

N.B.: De 'codelist' is dus hier een Referentielijst.

Wat gaat hier fout? Ik heb ook op het Domein-package de tag 'Afkorting' ingesteld, maar dat was niet de oplossing.

ArjanLoeffen commented 1 month ago

Nee, de naam van de lijst zoals die voorkomt in de EA speelt nu geen rol. Het gaat om de waarde die is ingevoerd als tagged value Locatie, zoals in:

image

We kunnen wel besluiten om de Parameterlijst naam van de referentielijst te gaan gebruiken als alternatief voor de naam of de URL zoals ingevoerd als waarde van de tagged value Locatie. Dat zou een uitbreiding zijn van de logica. Echter, hou er rekening mee dat dit zich dan onmiddellijk vertaalt naar een filenaam. Dat is niet zo fijn, omdat je in je model zelf onafhankelijk wilt zijn van filenaam conventies op het host systeem. Maar daar kunnen we ook wel een afspraak over maken.

Hoe dan ook, mijn voorstel zou zijn de tv Locatie altijd met URL of met een filenaam te vullen. Voor "locatie" is een afhankelijkheid van filenaam conventies wel acceptabel. Dus niets te veranderen: de foutmelding is dan helder en terecht.

jacobvosimpronotion commented 1 month ago

Ik begrijp nu hoe het werkt en dat is voldoende. Aanpassing is (i.i.g. nu) niet nodig.

Bij de referentielijst 'Aanpak' kan ik als Locatie invullen waarna het door Imvertor wordt herkend:

N.B.: Aanpak.xml werkt niet, want dan wordt naar /Aanpak.xml.xml gezocht.

ArjanLoeffen commented 1 month ago

Kijk dat is interessant; ik dacht dat we .xml extensies altijd verwijderden, dus dat .xml.xml niet kan voorkomen. Ik check dat nog even.

ArjanLoeffen commented 1 month ago

De aanpak wat betreft de .xml extensie is als volgt:

Voorbeeld voor BRO: <parameter name="doclist-xml-url">https://raw.githubusercontent.com/BROprogramma/[appinfo/model-abbreviation]/gh-pages/lists/[temp/latest-list-key].xml</parameter>

https://raw.githubusercontent.com/BROprogramma/EPL/gh-pages/lists/GeologischeEenheid.xml wordt gelezen.

GeologischeGrondsoort wordt omgezet naar https://raw.githubusercontent.com/BROprogramma/SAMPLE/gh-pages/lists/GeologischeGrondsoort.xml en die wordt gelezen.

https://raw.githubusercontent.com/BROprogramma/EPL/gh-pages/lists/GeologischeEenheid Hiervan wordt GeologischeEenheid begrepen als de latest-list-key, Die wordt ingevoegd in de doclist-xml-url. Dat levert in de praktijk geen bestaande lijst op: https://raw.githubusercontent.com/BROprogramma/SAMPLE/gh-pages/lists/GeologischeEenheid.xml

De verdubbeling van .xml kan ik dus niet thuisbrengen. Tenzij je alleen "Aanpak.xml" hebt opgevoerd als waarde van Locatie, dan heeft dat het beschreven effect.

jacobvosimpronotion commented 1 month ago

Quote: "Tenzij je alleen "Aanpak.xml" hebt opgevoerd als waarde van Locatie, dan heeft dat het beschreven effect."

Ja, dat schreef ik ook: "Aanpak.xml werkt niet".