Hallo zusammen,
beim iCal-Export steckt noch ein Fehler drin. Wenn man die Einträge des Exports anhand des Kalenders filtert, wird dieser Filter ignoriert und alle Einträge ausgegeben. Der Fehler kann in der Demo nachgestellt werden:
- Alle Einträge: http://demo.admidio.org/adm_program/mod ... e&cat_id=0
- Einträge des Kalenders Allgemein: http://demo.admidio.org/adm_program/mod ... &cat_id=10
Die jeweils erzeugten ics-Dateien sind gleich groß und enthalten alle Einträge.
Gruß Offe
iCal Export Filter Kalender
-
Thomas-RCV
- Former team member
- Beiträge: 786
- Registriert: 1. Aug 2011, 15:06
Re: iCal Export Filter Kalender
Hallo Offe,
hier wurde der Kalender nicht mit übergeben. Sollte nun passen.
Du kannst dir die Ical_dates.php aus unserem 3.0 Branch holen und es probieren.
https://github.com/Admidio/admidio/blob ... _dates.php
Einen guten Rutsch ins neue Jahr und schöne Sylvester.
VG Thomas
hier wurde der Kalender nicht mit übergeben. Sollte nun passen.
Du kannst dir die Ical_dates.php aus unserem 3.0 Branch holen und es probieren.
https://github.com/Admidio/admidio/blob ... _dates.php
Einen guten Rutsch ins neue Jahr und schöne Sylvester.
VG Thomas
Re: iCal Export Filter Kalender
Hallo Thomas,
vielen Dank für deine Korrektur. Der iCAL-Export funktioniert nun auch wieder mit dem Filter Kalender.
Zwei Dinge sind mir noch aufgefallen:
- Beim Merge der Änderung in den Master hat sich am Ende wieder das komische Zeichen eingeschlichen, das bereits den Fehler in der 3.0.5 verursacht hat: https://github.com/Admidio/admidio/comm ... 73b319895f (LIne 108)
- Außer den Filter auf den Kalender kann man auch auf das Beginn- und Enddatum filtern. Dieser Filter wird nicht in dem iCAL-Export berücksichtigt. Ich selbst habe hierzu keinen Use-Case, aber ist ein bisschen verwirrend, dass ein Teil der Filter Auswirkungen hat und ein anderer nicht.
Ich wünsche dir auch schöne Silvester und einen guten Start ins neue Jahr.
Gruß Offe
vielen Dank für deine Korrektur. Der iCAL-Export funktioniert nun auch wieder mit dem Filter Kalender.
Zwei Dinge sind mir noch aufgefallen:
- Beim Merge der Änderung in den Master hat sich am Ende wieder das komische Zeichen eingeschlichen, das bereits den Fehler in der 3.0.5 verursacht hat: https://github.com/Admidio/admidio/comm ... 73b319895f (LIne 108)
- Außer den Filter auf den Kalender kann man auch auf das Beginn- und Enddatum filtern. Dieser Filter wird nicht in dem iCAL-Export berücksichtigt. Ich selbst habe hierzu keinen Use-Case, aber ist ein bisschen verwirrend, dass ein Teil der Filter Auswirkungen hat und ein anderer nicht.
Ich wünsche dir auch schöne Silvester und einen guten Start ins neue Jahr.
Gruß Offe
-
Thomas-RCV
- Former team member
- Beiträge: 786
- Registriert: 1. Aug 2011, 15:06
Re: iCal Export Filter Kalender
Hallo Offe,
das Zeichen am Ende hängt mit dem Travis-Check zusammen. Hier habe ich rein gar nichts geändert.
Der andere Fehler war tatsächlich ein Problem, der damals bei meinem Commit aufgetreten ist.
Solange die Datei bei dir keinen Fehler verursacht, einfach ignorieren
Da gebe ich dir recht mit den Filtern. Das hängt alles mit der Historie zusammen. Der Ical wurde immer schon über die Systemeinstellung definiert. Da gab es den Zeitraumfilter noch nicht. Der hat dann später erst seinen Weg ins Modul gefunden.
Von daher stellt sich da die Frage, ob man das alles in der Tat mal bereinigt. Über den Zeitraum wären die Systemeinstellung für den Ical hinfällig.
Das müsste man sich vielleicht als Gesamtkonzept überlegen, wie es am meisten Sinn macht.
VG Thomas
das Zeichen am Ende hängt mit dem Travis-Check zusammen. Hier habe ich rein gar nichts geändert.
Der andere Fehler war tatsächlich ein Problem, der damals bei meinem Commit aufgetreten ist.
Solange die Datei bei dir keinen Fehler verursacht, einfach ignorieren
Da gebe ich dir recht mit den Filtern. Das hängt alles mit der Historie zusammen. Der Ical wurde immer schon über die Systemeinstellung definiert. Da gab es den Zeitraumfilter noch nicht. Der hat dann später erst seinen Weg ins Modul gefunden.
Von daher stellt sich da die Frage, ob man das alles in der Tat mal bereinigt. Über den Zeitraum wären die Systemeinstellung für den Ical hinfällig.
Das müsste man sich vielleicht als Gesamtkonzept überlegen, wie es am meisten Sinn macht.
VG Thomas
Re: iCal Export Filter Kalender
Die 3.0 Datei funktioniert ohne Probleme, die Datei im master habe ich noch nicht geprüft, dies werde ich dann mit den Beta Tests machen.
Ich denke auch, dass es sinnvoll ist, sich das komplette Module Dates mal anzuschauen. Teilweise habe ich sebst versucht den Fehler zu finden und habe dabei festgestellt, das gleicher Programmcode teilweise sowohl in den Modulen wie auch in den entsprechenden Classen vorhanden ist, Funktionen und Parameter teilweise nicht genutzt werden....
Danke und Gruß
Offe
Ich denke auch, dass es sinnvoll ist, sich das komplette Module Dates mal anzuschauen. Teilweise habe ich sebst versucht den Fehler zu finden und habe dabei festgestellt, das gleicher Programmcode teilweise sowohl in den Modulen wie auch in den entsprechenden Classen vorhanden ist, Funktionen und Parameter teilweise nicht genutzt werden....
Danke und Gruß
Offe
Re: iCal Export Filter Kalender
Das zeichen verdeutlicht nur das nach dem letzten zeichen keinen Zeilenumbruch mehr ist. Unser Travis Check sagt aber es sollte am schluss noch eine leerzeile sein. Das Zeichen (ist nur ein hinweis. Das zeichen steht selbst nicht im Quellcode) selbst hat aber nichts mit Travis zu tunBeim Merge der Änderung in den Master hat sich am Ende wieder das komische Zeichen eingeschlichen, das bereits den Fehler in der 3.0.5 verursacht hat: https://github.com/Admidio/admidio/comm ... 73b319895f (LIne 108)
