Hi.
Upper case image extentions are being rejected by the module. But what if camera makes them UPPER CASE? I don't want to edit every file manually and I believe nobody wants.

Could you please update your module Line #412 uc_option_images.module to include strtolower() function like in the working example below:

    if (!in_array(strtolower(pathinfo($file->filename, PATHINFO_EXTENSION)), array('png', 'jpg', 'gif'))){

This is also would be good to have an option to adjust (or read from php.ini) the maximum file size to upload as this is hard coded at the moment

Actually I am uploading changed file if someone has similar problem with maximum file size and uppercase image extensions. (remove .zip extension - the file isn't compressed)

Comments

ivrh’s picture

ivrh commented
Category: task » bug

changing to bug report instead of task.

tjholowaychuk’s picture

tjholowaychuk commented

Interesting, strange that a camera would do that, very odd convention! This really should be a mime check not an extension check but I do not have time to adjust this (anyone willing to maintain certainly can)

acouch’s picture

acouch commented
Status: Needs review » Closed (won't fix)

The 5.x version is no longer being supported for this module. We are cleaning up the 6.x version and preparing for the 7.x version.