Calendar_XH 2.0dev1

Ein CMSimple Support Forum für deutsch sprechende Nutzer und Entwickler
frase
Posts: 5085
Joined: Thu Apr 21, 2016 6:32 am
Location: Saxony
Contact:

Calendar_XH 2.0dev1

Post by frase » Sun Jan 14, 2018 4:41 pm

Betrifft: https://github.com/cmb69/calendar_xh/re ... ag/2.0dev1

Seite zum Bearbeiten der Termine ohne Register_XH oder Memberpages angelegt.

Code: Select all

<div>{{{editevents()}}}</div>
Bei Vorschau erscheint weiße Seite mit Fehlermeldungen:
Uncaught ArgumentCountError: Too few arguments to function editevents(), 0 passed in ... \cmsimple\functions.php(248) : eval()'d code on line 1 and exactly 1 expected in ... \plugins\calendar\index.php:48
Stack trace:
#0 ... \cmsimple\functions.php(248) : eval()'d code(1): editevents()
#1 ... \cmsimple\functions.php(248): eval()
#2 ... \cmsimple\functions.php(208): XH_evaluateSinglePluginCall('editevents()')
#3 ... \cmsimple\functions.php(305): evaluate_plugincall('
Last edited by frase on Sun Jan 14, 2018 4:45 pm, edited 2 times in total.

frase
Posts: 5085
Joined: Thu Apr 21, 2016 6:32 am
Location: Saxony
Contact:

Re: Calendar_XH 2.0dev1

Post by frase » Sun Jan 14, 2018 4:44 pm

ferien_sachsen_2018.zip
Test mit der dev-Version.
ics-Datei importiert.
Ergebnis: null

Lediglich in calendar.csv taucht auf:

Code: Select all

;;;;Herbstferien;;;
;;;;Weihnachtsferien;;;
;;;;Sommerferien;;;
;;;;Winterferien;;;
;;;;Osterferien;;;
;;;;Pfingstferien;;;
;;;;Pfingstferien;;;
Wie muss man da vorgehen?
Oder ist das Format falsch?
You do not have the required permissions to view the files attached to this post.

cmb
Posts: 14225
Joined: Tue Jun 21, 2011 11:04 am
Location: Bingen, RLP, DE
Contact:

Re: Calendar_XH 2.0dev1

Post by cmb » Sun Jan 14, 2018 5:02 pm

frase wrote:
Sun Jan 14, 2018 4:41 pm
Bei Vorschau erscheint weiße Seite mit Fehlermeldungen:
Uncaught ArgumentCountError: Too few arguments to function editevents(), 0 passed in ...
Danke – gefixt. Wenn du die Änderung nicht selbst einpflegen willst, dann einfach {{{editevents('')}}} verwenden.
frase wrote:
Sun Jan 14, 2018 4:44 pm
ferien_sachsen_2018.zip
Test mit der dev-Version.
ics-Datei importiert.
Ergebnis: null

Wie muss man da vorgehen?
Oder ist das Format falsch?
Das Format ist richtig, aber DTSTART;VALUE=DATE und DTEND;VALUE=DATE werden noch nicht unterstützt. Auf die schnelle kannst du bei diesen Zeilen am Ende einfach T0000 (bei DTSTART) bzw T2359 (bei DTEND) ergänzen; dann sollte der Import prinzipiell funktionieren.
Christoph M. Becker – Plugins for CMSimple_XH

frase
Posts: 5085
Joined: Thu Apr 21, 2016 6:32 am
Location: Saxony
Contact:

Re: Calendar_XH 2.0dev1

Post by frase » Sun Jan 14, 2018 5:17 pm

cmb wrote:
Sun Jan 14, 2018 5:02 pm
Danke – gefixt. Wenn du die Änderung nicht selbst einpflegen willst, dann einfach {{{editevents('')}}} verwenden.
Danke!
Beide Versionen funktionieren - also entweder geänderter Aufruf oder Fix.
cmb wrote:
Sun Jan 14, 2018 5:02 pm
Auf die schnelle kannst du bei diesen Zeilen am Ende einfach T0000 (bei DTSTART) bzw T2359 (bei DTEND) ergänzen; dann sollte der Import prinzipiell funktionieren.
Auch das funktioniert. Auch, wenn ich überhaupt nicht verstehe, was da passiert ... ;-)

cmb
Posts: 14225
Joined: Tue Jun 21, 2011 11:04 am
Location: Bingen, RLP, DE
Contact:

Re: Calendar_XH 2.0dev1

Post by cmb » Sun Jan 14, 2018 5:31 pm

frase wrote:
Sun Jan 14, 2018 5:17 pm
cmb wrote:
Sun Jan 14, 2018 5:02 pm
Auf die schnelle kannst du bei diesen Zeilen am Ende einfach T0000 (bei DTSTART) bzw T2359 (bei DTEND) ergänzen; dann sollte der Import prinzipiell funktionieren.
Auch das funktioniert. Auch, wenn ich überhaupt nicht verstehe, was da passiert ... ;-)
Das kann leicht geändert werden: normalerweise wird für DTSTART (Date-Time-Start) ein ISO 8601 konformer Zeitstempel (Datum- + Zeitangabe) erwartet. Wird aber VALUE=DATE angegeben, dann wird eben nur eine ISO 8601 konforme Datumsangabe erwartet (und damit kommt 2.0dev1 nicht klar).
Christoph M. Becker – Plugins for CMSimple_XH

frase
Posts: 5085
Joined: Thu Apr 21, 2016 6:32 am
Location: Saxony
Contact:

Re: Calendar_XH 2.0dev1

Post by frase » Sun Jan 14, 2018 5:38 pm

Aha.

lck
Posts: 2963
Joined: Wed Mar 23, 2011 11:43 am
Contact:

Re: Calendar_XH 2.0dev1

Post by lck » Mon Jan 15, 2018 2:11 pm

cmb wrote:Ich arbeite allerdings schon seit einer ganzen Weile an einem Fork von Calendar 1.2, und habe dort nun minimalen Import von .ics Dateien implementiert, und den derzeitigen Stand als Development-Preview freigegeben. Diese Version ist wirklich nicht für den Produktiveinsatz gedacht, sondern nur zum Ausprobieren: Calendar_XH-2.0dev1. Schau's dir gerne mal an – über Feedback was klappt und was nicht klappt, und wo noch nachgebessert werden könnte, würde ich mich freuen.
Sehr gut, danke!
Hier mein Feedback.
cmb wrote:Für diejenigen, die Calendar 1.2 bereits verwenden: 2.0dev1 ist weitestgehend kompatibel zu 1.2.10; allerdings müssen die Datendateien (eventcalendar.txt bzw. eventcalendar_XY.txt) aus dem Pluginordner in den allgemeinen content/ Ordner von CMSimple_XH verschoben werden, und zu calendar.csv umbenannt werden.
1) Kompatibel nicht, zumindest nicht von svastis Version 1.2.10 zu 2.0dev1, leider. Da war die Syntax noch eine andere.

2) Leider auch kein html mehr verwendbar (zumindest bisher :? ), wird jetzt als reiner Text ausgegeben.
Habe dies immer gern genutzt um Bereiche farblich hervorzuheben, Schriftgröße usw. und auch um Links oder Bilder einzufügen. Beispiel:

Code: Select all

30.06.2018;;;;<span style="color:darkorange">Max Muster</span><br>Beginn 20.00 Uhr<br>Einlass ab 19 Uhr<br>Eintritt 8,- €<br><a class="zoom" href="./userfiles/images/veranstaltungen/maxmuster.jpg"><img title="Bitte klicken für größere Ansicht" src="./userfiles/images/veranstaltungen/maxmuster.jpg" alt="" style="width:100px"></a>;Musterort;http://www.example.de/;Mehr Infos
3) Leerzeilen in der calendar.csv
In svastis Version habe ich die Events immer direkt in der eventcalendar_de.txt bearbeitet, da übersichtlicher, kopieren einfügen und anpassen das Ganze erleichterte. Dazu hatte ich zwecks Übersichtlichkeit zwischen den Monaten immer eine Leerzeile eingefügt. Das ist nun nicht mehr möglich, na ja möglich schon (¹ man darf halt das Backend zum Bearbeiten der Events nicht mehr benutzen), aber dann meldet sich der Debug-Mode beim Aufruf der Event-Page:
NOTICE: Undefined offset: 1
D:\xampp\htdocs\cmsimplexh172\plugins\calendar\classes\EventDataService.php:77
NOTICE: Undefined offset: 2
D:\xampp\htdocs\cmsimplexh172\plugins\calendar\classes\EventDataService.php:77
NOTICE: Undefined offset: 3
D:\xampp\htdocs\cmsimplexh172\plugins\calendar\classes\EventDataService.php:77
NOTICE: Undefined offset: 4
D:\xampp\htdocs\cmsimplexh172\plugins\calendar\classes\EventDataService.php:77
NOTICE: Undefined offset: 5
D:\xampp\htdocs\cmsimplexh172\plugins\calendar\classes\EventDataService.php:77
NOTICE: Undefined offset: 6
D:\xampp\htdocs\cmsimplexh172\plugins\calendar\classes\EventDataService.php:77
NOTICE: Undefined offset: 7
D:\xampp\htdocs\cmsimplexh172\plugins\calendar\classes\EventDataService.php:77
Eine andere Option wären 7 Semikolons in der Leerzeile, das geht und der Debug-Mode meldet nichts (aber siehe ¹).
„Bevor du den Pfeil der Wahrheit abschießt, tauche die Spitze in Honig!“   👉 Ludwig's XH-Templates for MultiPage & OnePage

frase
Posts: 5085
Joined: Thu Apr 21, 2016 6:32 am
Location: Saxony
Contact:

Re: Calendar_XH 2.0dev1

Post by frase » Mon Jan 15, 2018 3:14 pm

zu Ludwigs Post:

1)
ja, z.B. ist die Reihenfolge anders (Zeiten) und von bis

2)
HTML-Ausgabe - wie eingegeben - wünschte ich mir ebenfalls
(nicht bei Title in div calendar_calendar bzw. in Table calendar_main)

3)
wäre evtl. verschmerzbar - man kann sich mit 7 Semikolönnern helfen

cmb
Posts: 14225
Joined: Tue Jun 21, 2011 11:04 am
Location: Bingen, RLP, DE
Contact:

Re: Calendar_XH 2.0dev1

Post by cmb » Mon Jan 15, 2018 3:22 pm

lck wrote:
Mon Jan 15, 2018 2:11 pm
1) Kompatibel nicht, zumindest nicht von svastis Version 1.2.10 zu 2.0dev1, leider. Da war die Syntax noch eine andere.
Da sollte aber automatisch konvertiert werden. Also beim Lesen wird einfach nur intern konvertiert, beim Speichern sollte es dann auch in calendar.csv sichtbar sein.
lck wrote:
Mon Jan 15, 2018 2:11 pm
2) Leider auch kein html mehr verwendbar (zumindest bisher :? ), wird jetzt als reiner Text ausgegeben.
Habe dies immer gern genutzt um Bereiche farblich hervorzuheben, Schriftgröße usw. und auch um Links oder Bilder einzufügen. Beispiel:

Code: Select all

30.06.2018;;;;<span style="color:darkorange">Max Muster</span><br>Beginn 20.00 Uhr<br>Einlass ab 19 Uhr<br>Eintritt 8,- €<br><a class="zoom" href="./userfiles/images/veranstaltungen/maxmuster.jpg"><img title="Bitte klicken für größere Ansicht" src="./userfiles/images/veranstaltungen/maxmuster.jpg" alt="" style="width:100px"></a>;Musterort;http://www.example.de/;Mehr Infos
Ich halte es für keine gute Idee, beliebig HTML zu erlauben – das gibt im besseren Fall leicht mal invalides HTML ("Klaus & Klaus"), Darstellungsprobleme (HTML im title-Attribut), und im schlimmsten Fall sogar XSS-Lücken. In diesem Fall sehe ich aber ein, dass es sehr nützlich sein kann. Für welche Felder bräuchtest du es denn – nur für die Veranstaltung selbst?
lck wrote:
Mon Jan 15, 2018 2:11 pm
3) Leerzeilen in der calendar.csv
In svastis Version habe ich die Events immer direkt in der eventcalendar_de.txt bearbeitet, da übersichtlicher, kopieren einfügen und anpassen das Ganze erleichterte. Dazu hatte ich zwecks Übersichtlichkeit zwischen den Monaten immer eine Leerzeile eingefügt. Das ist nun nicht mehr möglich, […]
Hm, das kann leicht geändert werden. Quick-Fix: nach dieser und dieser Zeile einfügen:

Code: Select all

                if (trim($line) === '') continue;
Christoph M. Becker – Plugins for CMSimple_XH

frase
Posts: 5085
Joined: Thu Apr 21, 2016 6:32 am
Location: Saxony
Contact:

Re: Calendar_XH 2.0dev1

Post by frase » Mon Jan 15, 2018 3:29 pm

cmb wrote:
Mon Jan 15, 2018 3:22 pm
Für welche Felder bräuchtest du es denn – nur für die Veranstaltung selbst?
Ich denke es genügt nur für "Veranstaltung" - Ludwig?
Da wäre eigentlich ein Editor-Feld angebracht. Das gibt aber wieder Probleme mit narrow, medium und wide - was ich sowieso etwas heftig finde.
Eigentlich sollte es so aufgebaut sein, dass es immer passt.

Post Reply