Why not use JFile::getExt($filename) with uploads

9 Jahre 9 Minuten her #1078 von Gosling Cools
Why not use JFile::getExt($filename) with uploads wurde erstellt von Gosling Cools
Hi,

Is there a reason why you don't use JFile::getExt($filename) and JFile::stripExt($filename) in the \tables\XXXXX.php files?
For instance a filename with multiple dots will explode wrongly with your code:
//Replace any special characters in the filename
                  $filename = explode('.', $file['name']);
                  $filename[0] = preg_replace("/[^A-Za-z0-9]/i", "-", $filename[0]);
Regards,

Gosling

Bitte Anmelden oder Registrieren um der Konversation beizutreten.

8 Jahre 11 Monate her #1079 von Victor
Hi Gosling,

Thank you for pointing out this issue. We have patched our system to use Joomla functions instead of PHP ones.

Best regards,

Víctor

Back-end developer on Component Creator

Bitte Anmelden oder Registrieren um der Konversation beizutreten.

8 Jahre 11 Monate her #1080 von Gosling Cools
Thanks!

But now there's an error in the code.

This line:
$filename[0] = preg_replace("/[^A-Za-z0-9]/i", "-", $filename);
Should be
$filename = preg_replace("/[^A-Za-z0-9]/i", "-", $filename);

And this line:
$filename = md5(time()) . '-' . array($filename, '.', $extension);
Shoud/could be:
$filename = md5(time()) . '_' . $filename . '.' . $extension;
Regards,

Gosling

Bitte Anmelden oder Registrieren um der Konversation beizutreten.

8 Jahre 11 Monate her #1082 von Gosling Cools
And now remove the last ) please in:
//Add Timestamp MD5 to avoid overwriting
$filename = md5(time()) . '-' . $filename . '.' . $extension);
:-)

Bitte Anmelden oder Registrieren um der Konversation beizutreten.

8 Jahre 11 Monate her #1084 von Gosling Cools
this line:
$filename = md5(time() . '-' . $filename . '.' . $extension);
should be
$filename = md5(time()) . '-' . $filename . '.' . $extension;

Bitte Anmelden oder Registrieren um der Konversation beizutreten.

8 Jahre 11 Monate her #1086 von Victor
Done! :)

Back-end developer on Component Creator

Bitte Anmelden oder Registrieren um der Konversation beizutreten.

Ladezeit der Seite: 0.056 Sekunden
Powered by Kunena Forum

Wir verwenden eigene Cookies und Cookies von Drittanbietern, um Ihr Nutzererlebnis zu verbessern und Ihnen einen optimalen Service zu bieten. Wenn Sie die Website weiter nutzen, gehen wir davon aus, dass Sie mit unserer Cookie-Politik einverstanden sind.