Änderung im Safe Mode Hack - unbedingt testen

[leo]

Heute ist mir ein kleiner Fehler in der Klasse system/libraries/FTP.php in der Methode rename() aufgefallen (Zeile 186).

// Unix fix: rename case sensitively
if (strcasecmp($strOldName, $strNewName) !== strcmp($strOldName, $strNewName))
{
    @ftp_rename($this->resConnection, $GLOBALS['TL_CONFIG']['ftpPath'] . $strOldName, $GLOBALS['TL_CONFIG']['ftpPath'] . $strOldName . '__');
    $strOldName .= '__';
} 

Diese if-Abfrage sollte nur ausgeführt werden, wenn ein Dateiname gleich bleibt und sich lediglich die Groß-/Kleinschreibung ändert. De facto wird sie aber immer ausgeführt, da nur geprüft wird, ob der alte und der neue Name unterschiedlich sind. Daher habe ich folgende Änderung vorgenommen:

// Unix fix: rename case sensitively
if (($strcasecmp = strcasecmp($strOldName, $strNewName)) == 0 && $strcasecmp !== strcmp($strOldName, $strNewName))
{
    @ftp_rename($this->resConnection, $GLOBALS['TL_CONFIG']['ftpPath'] . $strOldName, $GLOBALS['TL_CONFIG']['ftpPath'] . $strOldName . '__');
    $strOldName .= '__';
} 

Dadurch wird zuerst geprüft, ob sich der Dateiname ändert, und nur wenn das nicht der Fall ist ($strcasecmp == 0), wird die weitere Anweisung ausgeführt. Falls ihr den Safe Mode Hack verwendet, bitte testet diese Änderung ausführlich und gebt mir hier Feedback. Vielen Dank.

[xtra]

Hi leo(s),
Der Bugfix ist bei mir auch valide, jedoch habe ich es noch nicht durch verschiedene locales durchgejagt. Da hierbei jedoch binarysafe auf unity gepryft wird, wird es auf jeden Fall bei einer case change ausgefyhrt.

Jedoch scheine ich eine Denkblockade zu haben, was spricht bitte gegen:

if ((strcasecmp($strOldName, $strNewName) === 0) && (strcmp($strOldName, $strNewName) !== 0)))
{
    @ftp_rename($this->resConnection, $GLOBALS['TL_CONFIG']['ftpPath'] . $strOldName, $GLOBALS['TL_CONFIG']['ftpPath'] . $strOldName . '__');
    $strOldName .= '__';
} 

Vermutlich bin ich gerade aber auch einfach zu myde um einen Fehler in meinem code zu entdecken.

[leo]

Nichts eigentlich.

Macht wahrscheinlich für den Compiler keinen Unterschied, ist aber für den Menschen einfacher nachzuvollziehen. Ich habe den Code also wie folgt geändert:

// Unix fix: rename case sensitively
if (strcasecmp($strOldName, $strNewName) === 0 && strcmp($strOldName, $strNewName) !== 0)
{
    @ftp_rename($this->resConnection, $GLOBALS['TL_CONFIG']['ftpPath'] . $strOldName, $GLOBALS['TL_CONFIG']['ftpPath'] . $strOldName . '__');
    $strOldName .= '__';
} 

Bitte trotzdem nochmal testen. Man kann ja nie wissen …

[leo]

Eventuell ist die Funktion move_uploaded_files() nicht binary-safe?

[Nina]

Sollen wir hier noch testen, oder ist diese Sache geklärt?

PS: Hab die andere Thematik in ein eigenes Thema geschoben, da hier sonst die Ursprungsthematik untergeht.