Procalendar Bug

hgs

Der DB Dump ist immer gut, das es ab Feld 8 nicht mehr geht, können wir bestätigen (siehe post dbs  :-o).
Also gib mal dein geändertes Modul mit dem DB Dump und wir schauen, was möglich ist.
Gerne auch als Mail, wenn du es nicht öffentlich posten möchtest.
Eine FTP-Zugang kann ich dir auch exclusiv anlegen,
alles geht - nichts muss
LG Harald

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

dbs

Habe Bild in Felder 4,7,8 und erhalte nun auch deine Meldung.
There was an uncatched exception
Argument 1 passed to ReplaceAbsoluteMediaUrl() must be of the type string, null given, called in .../modules/procalendar/save.php on line 257
in line (23) of (/modules/procalendar/functions.php):


[url="https://onkel-franky.de"]https://onkel-franky.de[/url]

msfrog

Hallo,
ich habs auch mal probiert. Feld 7 ging auch bei mir. Ich hab jetzt mal alle durchprobiert. Bei 4 und 7 hats funktioniert, bei den anderen nicht.

FTP-Zugang kann ich leider nicht machen, das ist ein Produktivsystem... Ich kann aber gern mal das Modul packen. Soll ich die DB-Tabellen auch mit dazupacken?

hgs

Nachtrag
Habe mit der Version 1.7.3 dev3 gerade getestet und ein zusätzliches "Bildfeld" an Position 7 eingefügt (war leer)
Es werden beide Bilder sowohl vom lokalen PC als auch vom Webserver übernommen (alle Möglichkeiten über Kreuz durchgetestet), gespeichert und auch im FE angezeigt.
LG Harald

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

hgs

Moin zusammen
Ganz vorne im Betrag hatte ich schon mal angeboten, dein Modul mit allen Änderungen zu packen und hier zur Verfügung zu stellen.

Wir haben natürlich nur in der Defaulteinstellung getestet und da gibt es 2 Möglichkeiten ein Bild in den Infoblock zu platieren, eine Möglichkeit davon hat nicht funktioniert und das wurde gefixt. (Es passte zu deiner Fehlerbeschreibung und so haben wir überlesen, dass du ein zusätzliches Custem-Feld angelegt hast.

Deshalb benötigen wir entweder
• Super-Admin und FTP Zugriff auf deine Testseite (am besten eine 1:1 Kopie der Liveseite) Idelal, da wir leider nicht die Möglichkeit einer Umlautdomäne  haben.
• Dein angepasstes Modul (wir könnten testen ob es auf unseren Testservern läuft)

Die 1. Option ist u.E. immer vorzuziehen, um auch alle "Servereinstellungen" vom Original zu haben.

Und bis jetzt haben wir noch alle Probleme erfolgreich hinbekommen.
Zugangsdaten, wenn du uns dein Vertrauen schenkst, bitte an Luisehahne, dbs oder mich per pm
LG Harald

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

msfrog

Hallo Dietmar,
hab ich zwar, aber immer nur dann an, wenn ichs grad mal brauche. E-Mail, Signal oder Telegram könnt ich sonst noch bieten.

Noch ne Sache, die mir aufgefallen ist... Versuche ich ein GIF hochzuladen, kommt eine Fehlermeldung (bei der hinten ein a zu viel steht). Klicke ich bei der Meldung auf Schließen, wird versucht die /modules/procalendar/index.php aufzurufen, was in einem "Not Found" resultiert.

Luisehahne

Danke für dein Feedback, wäre nett wenn wir in Verbindung bleiben und du für uns den Procalendar testet. Hast du Skype??

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

msfrog

Guten Morgen!
Vielen Dank für eure Mühe, auf dem gestern aufgesetzten Testsystem geht es - unter Vorbehalt. Folgendes passiert:

Ich hatte das Testsystem auf den Default-Einstellungen gelassen. Dabei ist ja bereits ein Bild-Custom-Feld vorkonfiguriert. Damit funktioniert das Ganze auch, die Bilder werden übernommen. Als ich die dev3 dann auf dem anderen System installierte, kam wieder die Fehlermeldung. Meine Vermutung war, dass es evtl. mit der Konfiguration des Addons zusammenhängt, also hab ich Schritt für Schritt die Einstellungen auf das Testsystem übernommen. Nachdem ich die Custom-Felder angepasst hatte, trat auch der Fehler wieder auf.

Offenbar funktioniert das mit den Bildern ausschließlich dann, wenn das Bild in "Eigenes Feld 4" liegt. Ich hab noch 3 und 5 getestet, bei beiden kommt dann der Fehler, sobald man einen Termin speichern will.

Beim Testen ist mir noch ein kleiner Fehler aufgefallen. Beim Löschen eines Termins überprüft das System, ob alle Pflichtfelder ausgefüllt sind. Macht imho wenig Sinn.

dbs

Damit gehts aber nun hoffentlich.
Evtl noch Probleme wegen Umlautdomain, weil wir das nicht testen können.
[url="https://onkel-franky.de"]https://onkel-franky.de[/url]

dbs

Du liegst da schon richtig.
Bei neuinstall oder upgrade wird die install_struct.sql verwendet.
In unseren Tests tauchte kein Problem auf. Wahrscheinlich weil immer nur upgrade.
Chef kümmert sich.
[url="https://onkel-franky.de"]https://onkel-franky.de[/url]

msfrog

Ich hab mir mal die install_struct.sql angesehen. Zeile 130 schließt mit nem ; ab, müsste aber eigentlich ein , sein, denke ich. Leider werden die Datenbanken trotzdem nicht installiert, da muss noch was anderes schief laufen...

msfrog

#20
Die dev funzt leider auch nicht. Ich würds mal so machen: Morgen leg ich ne neue Subdomain an und installier mal nen nackigen WebsiteBaker mit der Procalendar 1.7.2 und teste es damit. Falls der Test wieder negativ ist, geb ich euch den Zugang zu dem Testsystem, damit ihr mal gucken könnt. Passt das? Was genau braucht ihr alles, reicht ein Login für den Baker?

Edit: Ich hab mich sofort mal drangesetzt. WB 2.13.0 r60 installiert, dann PC 1.7.2 dazu. Ergebnis: Der Admin des Kalenders wird gar nicht erst angezeigt, weil bei der Installation offenbar die Datenbank-Tabellen für PC nicht angelegt werden. Die 3 wb_mod_procalendar_xxx fehlen. Ich hab dann mal die 1.6.6 installiert und direkt zur 1.7.2 geupdatet. Der Admin wird jetzt angezeigt, aber beim Speichern eines Termins wird wieder der gleiche Fehler geworfen.

Wird evtl. beim Update eine nötige Änderung an der DB nicht gemacht? Welche könnte das sein?

Luisehahne

#19
Versuchst du bitte diese mal. Habe zur besseren Unterscheidung eine dev daraus gemacht

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

Luisehahne

Problem ist, keiner von uns hat die Möglichkeit auf einer Umlautedomain zu testen. Gibt es die Möglichkeit uns einen Testserver mit Umlatedomain oder Subdomain einzurichten?

Dann könnte ndbs und ich testen

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

msfrog

Hab nochmal drüberinstalliert, keine Änderung. Kanns evtl. dran liegen, dass das auf ner Umlautdomain läuft?

Ansonsten ist da nicht viel dahinter. Startdatum, Bezeichnung und 2 Custom-Felder sind noch angelegt und eine Kategorie ausgewählt. Also nichts Extravagantes...

dbs

Kann ich nicht bestätigen.
Was wählst du denn alles aus in dem Termin?
Kannst das Modul ganz normal installieren. Solange die installierte Version nicht größer ist, geht das.
[url="https://onkel-franky.de"]https://onkel-franky.de[/url]

msfrog

Habs wieder hochgeladen, upgrade.php laufen lassen und dann mal getestet. Wieder die Fehlermeldung, sobald ich einen Termin speichere.

dbs

#14
Wir hatten da mit mehreren Versionen der 1.7.1 zu tun.
Nur um sicherzugehen, dass du die letzte Version hast, probiere die angehänte nochmal. Einfach drüberinstallieren.
[url="https://onkel-franky.de"]https://onkel-franky.de[/url]

msfrog

#13
Vielen Dank! Habs gleich mal installiert. Jetzt kommt ne Fehlermeldung:

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

Kanns evtl. daran liegen, dass bei den einzelnen Options der Pfad so dargestellt wird?
<option value="{SYSVAR:MEDIA_REL}calendar/xyz.jpg">xyz.jpg</option>

hgs

#12
Ok, und hier die Antwort aus der Entwicklungsabteilu ng im Anhang zum download und testen.
Über ein Feedback würden wir uns freuen.
Es gab auch noch  "Verbesserungen". Wenn du es brauchst, kannst du jetzt unter Optionen--> Eigene Felder diese als "Zwangsfelder" ausweisen.
Damit werden die Autoren liebevoll zu Einträgen gezwungen.

Einfach "required" anwählen, fertig und schon ist diese Feld ein Feld, welches nicht mehr leer bleiben darf.
LG Harald

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

msfrog


hgs

#10
Da kommen wir der Sache doch schon näher, Das scheint noch ein Bug zu sein, Danke für die Meldung. Wir habe es an den Entwickler weiter gegeben.
LG Harald

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

msfrog

Genau. Hochgeladene Bilder werden übernommen, per Dropdown ausgewählte nicht. Wenn ich einen Termin bearbeite, der ein Bild hatte, z.B. indem ich den Text bearbeite und dann speichere, geht das Bild auch wieder verloren.

dbs

Moin,
gestern mit r60, PHP 7.4 und PC 1.7 ein Bild hochgeladen. Dieser Termin zeigte das Bild auch im Frontend.
Heute 2 weitere Termine angelegt und das gleiche Bild per Dropdown ausgewählt.
Bei allen 3 Terminen wird nun kein Bild angezeigt.
Ist das so wie bei dir?

[url="https://onkel-franky.de"]https://onkel-franky.de[/url]

msfrog

Danke, ich hab die Dateien hochgeladen und die update.php laufen lassen. Version wird jetzt auch als 1.7.0 ausgegeben. Leider ist der Fehler auch in dieser Version vorhanden.

Ich hab gestern mal in die save.php reingeschaut. Ich seh da nirgends Code, der dieses Input-Feld überhaupt behandelt. Kanns sein, dass das schlicht vergessen wurde?