Why not use JFile::getExt($filename) with uploads
Gosling Cools
Junior Member
Posts: 20
8 lata 11 miesiąc temu #1078
przez Gosling Cools
Why not use JFile::getExt($filename) with uploads was created by 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:Regards,
Gosling
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]);
Gosling
Proszę Zaloguj lub Zarejestruj się, aby dołączyć do konwersacji.
Victor
Administrator
Posts: 19
8 lata 11 miesiąc temu #1079
przez Victor
Back-end developer on Component Creator
Replied by Victor on topic Why not use JFile::getExt($filename) with uploads
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
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
Proszę Zaloguj lub Zarejestruj się, aby dołączyć do konwersacji.
Gosling Cools
Junior Member
Posts: 20
8 lata 11 miesiąc temu #1080
przez Gosling Cools
Replied by Gosling Cools on topic Why not use JFile::getExt($filename) with uploads
Thanks!
But now there's an error in the code.
This line:Should be
And this line:Shoud/could be:Regards,
Gosling
But now there's an error in the code.
This line:
$filename[0] = preg_replace("/[^A-Za-z0-9]/i", "-", $filename);
$filename = preg_replace("/[^A-Za-z0-9]/i", "-", $filename);
And this line:
$filename = md5(time()) . '-' . array($filename, '.', $extension);
$filename = md5(time()) . '_' . $filename . '.' . $extension;
Gosling
Proszę Zaloguj lub Zarejestruj się, aby dołączyć do konwersacji.
Gosling Cools
Junior Member
Posts: 20
8 lata 11 miesiąc temu #1082
przez Gosling Cools
Replied by Gosling Cools on topic Why not use JFile::getExt($filename) with uploads
And now remove the last ) please in:
//Add Timestamp MD5 to avoid overwriting
$filename = md5(time()) . '-' . $filename . '.' . $extension);
Proszę Zaloguj lub Zarejestruj się, aby dołączyć do konwersacji.
Gosling Cools
Junior Member
Posts: 20
8 lata 11 miesiąc temu #1084
przez Gosling Cools
Replied by Gosling Cools on topic Why not use JFile::getExt($filename) with uploads
this line:should be
$filename = md5(time() . '-' . $filename . '.' . $extension);
$filename = md5(time()) . '-' . $filename . '.' . $extension;
Proszę Zaloguj lub Zarejestruj się, aby dołączyć do konwersacji.
Victor
Administrator
Posts: 19
8 lata 11 miesiąc temu #1086
przez Victor
Back-end developer on Component Creator
Replied by Victor on topic Why not use JFile::getExt($filename) with uploads
Done!
Back-end developer on Component Creator
Proszę Zaloguj lub Zarejestruj się, aby dołączyć do konwersacji.
Czas generowania strony: 0.055 s.