Offen php 5.4 - Fehlermeldung: mktime() durch time() ersetzen in boxen.php

[Der_Bob]

Wir sind gestern auf einen neuen Server gewechselt mit php 5.4 (vorher 5.2) und ich bekomm auf unserer Test-Umgebung seitdem den Fehler:

"Strict Standards: mktime(): You should be using the time() function instead in ../includes/boxen.php on line 989"
(auf der Test-Umgebung lass' ich mir die Fehler ausgeben)


Der Fehler scheint undkritisch zu sein, aber wird eben als php-Fehler erkannt (auch von Google?), daher würde ich ihn gerne beseitigen. Wenn ich das richtig verstehe, macht mktime() (ohne Parameter) auch eh das Gleiche wie time()

Weiß jemand mehr?

 

[martinwolf]

AW: php 5.4 - Fehlermeldung: mktime() durch time() ersetzen in boxen.php

Wir sind gestern auf einen neuen Server gewechselt mit php 5.4 (vorher 5.2) und ich bekomm auf unserer Test-Umgebung seitdem den Fehler:

"Strict Standards: mktime(): You should be using the time() function instead in ../includes/boxen.php on line 989"
(auf der Test-Umgebung lass' ich mir die Fehler ausgeben)


Der Fehler scheint undkritisch zu sein, aber wird eben als php-Fehler erkannt (auch von Google?), daher würde ich ihn gerne beseitigen. Wenn ich das richtig verstehe, macht mktime() (ohne Parameter) auch eh das Gleiche wie time()

Weiß jemand mehr?

Dann änder doch besagte Funktion in boxen.php in Zeile 989 von mktime zu time. Bedenke aber bei einem möglichen Update diesen Fix ebenfalls einzuspielen. Sofern JTL das beim nächsten Versionsupdate nicht eh schon selbst gemacht hat. Weiter sollte der Fehler von Google nur gesehen werden, wenn das Errorreporting in der defines_inc.php auch aktiviert ist. Aber das sollte generell auf 0 sitzen weil der Shop einfach zuviele PHP Notice wirft.

 

[Der_Bob]

AW: php 5.4 - Fehlermeldung: mktime() durch time() ersetzen in boxen.php

Dann änder doch besagte Funktion in boxen.php in Zeile 989 von mktime zu time. Bedenke aber bei einem möglichen Update diesen Fix ebenfalls einzuspielen. Sofern JTL das beim nächsten Versionsupdate nicht eh schon selbst gemacht hat.

Genau das ist ja die Frage, ob die Änderung unkritisch ist bzw. auch ein Hinweis an JTL für die nächste Version. Soweit ich das sehe, ist eine Änderung an dieser Stelle zu time() aber kein Problem.

Weiter sollte der Fehler von Google nur gesehen werden, wenn das Errorreporting in der defines_inc.php auch aktiviert ist. Aber das sollte generell auf 0 sitzen weil der Shop einfach zuviele PHP Notice wirft.

Da bin ich mir nicht sicher. Wir haben auf der Testumgebung Errorreporting auf 1 und im Live- Shop auf 0. Bei sitealert.de bekommen wir aber für den Live-Shop auch immer php-Fehler gemeldet, auch wenn das errorreporting deaktiviert ist und kein Fehler angezeigt wird.