Plugin calendar_xh und PHP 8 - fataler Fehler

Ein CMSimple Support Forum für deutsch sprechende Nutzer und Entwickler
cmb
Posts: 14225
Joined: Tue Jun 21, 2011 11:04 am
Location: Bingen, RLP, DE
Contact:

Re: Plugin calendar_xh und PHP 8 - fataler Fehler

Post by cmb » Mon Jan 30, 2023 2:36 pm

frase wrote:
Mon Jan 30, 2023 1:29 pm
Ich habe mich schon gewundert, warum das bisher noch keinem aufgefallen ist. Läuft da was bei mir schief?
Ja, du hast die CMSimple_XH Installation in einem Unterordner namens calendar/, denke ich. Benenne das mal um. :)

Zum Rest etwas später mehr.
wolfgang_58 wrote:
Mon Jan 30, 2023 2:29 pm
Das kann doch nicht jede sehen und dem Beispiel folgen. Ich jedenfalls nicht.
Wir versuchen hier herauszufinden, was alles bei Calendar 1.4.10 noch korrigiert werden muss. Dazu ist eben etwas Austausch nötig. Wenn das abgeschlossen ist, dann gibt es Calendar 1.4.11, und dann ist hoffentlich alles wieder gut.
Christoph M. Becker – Plugins for CMSimple_XH

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

Re: Plugin calendar_xh und PHP 8 - fataler Fehler

Post by frase » Mon Jan 30, 2023 4:02 pm

cmb wrote:
Mon Jan 30, 2023 2:36 pm
frase wrote:
Mon Jan 30, 2023 1:29 pm
Ich habe mich schon gewundert, warum das bisher noch keinem aufgefallen ist. Läuft da was bei mir schief?
Ja, du hast die CMSimple_XH Installation in einem Unterordner namens calendar/, denke ich. Benenne das mal um. :)
Beinahe ;-)
Der Ordner-Ort lautet in voller Länge:
d:\xampp_820\htdocs\sites\CMSXH\xh-tests-820\175-calendar\
Und da ist eben das entscheidende „calendar“ vor dem (Back)Slash enthalten :(
Umbenannt – und in der index.php wieder den originalen Security-Check eingefügt: Läuft!
Das habe ich sogar verstanden. ;-)
Nichtsdestoweniger kann das nicht so bleiben, denke ich. Und es gibt ja schon zwei Vorschläge, wie das gelöst werden könnte. Ich bin für den zweiten (nur nach Gefühl!).

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

Re: Plugin calendar_xh und PHP 8 - fataler Fehler

Post by frase » Mon Jan 30, 2023 4:27 pm

Noch etwas, sehr unangenehm und hässlich:
Wenn man beim Bearbeiten der Termine (egal, ob im Plugin oder mit editevents) in irgendeinem Eingabefeld die [ENTER]-Taste drückt, wird dieser Termin sang- und klanglos gelöscht. Ohne jegliche Warnung!
Es gibt zwar die Backups, die man wiederherstellen kann – doch scheee is des nich!

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

Re: Plugin calendar_xh und PHP 8 - fataler Fehler

Post by cmb » Mon Jan 30, 2023 6:41 pm

lck wrote:
Mon Jan 30, 2023 1:21 pm
Einzig allein bei Link "Feiertage" beim Sichern gibt's noch Hinweise.
Ich habe noch zwei weitere Stellen gefunden; alle 3 gefixt.
lck wrote:
Mon Jan 30, 2023 1:21 pm
Aber wenn man in der _XHdebug.txt eine 5 satt 6 einträgt, wäre auch das verschwunden. ;)
Ja, die Notices wären weg, aber ich bezweifle, dass sich das Plugin wie gedacht verhalten würde. Es werden ja Änderungen an der Sprachdatei vorgenommen (nicht aber an den Variablen), und dann wird versucht die Sprachdatei neu zu laden, damit die geänderten Sprachvariablen verfügbar sind; aber eben das klappt nicht. Daher der "Trick" durch eine Weiterleitung auf dieselbe Seite das Laden der Variablen zu erzwingen.
frase wrote:
Fri Jan 27, 2023 12:28 pm
Was mir noch auffiel:
Auf einer Seite:

Code: Select all

{{{bigcalendar();}}}
Darunter steht dann der Hinweis:
Mouse over date => details
Das kann man in den Sprachdateien ändern wie man will – immer nach irgendwelchen Speichervogängen, steht es wieder in Englisch da.
Kann ich auch nicht wirklich reproduzieren. Es ist halt zu Bedenken, dass die Sprachdateien nicht nur wie sonst üblich in den Spracheinstellungen geändert werden können, sondern auch über andere Features (diese Sprach-Einstellung eben in der Bigcalendar-Konfiguration).
frase wrote:
Mon Jan 30, 2023 4:27 pm
Noch etwas, sehr unangenehm und hässlich:
Wenn man beim Bearbeiten der Termine (egal, ob im Plugin oder mit editevents) in irgendeinem Eingabefeld die [ENTER]-Taste drückt, wird dieser Termin sang- und klanglos gelöscht. Ohne jegliche Warnung!
Es gibt zwar die Backups, die man wiederherstellen kann – doch scheee is des nich!
Das ist Standard-HTML-Formular-Verhalten. Drückt man Enter, dann wird der erste Submit-Button getriggert; und der ist hier halt "Löschen". Enter allgemein zu verbieten, geht hier nicht, da das in den Textareas und WYSIWYG-Editoren dann auch nicht ginge. Auf die Schnelle mal einen unsichtbaren Button vor dem "Löschen" eingefügt, also

Code: Select all

 includes/eventform.php | 1 +
 1 file changed, 1 insertion(+)

diff --git a/includes/eventform.php b/includes/eventform.php
index abfa7b0..03b89a1 100644
--- a/includes/eventform.php
+++ b/includes/eventform.php
@@ -623,6 +623,7 @@ foreach($events as $entry) {
     //===================================
     $o .= '<td class="column6" style="text-align:right;vertical-align:middle">'
        .  tag('input type="hidden" value="'.$i.'" name="dataset"') . "\n"
+       .  tag('input type="submit" style="display:none;"'). "\n"
        .  tag('input type="image" src="' .  $imageFolder
        .  '/delete.png" style="width:16;height:16;" name="delete[0]" title="'
        .  $plugin_tx['calendar']['hint_icon_delete'] . '"'). "\n"
Aber das ist auch nicht schön. Weiß nicht wirklich, wie man das auf die Schnelle lösen könnte. "Hinzufügen"-Schalter nach links? Wäre zumindest eine Option.
Christoph M. Becker – Plugins for CMSimple_XH

knollsen
Posts: 459
Joined: Wed Nov 06, 2013 2:28 pm

Re: Plugin calendar_xh und PHP 8 - fataler Fehler

Post by knollsen » Tue Jan 31, 2023 10:23 am

Soll ich den urigen orig. Kalender den ich modifiziert habe (mit Kategorien) nun umbenennen oder nicht? Was meint Ihr?

olape
Posts: 2731
Joined: Fri Mar 13, 2015 8:47 am
Contact:

Re: Plugin calendar_xh und PHP 8 - fataler Fehler

Post by olape » Tue Jan 31, 2023 10:48 am

knollsen wrote:
Tue Jan 31, 2023 10:23 am
Soll ich den urigen orig. Kalender den ich modifiziert habe (mit Kategorien) nun umbenennen oder nicht? Was meint Ihr?
Hast du denn einen Überblick, wie oft der in Verwendung ist?
Es ist ja nicht unbedingt ratsam, die User mit derlei Dingen zu verärgern.
Gruß Olaf, Plugins for CMSimple_XH

Ich habe schon lange den Verdacht, dass so viele so eifrig auf Gender, Trans und Queer machen:
Weil sie für das Fachliche ganz einfach zu doof sind.

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

Re: Plugin calendar_xh und PHP 8 - fataler Fehler

Post by frase » Tue Jan 31, 2023 10:53 am

knollsen wrote:
Tue Jan 31, 2023 10:23 am
Soll ich den urigen orig. Kalender den ich modifiziert habe (mit Kategorien) nun umbenennen oder nicht? Was meint Ihr?
Ich fände das in deinem Fall gar nicht so schlecht.
„calendar“ sollte aber im Namen bleiben – nur irgendetwas vorne oder hinten dran.
Ich weiß nicht, ob du hier alles mitgelesen hast. Aber den weißen Bildschirm mit „Access denied“ solltest du auch noch vermeiden. Das wird zwar sicher selten vorkommen, ist aber möglich.
Christoph hat es hier gelöst. Soweit ich das überblicke (?) steht das z.B. in deiner index.php schon drin, aber Zeile 8 müsste dann bei dir gelöscht werden.

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

Re: Plugin calendar_xh und PHP 8 - fataler Fehler

Post by frase » Tue Jan 31, 2023 11:07 am

cmb wrote:
Mon Jan 30, 2023 6:41 pm
Kann ich auch nicht wirklich reproduzieren. Es ist halt zu Bedenken, dass die Sprachdateien nicht nur wie sonst üblich in den Spracheinstellungen geändert werden können, sondern auch über andere Features (diese Sprach-Einstellung eben in der Bigcalendar-Konfiguration).
Das lag wohl bei mir noch an den falschen Pfaden.
Jetzt teste ich mit dem aktuellen Master, und da ist es OK.
cmb wrote:
Mon Jan 30, 2023 6:41 pm
Aber das ist auch nicht schön. Weiß nicht wirklich, wie man das auf die Schnelle lösen könnte. „Hinzufügen“-Schalter nach links? Wäre zumindest eine Option.
Bei einzeiligen Eingabefeldern bewirkt [ENTER] im Normalfall das Absenden des Formulars.
So ist es übrigens bei XH auch in allen Config-Formularen.
Also habe ich den „Speichern-Button“ (das Bild mit dem Häkchen) mal an den Anfang verschoben.
eventform.php ab Zeile 622 (aktueller Master):

Code: Select all

    //delete, copy, add and save buttons
    //===================================
    $o .= '<td class="column6" style="text-align:right;vertical-align:middle">'
       .  tag('input type="image" src="'
       .  $imageFolder
       .  '/ok.png" style="width:16;height:16" name="change[0]" title="'
       .  $plugin_tx['calendar']['hint_icon_ok'] . '"') . "\n"
       .  tag('input type="hidden" value="'.$i.'" name="dataset"') . "\n"
       .  tag('input type="image" src="' .  $imageFolder
       .  '/delete.png" style="width:16;height:16;" name="delete[0]" title="'
       .  $plugin_tx['calendar']['hint_icon_delete'] . '"'). "\n"
       .  tag('input type="image" src="'
       .  $imageFolder
       .  '/copy.png" style="width:16;height:16" name="copy[0]" title="'
       .  $plugin_tx['calendar']['hint_icon_copy'] . '"') . "\n"
       .  tag('input type="image" src="'.$imageFolder
       .  '/add.png" style="width:16;height:16;" name="add[0]" title="'
       .  $plugin_tx['calendar']['hint_icon_add'] . '"') . "\n"
       . "</td>\n"
       .  "</tr>\n";
So scheint es mir in Ordnung zu sein. Bei [ENTER] wird das Formular abgeschickt und die Änderungen werden gespeichert.
So wie man es erwartet – NICHT LÖSCHEN.

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

Re: Plugin calendar_xh und PHP 8 - fataler Fehler

Post by lck » Tue Jan 31, 2023 11:37 am

cmb wrote:
Mon Jan 30, 2023 6:41 pm
lck wrote:
Mon Jan 30, 2023 1:21 pm
Einzig allein bei Link "Feiertage" beim Sichern gibt's noch Hinweise.
Ich habe noch zwei weitere Stellen gefunden; alle 3 gefixt.
Notices are gone! 👍🏻
„Bevor du den Pfeil der Wahrheit abschießt, tauche die Spitze in Honig!“   👉 Ludwig's XH-Templates for MultiPage & OnePage

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

Re: Plugin calendar_xh und PHP 8 - fataler Fehler

Post by lck » Tue Jan 31, 2023 11:39 am

cmb wrote:
Mon Jan 30, 2023 6:41 pm
"Hinzufügen"-Schalter nach links? Wäre zumindest eine Option.
frase wrote:
Tue Jan 31, 2023 11:07 am
Also habe ich den „Speichern-Button“ (das Bild mit dem Häkchen) mal an den Anfang verschoben.
Ich auch. Ist ok.
„Bevor du den Pfeil der Wahrheit abschießt, tauche die Spitze in Honig!“   👉 Ludwig's XH-Templates for MultiPage & OnePage

Post Reply