Wenn ich die Funktion cApiStrCleanURLCharacters testhalber aufrufe, landet folgender Eintrag in der deprecatedlog.txt:
Code: Alles auswählen
// Mit Stacktrace
Deprecated call: cApiStrCleanURLCharacters() [functions.api.string.php(251)]: "This method is deprecated and is not needed any longer"
cApiStrCleanURLCharacters() called in file cli.php(23)
// Ohne Stacktrace
Deprecated call: cApiStrCleanURLCharacters() [functions.api.string.php(251)]: "This method is deprecated and is not needed any longer"
Mit der vorgeschlagenen Änderung wäre das Ergebnis wie folgt:
Code: Alles auswählen
// Mit Stacktrace
Deprecated call: cApiStrCleanURLCharacters() [functions.api.string.php(251)]: "Function cApiStrCleanURLCharacters is deprecated and is not needed any longer"
cApiStrCleanURLCharacters() called in file cli.php(23)
// Ohne Stacktrace
Deprecated call: cApiStrCleanURLCharacters() [functions.api.string.php(251)]: "Function cApiStrCleanURLCharacters is deprecated and is not needed any longer"
Das sind in beiden Fällen, also mit und ohne Stacktrace, zu viele Vorkommen von "cApiStrCleanURLCharacters" in einem Logeintrag. Die Funktion cDeprecated() übernimmt schon den Namen der aufrufenden Stelle, die betroffene Datei und die Zeile und daraus kann man ableiten wo die cDeprecated() aufgerufen wurde.
Mit dem jetzigen Status bin ich auch nicht sehr glücklich, das könnte man besser und verständlicher machen. Ich denke, wir sollten uns überlegen, welche Informationen wir in die Log-Einträge (error, warning, deprecated) schreiben wollen und wie das Format genau sein soll.
Wie im Beitrag "
function cDeprecated * ergänzung" angemerkt, macht es Sinn, das in den Funktionen cDeprecated(), cWarning(), cError() und cLogError() zu vereinheitlichen.