CMSimple_XH–Forum

to whom it may concern...

Habe unter xampp mal eine cmsimple_xh 1.7.2 Basisinstallation mit PHP 7.4.1 im Debug-Mode laufen lassen.
Anbei ein Patch mit ein paar Änderungen, um die gemeldeten Warnings/Errors zu beseitigen.
Bei der Anpassung im updatecheck bin ich mir nicht ganz sicher...

Whoopsadaisy wrote: ↑

Thu Feb 27, 2020 12:43 pm

Anbei ein Patch mit ein paar Änderungen, um die gemeldeten Warnings/Errors zu beseitigen.

Danke für den Patch. Sieht gut aus!

Whoopsadaisy wrote: ↑

Thu Feb 27, 2020 12:43 pm

Bei der Anpassung im updatecheck bin ich mir nicht ganz sicher...

Ich auch nicht. $pluginloader_cfg wurde schon vor sehr langer Zeit entfernt, so dass der entsprechende Teil der Bedingung wohl ganz entfallen könnte.

Bezüglich der getimagesize() Warnung: man könnte Dateien mit weniger als 12 Byte wohl wirklich von dieser Prüfung ausschließen, wie im Patch getan. Andererseits finde ich es aber auch nicht wirklich gut, dass PHP hier unter Umständen eine Notice erzeugt. Umgekehrt vielleicht auch nicht mehr so problematisch, da relevante Notices immer mehr zu Warnings angehoben werden, so dass man vielleicht irgendwann Notices weitestgehend ignorieren kann.

Getestet mit PHP 7.4.2.0
- alle Debug-Meldungen sind weg
SUPER!

Wer ist denn der anonyme Code-Spender?
Solche Leute werden bei uns dringend gebraucht!
Irgendwer muss es ja auch auf Github einarbeiten.

@Christoph:
Wieder aktiv ???

Das sind ja mal zwei sehr erfreuliche Posts.
Anpassung an 7.4 und ein Lebenszeichen von Christoph.

Mit filesize($path)>12 bin ich wohl etwas über das Ziel hinausgeschossen.
Ein filesize($path)>11 reicht wohl auch schon. Damit ist es wohl ähnlich
zu dem Verhalten der Funktion exif_imagetype.
Siehe hier: https://www.php.net/manual/de/function. ... .php#79283

Der Inhalt des zip basiert wohl auf einem Tool. Darf man erfahren auf welchem ?

Das ist das diff Kommando aus der Unix-Welt.

diff -u -r <Verzeichnis1> <Verzeichnis2>

Für Windows gibt es auch diverse Binaries im Netz zu finden.
Ich habe es von hier: https://www.di-mgt.com.au/wdiff-for-windows.html

Mit der Option -u erhält man einen Output, den man als Input für den patch Befehl
verwenden kann.

@cmss: Das git Binary hat auch einen diff Befehl an Board.

Ich hatte schon an einen kostenpflichtigen dienst wie Scrutinizer (bzw https://chrome.google.com/webstore/deta ... jbcnhkbflg oder Xdebug https://chrome.google.com/webstore/deta ... dgkmoaaaoc) gedacht. In phpstorm werden seitens der Plugins auch nur Verbesserungsvorschläge gemacht.
Die Lösungen (-,+) sind aber doch wohl händisch gemacht ?

Die Änderungen... ja da bin ich einfach die Liste der Meldungen im Debugmodus durchgegangen und habe einzeln nach Lösungen gesucht.