search

der-On / Contao-MetaModels-OpenImmo

This is a module for the cms contao (www.contao.org) wich allows import of OpenImmo-Data (www.openimmo.de) into the cms with the use of the MetaModels-module.
9 stars 5 forks source link

[1.1.0] bei Import/Sync wird leeres Item in MM erstellt #11

Closed zonky2 closed 8 years ago

zonky2 commented 8 years ago

Beim Import bzw. Synchronisieren wird nur ein leerer Datensatz erstellt.

Ich glaube ich habe den Fehler gefunden - siehe https://github.com/der-On/Contao-MetaModels-OpenImmo/blob/master/src/MetaModelsOpenImmo/MetaModelsOpenImmo.php#L653 => ist der Knoten nicht vom Typ "Dateipfad" - also für alle "normalen Textwerte" ist an der Stelle results immer leer...

siehe PR #11

Warum bekommt pid den Wert seiner eigenen MM-ID? siehe https://github.com/der-On/Contao-MetaModels-OpenImmo/blob/master/src/MetaModelsOpenImmo/MetaModelsOpenImmo.php#L488

der-On commented 8 years ago

Danke nochmal für den hotfix. Ich habe an der Stelle kürzlich den Code umgebaut und dabei kaputt gemacht. Das Modul braucht an dieversen Stellen einen ordentlichen Refacture und vorallemn automatische Tests.

PID entspricht hier dem metamodel, was syntaktisch ja schon das "Elternelement" eines Immobilienobjektes ist.

zonky2 commented 8 years ago

PR #12