svoop / aipp

AIP parser to convert AIP publications to AIXM or OFMX
https://bitcetera.com
MIT License
10 stars 2 forks source link

LFMN TWR - TWR is listed twice #6

Closed svoop closed 4 years ago

svoop commented 5 years ago

Reported by Stijn

svoop commented 4 years ago

The unit is created twice e.g. once for the service TWR and once again for the service ATIS.

Screen Shot 2019-10-11 at 22 22 01
svoop commented 4 years ago

@ovord Das Problem hier: Bei Units ist nur der Name Teil der UniUid, wenn also der codeType nicht im Namen der Unit mit drinne steht, dann wird es bei mehreren Types Fehler geben, da sie alle dieselbe UniUid haben. Ich kann es entweder so lassen, dass die Abkürzung im Namen drin bleibt LFMN TWR oder ich schreibe es an der Stelle aus zu LFMN TOWER. Letzteres macht es aber schwieriger, wenn du es im Editor ausfiltern möchtest.

Was meinst du?

svoop commented 4 years ago

Ich habe den Uni->codeType in die UniUid verschoben (c7a66a6 und https://github.com/svoop/aixm/commit/4ced63dfc90028ba82cb7a3649cc3ecc81506223).

@ovord Das bedingt eine Anpassung am Editor, ich habe im Issue https://github.com/openflightmaps/ofmx/issues/1 einen entsprechenden Punkt unter Uni hinzugefügt. Nicht vergessen: Dazu war eine Mutation an den OFMX XSL nötig, falls du die also nicht per URL live beziehst, musst du sie lokal mit den aktuellen XSL ersetzen.

Screen Shot 2019-11-10 at 20 32 02