Per the documentation, this module requires the client to be using Flash 9 or 10. While this covers the majority of users, there are users with older versions of flash. In my case, when I tried the module on an IE client with Flash 6, the upload icon would not appear and no other message was presented to the user.

Adobe provides a toolkit for detecting the client's flash version here:
http://www.adobe.com/products/flashplayer/download/detection_kit/

This is a request to enhance the module to provide additional detection of flash and providing a cleaner message to the user if they don't have a version high enough to work with image_fupload.

Comments and thoughts?

CommentFileSizeAuthor
#5 image_fupload-flash_version_detection-625904-2.patch5.03 KBraphael waeselynck
#2 image_fupload-flash_version_detection-625904-2.patch17.2 KBraphael waeselynck

Comments

jzornig’s picture

jzornig commented

Can I request that the module fall back gracefully in the case where the user does not have flash to still allow single image upload. Installing this module makes your site unusable if the user does not have flash.

raphael waeselynck’s picture

raphael waeselynck commented
StatusFileSize
new image_fupload-flash_version_detection-625904-2.patch17.2 KB

Hi,
Since this feature was a lack and I really needed it, I developed a solution. Please have a look at the attached patch and tell me if it is working fine for you.

rjbrown99’s picture

rjbrown99 commented
Status: Active » Needs review
grandcat’s picture

grandcat commented

The patch is corrupt (just removals). Please create agaian a patch against head.

raphael waeselynck’s picture

raphael waeselynck commented
StatusFileSize
new image_fupload-flash_version_detection-625904-2.patch5.03 KB

My apologies for the previous patch (-_-)", guess I was tired !
Here is a new version, hope this one is working, at least it does add code instead of removing this time ;-)
Do not forget to add the following js from SWFUpload : swfupload.swfobject.js, I did not include it in the patch, just modified the readme.txt to warn user they should include it.
Regards,
Raphael

grandcat’s picture

grandcat commented

Thank you very much! I'll do some testings next week.

dafeder’s picture

dafeder
he/him
Location Philadelphia, PA
commented

This is working for me, except for the fact that it allows people to add more than one image to their node now, whereas before image_fupload was creating a new node for each image. Working on a patch to only allow one value in the noflash version.

dafeder’s picture

dafeder
he/him
Location Philadelphia, PA
commented

I ended up doing this via hook_form_alter:

function mymodule_form_alter(&$form, &$form_state, $form_id) {
  if ($form_id == 'image_node_form') {
    $form['field_image'][0]['#delta'] = 1;    
    unset($form['field_image']['field_image_add_more']);    
  }
}