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

Contao 4 #23

Open daniel-berger-nemeth opened 5 years ago

daniel-berger-nemeth commented 5 years ago

Hallo! Da es nunmehr MM für Contao 4 gibt wäre meine Frage, ob es Pläne gibt, deine Erweiterung entsprechend für C4 zu adaptieren?

Danke für weitere Infos, LG Daniel

der-On commented 5 years ago

@daniel-nemeth Von Meiner Seite her vermutlich erst, wenn ich das nächste Contao 4 Projekt mit OpenImmo habe. Dies könnte allerdings etwas dauern. Welche Schritte wären denn theoretisch nötig, um es für Contao 4 anzupassen?

daniel-berger-nemeth commented 5 years ago

Ich werd mal ne kopie ziehen und nur die composer.json so bearbeiten, das diese mit contao 4 kompatibel ist. Dann test ich das ganze mal.

Danke!

cmichael-de commented 4 years ago

Ich habe es unter Contao 4.4.49 getestet. Die XML und ZIP Files werden eingelesen und die Objekte im Catalog soweit synchronisiert.

Probleme gibt es (bei mir) noch bei den Bildern und dem Panorama Snippet.

Arbeite nicht so häufig mit MetaModels von daher weiß ich nicht ob die Probleme nur eine Konfigurationsfrage sind oder ob es Probleme hinsichtlich der Contao 4.4.x Unterstützung sind.

In meiner XML Export Datei (von onoffice) steht ein externer Pfad bei den Bildern zu onoffice. Ich habe versucht herauszufinden, ob MetaModels-OpenImmo externe Bilder selbständig herunterlädt und im angegeben Bilder Ordner speichert - scheinbar ist das nicht der Fall - kann das jemand bestätigen? Am liebsten würde ich den externen Pfad im Catalog speichern. Da hatte ich allerdings schon Probleme beim Panorama Snippet.

Auch die Panorama-Ansicht ist nur ein externer Pfad. Das "pre-made template" legt das Panorama Feld an. Ich konnte aber keine Verknüpfung finden und habe selbst eine angelegt (siehe Screenshot). Leider wird der Pfad aber nicht beim synchronisieren gespeichert. Habe mal testweise einen Standardwert "blabla" in die Verküpfung eingetragen, dieser wird ordentlich im Calaog gespeichert. Habe den Metamodels Cache geleert aber die Verknüpfung speichert den Wert im XML leider nicht im Catalog:

image

Ich vermute, dass es mit der Bedingung zusammenhängt. Denn auch die Bilder werden mit einer Verknüpfung importiert, die eine ähnliche Bedingung haben. Hatte die Bilder mal auf Langtext (statt) Datei umgestellt und probiert, ob da Pfade ankommen - leider negativ.

Ist hier eine Änderung im Modul selbst erforderlich oder mache ich bei der Konfiguration einen Fehler?

der-On commented 4 years ago

@cmichael-de Die extension geht davon aus, dass Bilder in der Zip mitgeliefert werden und versucht diese lokal zu speichern und als File-Model in der Datenbank abzuspeichern. Externe Bildquellen werden aktuell noch nicht verarbeitet. Da ich aktuell Contao nicht mehr nutze, werde ich in nächster Zeit nicht dazu kommen dieses Feature nachzurüsten. Ich nehme aber PRs dankend entgegen.

Ein paar Anhaltspunkte kann ich geben:

  1. XML Export Datei genauer ansehen und rausfinden in welchem Attribut und welchem Knoten der Externe Pfad gespeichert ist.
  2. Gggf. diesen Knoten in die Definitionsliste mit aufnehmen: https://github.com/der-On/Contao-MetaModels-OpenImmo/blob/master/src/MetaModelsOpenImmo/OpenImmo.php#L61
  3. Eventuell hilft schon die Verwendung eines Field Callbacks: https://github.com/der-On/Contao-MetaModels-OpenImmo#field-callbacks oder eines Hooks: https://github.com/der-On/Contao-MetaModels-OpenImmo#hooks um diesen Sonderfall zu handhaben und die externe Adresse in der Datenbank zu speichern.