search

cgoIT / contao-calendar-ical-bundle

Contao extension calendar_ical
Other
0 stars 4 forks source link

Felderzuordnung beim Import #23

Open kroerig opened 1 year ago

kroerig commented 1 year ago

Was mir aufgefallen ist:

cgoIT commented 1 year ago

Hi @kroerig,

welcher Importer bricht ab? Mit welcher Meldung? Was steht im Log? Wie kann ich das nachstellen? Hast du dazu eine ics-Datei, die ich verwenden kann?

Bitte zukünftig für Bugs das entsprechende Template bei der Erstellung wählen und soweit möglich ausfüllen. Da werden genau diese Infos abgefragt.

kroerig commented 1 year ago

Ich hatte das extra nicht als Bug angelegt, da das Verhalten möglicherweise so gewollt bzw. im Fall des Mailto auch richtig sein könnte.

Die UID Geschichte hattest Du ja schon rausgenommen. Beim MAILTO muss ich mal schauen, ob das bei Teilnehmern möglicherweise ein Pflichtfeld ist, dann wäre das Verhalten richtig. Wobei die Frage wäre ob Contao mit den Feldern Organisator und Teilnehmern überhaupt etwas sinnvolles machen kann.

cgoIT commented 1 year ago

Das war auch nicht als Vorwurf gemeint. ;) Es hilft nur. Und sagen "works as designed" kann man ja immer noch. Ich hatte mal versucht über Google nen Kalender mit Teilnehmern ohne Email-Adresse anzulegen und bin gescheitert. Daher die Frage nach ner Beispiel ics-Datei und dem genauen Fehler.

kroerig commented 1 year ago

Du kannst es mit dieser Datei/URL testen: http://sis.leibniz-remscheid.de/calendar/public.ics

Der Import bricht ab, weil die Teilnehmer einen MAILTO= Eintrag haben. Wie gesagt, es kann durchaus sein, dass dieses Feld nicht leer sein darf. Dann hat die Import-Klasse recht mit dem Fehler.

cgoIT commented 1 year ago

Ich habe den Fehler mit deiner Datei nachstellen können. Die Fehlermeldung kommt nicht von dieser Erweiterung hier sondern von der Bibliothek, die das parsen der ics-Datei übernimmt. Handelt sich dabei um diese Bibliothek: https://github.com/iCalcreator/iCalcreator. Wenn, dann müsste man also dort einen "Fehler" melden. Ich habe selbst den RfC noch nicht gelesen (komme ich im Moment auch zeitlich nicht dazu). Ich weiß nicht, was der zu leeren Email-Adressen sagt. Zumindest meckert auch der iCalendar Validator. Allerdings wegen nicht eindeutiger UIDs. Also (noch) ein anderes Thema.

kroerig commented 1 year ago

Der UID Fehler in der Datei ist jetzt weg und der Validator meldet "No errors found".

cgoIT commented 1 year ago

Dann müsstest du dich für den Rest an den Ersteller des iCalcreators wenden. Der müsste das dann beheben.

kroerig commented 1 year ago

Vermutlich ehr, die Sofware, die das ICS File erstellt. Bzw. wäre die Frage, ob Contao diese Daten in irgendeiner Form verarbeiten kann. Sonst würde ich nämlich ehr dafür sorgen wollen, dass sie erst gar nicht exportiert werden.

kroerig commented 10 months ago

Das Projekt für das ich die Erweiterung nutzen wollte ist aktuell auf Eis gelegt

kroerig commented 5 months ago

Ich habe mir nochmal das RFC angeschaut. Das MAILTO Feld ist Pflicht. Und dann klappt auch der Import.

Weiterhin stellt sich aber die Frage: Was tun mit den ganzen Daten, die ein Termin so enthalten kann und wie kann man sie sinnvoll in Contao darstellen. Z.B. die hier erwähnten Teilnehmer bzw. im Fall der Schule sind das die Klassen für die dieser Termin gültig ist.

Es gibt ja https://github.com/terminal42/contao-calendar-categories Das ist zwar noch nicht für Contao 5.3 freigegeben, aber ließen sich damit die über iCal Import kommenden Kategorien bei den Terminen übernehmen und anzeigen? Was dann aber wohl #15 voraussetzen würde.