openoereb / mgdm2oereb

House of the rules to trafo the (ili oereb) world
0 stars 0 forks source link

Bei KbS Transferstrukutr fehlt im Amt das Element AmtImWeb #47

Closed romefi closed 1 year ago

romefi commented 1 year ago

@vvmruder Bei der Trafo vom KbS fehlt das AmtImWeb in der Klasse Amt: image

Die AmtImWeb sollte aus dem MGDM Belastete_Standorte.ZustaendigkeitKataster.URL_Kataster (ist ein Multilingual-Element) übernommen werden.

Nach ersten Tests könnte das auch den Fehler An empty sequence is not allowed as the first argument of decodeURL()] bei der PDF-Aufbereitung auslösen.

vvmruder commented 1 year ago

@romefi Ich schaue mir das an.

vvmruder commented 1 year ago

@romefi Ich habe den Fehler behoben und bereits neu ausgerollt. Ihr könnt testen.

romefi commented 1 year ago

@vvmruder Hab ein durcheinander gemacht. Statt URL_Kataster muss die URL_Behoerde rein.

Ich hab das mal selber versucht und ein Pull-Request gemacht: https://github.com/openoereb/mgdm2oereb/pull/50

Ausserdem habe ich noch beim Link in den Dokumenten einen Fehler entdeckt (auch wenn es der Filterfunktion widerspricht) und auch versucht zu korrigieren, Pull-Request wäre: https://github.com/openoereb/mgdm2oereb/pull/49

Könntest du diese zwei Pull-Requests anschauen und falls OK mergen?

Mit den vorhandenen AmtImWeb tritt der decodeURL()-Fehler nicht mehr auf. Die Ziellinie kommt immer näher.