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

8 yıl 11 ay önce #1078 Yazan: Gosling Cools
Why not use JFile::getExt($filename) with uploads, Gosling Cools tarafından oluşturuldu
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

Lütfen sohbete katılmak için Giriş ya da Hesap açın.

8 yıl 11 ay önce #1079 Yazan: Victor
Victor tarafından Why not use JFile::getExt($filename) with uploads konusunda yanıtlandı
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

Lütfen sohbete katılmak için Giriş ya da Hesap açın.

8 yıl 11 ay önce #1080 Yazan: Gosling Cools
Gosling Cools tarafından Why not use JFile::getExt($filename) with uploads konusunda yanıtlandı
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

Lütfen sohbete katılmak için Giriş ya da Hesap açın.

8 yıl 11 ay önce #1082 Yazan: Gosling Cools
Gosling Cools tarafından Why not use JFile::getExt($filename) with uploads konusunda yanıtlandı
And now remove the last ) please in:
//Add Timestamp MD5 to avoid overwriting
$filename = md5(time()) . '-' . $filename . '.' . $extension);
:-)

Lütfen sohbete katılmak için Giriş ya da Hesap açın.

8 yıl 11 ay önce #1084 Yazan: Gosling Cools
Gosling Cools tarafından Why not use JFile::getExt($filename) with uploads konusunda yanıtlandı
this line:
$filename = md5(time() . '-' . $filename . '.' . $extension);
should be
$filename = md5(time()) . '-' . $filename . '.' . $extension;

Lütfen sohbete katılmak için Giriş ya da Hesap açın.

8 yıl 11 ay önce #1086 Yazan: Victor
Victor tarafından Why not use JFile::getExt($filename) with uploads konusunda yanıtlandı
Done! :)

Back-end developer on Component Creator

Lütfen sohbete katılmak için Giriş ya da Hesap açın.

Sayfa oluşturma süresi: 0.057 saniye
Geliştiren: Kunena Forum