Procalendar Bug

msfrog

Hallo Dietmar,
danke für die Rückmeldung. Dass Entwickler auch nur Menschen sind wird ihnen keiner ankreiden. Falls das von mir so rübergekommen sein sollte, tuts mir leid, das war nicht beabsichtigt.

Auf jeden Fall vielen Dank fürs Beheben!

Luisehahne

Quote from: msfrog on April 21, 2021, 10:11:16 PM
Danke, also war tatsächlich noch was "faul". Aber mich würde brennend interessieren, wo der Fehler genau lag. Ich halt mich eigentlich nicht für nen DAU und wüsst gern, was ich da falsch gemacht haben kann :)

Du hast nicht unbedingt was falsch gemacht. PHP wird von Version zu Version strikter. Wir gehen diesen Trend aus Sicherheitsgründen in WB mit. Deswegen müssen Leereingaben in Feldern vor dem speichern überprüft werden.

Umso wichtiger sind für uns solche Fehlermeldungen. Auch Entwickler sind nur Menschen.

Dietmar
Note: Once the code has been generated, it is easy to debug. It's not a bug, it's a feature!

msfrog

Danke, also war tatsächlich noch was "faul". Aber mich würde brennend interessieren, wo der Fehler genau lag. Ich halt mich eigentlich nicht für nen DAU und wüsst gern, was ich da falsch gemacht haben kann :)

hgs

#53
Gerade Nachricht aus der Entwicklerabteilung bekommen,
Es wurde online auf der Testseite von uns gefixt, ich sag mal aus meiner DAU-Sicht "Ein DAU-Anwenderfehler wurde abgefangen"
Tolle Leistung (Y)

Ein Anwender hatte mir mal für mehrere Tage mit einer "Falsch-Anwendung" im ProCalendar meine Seite gecrasht.
Auch diese Falschbedienung führte zu einer DAU-festeren Version. :wink:

Edit hgs:
Der DAU bezieht sich nur auf mich und ist nicht auf andere USER zu verstehen.
Hoffe, es fühlt sich niemand auf den Schlips getreten, dass war nicht meine Absicht.
LG Harald

"Fange nie an, aufzuhören - höre nie auf, anzufangen." Marcus Tullius Cicero (106-43 v.Chr.)

msfrog

Danke.

Ich hab grade mal den gleichen Test gemacht, der vorhin noch erfolgreich zum Fehler geführt hat (neu installiert, erst 5, dann 6 auf Bild, danach 1 auf Bild -> Fehler). Das funktioniert jetzt problemlos. Ich hab jetzt auch mal alle Felder als Bild eingestellt, auch das geht durch.

hgs

Hab die Frage mal an die Entwicklungsabteilung weiter geleitet, warte auf eine Antwort. Die Kommt sobald ich was höre.
LG Harald

"Fange nie an, aufzuhören - höre nie auf, anzufangen." Marcus Tullius Cicero (106-43 v.Chr.)

msfrog

Nein, das Teil hab ich noch nie benutzt. Habs aber hinbekommen.

Jetzt funktionierts offenbar. Hab das alte Modul deinstalliert, das runtergeladene installiert, nen Kalender angelegt, Feld 1 als Bild und alle anderen auf deaktiviert gesetzt. Ging auf Anhieb. Ich werd nochmal ein bisschen rumspielen, aber das sieht vielversprechend aus.

Habt ihr da jetzt noch was geändert oder muss ich bei mir mal schauen, ob irgendwas kaputt ist?

hgs

Teste es mal bei dir, indem du dir das Modul von unserem Testserver über den AFE als zip auf deinen PC ziehst und dann noch mal bei dir installieren.

Weißt du wie das mit dem AddonFileEditor geht? Ist dort unter Admin installiert

LG Harald

"Fange nie an, aufzuhören - höre nie auf, anzufangen." Marcus Tullius Cicero (106-43 v.Chr.)

msfrog

#48
Quote from: hgs on April 21, 2021, 11:49:47 AM
Wäre ein wichtiger Hinweis gewesen

Naja, ich hab doch oben genau beschrieben, was ich gemacht habe, um den Fehler hervorzurufen... Was fehlte denn da jetzt?

Quote
Wir suchen im Modul, von einem eigenen Script hören wir jetzt zum 1. Mal
Wie sieht dein script aus und wo bindest du es ein?

Der Fehler tritt ja auch im Modul auf, mit meinem Skript hat das nichts zu tun. In der Testumgebung ist das ja noch nichtmal irgendwo eingebaut / in Verwendung. Das Ding macht nichts anderes, als die DB-Felder auszulesen und das Ergebnis in HTML zu verpacken. Aber wie gesagt - der Fehler ist davon *komplett* unabhängig. Eingebaut ist es über ein Code-Feld, auf der Seite, wo der Inhalt erscheinen soll.

Ich sehe grade, dass es auf dem Testserver jetzt mit nem Bild in Feld 1 funktioniert. Wie hast du das hinbekommen?

hgs

Quote from: hgs on April 21, 2021, 11:49:47 AM
Erkläre uns dein Vorgehen?
Warum sind alle Felder bei dir leer und auf "nicht genutzt" eingestellt?
und wo stellst du dann ein zusätzliches Bild ein?

Wäre ein wichtiger Hinweis gewesen
Quote
Den Code brauch ich nicht, nur den Link des Bildes. Ich lese mir die Termine mit nem eigenen Script aus der DB aus. Der Procalendar dient mir quasi nur als Admin-Oberfläche zum Eintragen. Ist aber auch unerheblich, denn in Feld 4 funktionierts perfekt, auch, wenn ich mir nur [CUSTOM_CONTENT] ausgeben lasse.

Wir suchen im Modul, von einem eigenen Script hören wir jetzt zum 1. Mal
Wie sieht dein script aus und wo bindest du es ein?

LG Harald

"Fange nie an, aufzuhören - höre nie auf, anzufangen." Marcus Tullius Cicero (106-43 v.Chr.)

msfrog

Quote from: hgs on April 21, 2021, 12:00:16 PM
OK, sehe gerade das "eigenes Feld 1" auf" Bild" gestellt ist, aber kein html Code für die ausführung darunter steht.

Da müsste
<div class="field_line">
    <img src="[CUSTOM_CONTENT]" border ="0" alt="[CUSTOM_NAME]" />
</div>

stehen.

Den Code brauch ich nicht, nur den Link des Bildes. Ich lese mir die Termine mit nem eigenen Script aus der DB aus. Der Procalendar dient mir quasi nur als Admin-Oberfläche zum Eintragen. Ist aber auch unerheblich, denn in Feld 4 funktionierts perfekt, auch, wenn ich mir nur [CUSTOM_CONTENT] ausgeben lasse.

Quote from: hgs on April 21, 2021, 12:00:16 PM
Ich teste das gerade mal, wie geschrieben bei 4,7+8 läuft es wenn man 1-6 so lässt wie das Modul es vorgibt.

Es ist doch aber nicht Sinn und Zweck der Sache, dass sowas nur mit den Default-Einstellungen funktioniert. Zumindest hast du Recht, dass es zu klappen scheint, sofern ich die ersten 3 Felder nicht anfasse. Ich hab eben auf meinem eigenen Testserver mal die 1.7.2 dev3 neu installiert. Felder 1 - 4 habe ich Default gelassen, Feld 5 zusätzlich als Bild definiert -> geht. Dann noch Feld 6 zusätzlich als Bild -> geht auch. Dann Feld 1 auf Bild -> Fehler.

msfrog

Feld 1 ist auf Bild gesetzt, auch auf eurem Server. Die anderen Felder sind auf "nicht genutzt" gesetzt, weil für den Test unerheblich. Versuche mal einen neuen Termin anzulegen und ein Bild aus dem Dropdown einzufügen (oder ein neues hochzuladen, spielt keine Rolle). Es kommt immer der Fehler:

There was an uncatched exception
Argument 1 passed to ReplaceAbsoluteMediaUrl() must be of the type string, null given, called in /www/htdocs/xxx/WB_Testbereich/msfrog/modules/procalendar/save.php on line 257
in line (23) of (/modules/procalendar/functions.php):

Bei mir funktioniert das Bildfeld ausschließlich als Feld 4 korrekt. Setze ich ein anderes Feld auf Bild (egal ob zusätzlich zu Feld 4 oder nur das), gibts beim Anlegen eines Termins Fehlermeldungen. Nur mit Feld 7 hat es bei einem Test mal geklappt, aber auch nur einmal. Nachdem ich ein anderes Feld versucht und dann nochmal die 7 probiert hatte, gings wieder nicht.

Ich mein, man kann doch dabei eigentlich nichts falsch machen...

hgs

OK, sehe gerade das "eigenes Feld 1" auf" Bild" gestellt ist, aber kein html Code für die ausführung darunter steht.

Da müsste
<div class="field_line">
    <img src="[CUSTOM_CONTENT]" border ="0" alt="[CUSTOM_NAME]" />
</div>

stehen.

Ich teste das gerade mal, wie geschrieben bei 4,7+8 läuft es wenn man 1-6 so lässt wie das Modul es vorgibt.
LG Harald

"Fange nie an, aufzuhören - höre nie auf, anzufangen." Marcus Tullius Cicero (106-43 v.Chr.)

hgs

Testdomäne mit deinen Daten angelegt. Die "eigenen Felder" sind "immer noch/oder schon wieder" leer.
Wie wir es hinbekommen steht im #38
https://forum.websitebaker.org/index.php/topic,31890.msg223293.html#msg223293

Erkläre uns dein Vorgehen?
Warum sind alle Felder bei dir leer und auf "nicht genutzt" eingestellt?
und wo stellst du dann ein zusätzliches Bild ein?
LG Harald

"Fange nie an, aufzuhören - höre nie auf, anzufangen." Marcus Tullius Cicero (106-43 v.Chr.)

msfrog

Grüß euch,
ich habs leider heute erst geschafft, mich weiter drum zu kümmern. Hab die "defekte" Installation auf den Testserver geladen, bin gespannt, obs bei euch funktioniert...

hgs

So bin wieder nach einer lannnngen Veranstalltung online.
Bitte mach es bei unserem Testsystem bitte mal so wie es bei dir nicht funktioniert.
Wenn es schneller mit deinem Dump geht, gibt ihn her, oder du stellst es auf der Testdomäne alles so ein.
Werde dir den FTP-Zugang auf das Verzeichnis legen. Bekommst per pm die ZUgangsdaten und ein Admin-Passwort für die Testdonäne.
LG Harald

"Fange nie an, aufzuhören - höre nie auf, anzufangen." Marcus Tullius Cicero (106-43 v.Chr.)

msfrog

Ich hab jetzt nochmal einen Dump erstellt, wo der Fehler geworfen wird. Willst du das nochmal probieren?

Dass die Felder alle geleert sind, ist nur bei dem Testsystem der Fall, das spielt für das Auftreten des Fehlers keine Geige. Ich hatte das gemacht, um die Felder alle einzeln durchprobieren zu können.

hgs

Da ich ein "Nur-Anwender" bin, vermute ich, dass durch dein "löschen" der Custom-Felder, der Fehler mit reingekommen ist. (Aber das ist nur eine Vermutung)
Da nur das ausgegeben wird, was der USER eingibt, stören die nicht genutzen Custom-Fehler ja nicht und ich auch DAU würde diese nie löschen/verändern.

Hoffe das dir das weiterhilft.
LG Harald

"Fange nie an, aufzuhören - höre nie auf, anzufangen." Marcus Tullius Cicero (106-43 v.Chr.)

hgs

#38
https://msfrog.umojasingers.de/pages/test.php?test%20hgs&month=4&year=2021&day=16&show=0&id=1&section_id=5&detail=1

3 Bilder in verschiedenen Felder 4, 6 und 7 glaube ich.
Kooektur, sind die Custom-Felder 4, 7 + 8

Und ich habe es folgendermaßen gemacht, da du alle Custom-Feler leer gemacht hattest und auch alle auf "nicht genutzt" standen.
1. Section mit ProCalender gelöscht
2. ProCalendar gelöscht
3. ProCalendar 1.7.2 dev3 installiert
4. zusätzliche Sektion mit ProCalendar wieder angelegt
5. alle Custom-Felder so gelassen wie sie sind
6. zwei zusätzliche Bilder-Felder durch Auswahl vom "Typ Bild" und dem html-Code aus Custom-Feld 4 zusätzlich erzeugt
7. einen Termin angelegt und 3 Bilder ausgesucht.

Fertig
LG Harald

"Fange nie an, aufzuhören - höre nie auf, anzufangen." Marcus Tullius Cicero (106-43 v.Chr.)

msfrog

Das ist mit Bild in Custom 4, so wie es hier funktioniert. Versuch mal Custom 1 z.B. zu Bild zu machen und ein Bild in nen Termin zu speicherrn. Dann sollte der Fehler kommen.

hgs

LG Harald

"Fange nie an, aufzuhören - höre nie auf, anzufangen." Marcus Tullius Cicero (106-43 v.Chr.)

msfrog

So, jetzt geht es mal wieder mit der 7. Ich hab jetzt auch mal ein paar Termine angelegt und geändert, es funktioniert offenbar zuverlässig, wenns einmal geht. Wenn ich aber z.B. ein anderes Custom-Feld zu Bild mache und es nicht geht, dann funktionierts auch zuverlässig nicht. In der Datenbank werden die jeweiligen "usecustom"-Felder auf 4 gesetzt, daran liegts also nicht.

Ich kann mir da keinen Reim drauf machen... Ne Idee, was ich noch testen könnte?

msfrog

Hm... Ich hab auch nochmal bisschen rumgetestet. Wirklich zuverlässig funktionierts bei mir nur mit Feld 4. Die 7 geht jetzt z.B. auch wieder nicht. Wenn ich nachsehe, ist aber Bild ausgewählt...

Ich hab auf dem Testserver mal den kompletten Baker und den DB-Dump in ein Verzeichnis "msfrog" gelegt. Ist nur mein Testsystem, an dem anderen hab ich jetzt erstmal nichts mehr gemacht.

dbs

Ich widerrufe. Habe jetzt 4 Bilder drin. Felder 4,7,8,9
Geht.
Was auffiel war, dass zwischendurch ein Bild Feld von "Bild" zu "nicht genutzt" sprang.
Nach Umstellen auf Bild und speichern ging alles wie gewünscht.
[url="https://onkel-franky.de"]https://onkel-franky.de[/url]

hgs

Hab dir ne PM geschickt
LG Harald

"Fange nie an, aufzuhören - höre nie auf, anzufangen." Marcus Tullius Cicero (106-43 v.Chr.)