Debugmodus 3.0

[Offe]

Hallo zusammen,

in den alten 2.4.x Stableversionen war der Debugmodus standardmäßig aus. Seit der 3.0 Betaphase wird in den Systemeinstellungen auch in den Stableversionen dieser als ein angezeigt. In der comman.php habe ich jedoch gelesen, dass dieser Wert eigentlich standardmäßig auf 0 gesetzt ist, wenn dies in der config.php nicht anders deklariert ist. In der config.php habe ich keien Einstellungen zu $gDebug getroffen, demzufolge müsste der Debugmodus aus sein.

In der Preferences.php ist meiner Einschätzung nach in Zeile 471 ein Fehler:

if(isset($gDebug))

müsste durch

if($gDebug)

ersetzt werden.

Gruß Offe

[XimeX]

Ja da hast du recht. Wird in 3.0.2 gefixed.

https://github.com/Admidio/admidio/issues/90

[Offe]

Hallo XimeX,

ich habe gesehen, dass ihr für den Fehler einen zweiten Fix eingestellt habt: https://github.com/Admidio/admidio/comm ... ff3c1f1a3f

In der common.php wird ab Zeile 21 der isset-Fall bereits abgehandelt, sodass meiner Meinung nach der zweite Fix nicht notwendig ist:

Code: Alles auswählen

// if there is no debug flag in config.php than set debug to false
if(isset($gDebug) == false || $gDebug != 1)
{
    $gDebug = 0;
}

Gruß Offe

[XimeX]

doppelt hält besser
Fasse hat gesagt ich soll den 2 fix einbauen. Mal schaun was er dazu sagt

[fasse]

Die Stelle muss trotzdem drin bleiben. Ist $gDebug nicht gesetzt, dann würde es hier einen harten Fehler geben, da diese Variable nicht bekannt ist. Durch die dann negative isset Prüfung hier, wird der 2. Fall garnicht geprüft und es kommt nicht zu einem Fehler.