search

ithanshui / jquery-multifile-plugin

Automatically exported from code.google.com/p/jquery-multifile-plugin
0 stars 0 forks source link

Bug Fix IE browsers #120

Open GoogleCodeExporter opened 8 years ago

GoogleCodeExporter commented 8 years ago 
Hello,

This module doesn't work with IE 6, 7, 8 because input.files.length doesn't 
exists.

To fix this problem, you need to replace at line ~ 300 :

for ( var i = 0; i < slave.files.length; i++) {
            v = String(slave.files[i].name || ''/*.attr('value)*/);
            a += '<span class="MultiFile-title" title="'+MultiFile.STRING.selected.replace('$file', v)+'">'+MultiFile.STRING.file.replace('$file', v.match(/[^\/\\]+$/gi)[0])+'</span>';
            if(i < slave.files.length-1){
                a += '<br />';
            }

BY :

if (typeof slave.files === "undefined"){
        v = String(slave.value || ''/*.attr('value)*/);
        a += '<span class="MultiFile-title" title="'+MultiFile.STRING.selected.replace('$file', v)+'">'+MultiFile.STRING.file.replace('$file', v.match(/[^\/\\]+$/gi)[0])+'</span>';
    }else{
           for ( var i = 0; i < slave.files.length; i++) {
            v = String(slave.files[i].name || ''/*.attr('value)*/);
            a += '<span class="MultiFile-title" title="'+MultiFile.STRING.selected.replace('$file', v)+'">'+MultiFile.STRING.file.replace('$file', v.match(/[^\/\\]+$/gi)[0])+'</span>';
            if(i < slave.files.length-1){
                a += '<br />';
            }
        }         
        }

Good luck !

Original issue reported on code.google.com by Gaar...@gmail.com on 11 Aug 2011 at 10:09

GoogleCodeExporter commented 8 years ago 
The project has been dormant for over a year and many of the support requests 
have been resolved and/or are invalid. So we're going to archive them all, 
pending review.

If you're still having a problem with the plugin, please visit the project page 
and re-open the issue. We do apologize for any inconvenience this may cause, 
but please bear with us...  we'll be making improvements to the plugin and its 
documentation over the coming weeks.

Stay tuned; and thanks for your support!

The Fyneworks Team
http://twitter.com/fyneworks

Original comment by fyneworks on 26 Jan 2012 at 1:37