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

Gosling Cools

Junior Member

Posts: 20

9 năm 1 tuần trước #1078 bởi 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:

//Replace any special characters in the filename
                  $filename = explode('.', $file['name']);
                  $filename[0] = preg_replace("/[^A-Za-z0-9]/i", "-", $filename[0]);

Regards,

Gosling

Vui lòng Đăng nhập hoặc Tạo tài khoản để tham gia cuộc hội thoại.

Victor

Administrator

Posts: 19

9 năm 5 ngày trước #1079 bởi Victor

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

Back-end developer on Component Creator

Vui lòng Đăng nhập hoặc Tạo tài khoản để tham gia cuộc hội thoại.

Gosling Cools

Junior Member

Posts: 20

9 năm 5 ngày trước #1080 bởi 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:

$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

Vui lòng Đăng nhập hoặc Tạo tài khoản để tham gia cuộc hội thoại.

Gosling Cools

Junior Member

Posts: 20

9 năm 5 ngày trước #1082 bởi 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);

Vui lòng Đăng nhập hoặc Tạo tài khoản để tham gia cuộc hội thoại.

Gosling Cools

Junior Member

Posts: 20

9 năm 5 ngày trước #1084 bởi Gosling Cools

Replied by Gosling Cools on topic Why not use JFile::getExt($filename) with uploads

this line:

$filename = md5(time() . '-' . $filename . '.' . $extension);

should be

$filename = md5(time()) . '-' . $filename . '.' . $extension;

Vui lòng Đăng nhập hoặc Tạo tài khoản để tham gia cuộc hội thoại.

Victor

Administrator

Posts: 19

9 năm 5 ngày trước #1086 bởi Victor

Replied by Victor on topic Why not use JFile::getExt($filename) with uploads

Done!

Back-end developer on Component Creator

Vui lòng Đăng nhập hoặc Tạo tài khoản để tham gia cuộc hội thoại.

Thời gian tải trang: 0.058 giây