Question

This is the part of code:

inprogress = false;

function getid(id) {
    return document.getElementById(id); 
}

var uploader = new plupload.Uploader({
    runtimes : 'gears,html5,flash,silverlight,browserplus',
    browse_button : 'link-browse',
    max_file_size : '100mb',
    url : 'site/upload/process.php?dir=<?php echo $uplid; ?>',
    flash_swf_url : 'site/upload/plupload.flash.swf',
    silverlight_xap_url : 'site/upload/plupload.silverlight.xap',
});
uploader.bind('Init', function(up, params) {
    //$('filelist').innerHTML = "<div>Current runtime: " + params.runtime + "</div>";
});
uploader.bind('FilesAdded', function(up, files) {
    if(uploader.files.length <= 0){
        var element = document.getElementById('standby');
        element.parentNode.removeChild(element);
    }
    if(up.files.length > 4 || uploader.files.length > 4)
    {
        alert('Only 5 files per upload !');

        return false;
    }
    for (var i in files) {
        getid('filelist').innerHTML += '<div class="item" id="' + files[i].id + '"><div class="name">' + files[i].name + '</div><div onclick="removeme(\''+files[i].id+'\')" id="remove-'+files[i].id+'" class="remove"></div><div class="size">[ ' + plupload.formatSize(files[i].size) + ' ]</div><div class="percent"></div></div>';
    }
});
uploader.bind('UploadFile', function(up, file) {
    getid('submit-form').innerHTML += '<input type="hidden" name="file-' + file.id + '" value="' + file.name + '" />';
});
uploader.bind('UploadProgress', function(up, file) {
    getid(file.id).getElementsByTagName('div')[3].innerHTML = '<span>' + file.percent + "%</span>";
});
uploader.bind('StateChanged', function(uploader) {
        if (uploader.files.length === (uploader.total.uploaded + uploader.total.failed)) {
         window.location = "./dl/<?php echo $uplid; ?>"
        }
    });
getid('link-upload').onclick = function() {
    if(uploader.files.length < 1){
        alert('Please select files first.');
        return false;
    }
    inprogress = true;
    uploader.start();
    return false;
};
uploader.init();
function removeme(id){
    if(inprogress) return false;
    if(uploader.files.length == 1)
    getid('filelist').innerHTML += '<div id="standby"></div>';

    var element = document.getElementById(id);
    element.parentNode.removeChild(element);

    var toremove = '';

    for(var i in uploader.files){
        if(uploader.files[i].id === id){
            toremove = i;
        }
    }
    uploader.files.splice(toremove, 1);
}

I can limit of files being uploaded,
And if I have 4 files selected, and I select 5 more -> it will show error

but if I at first select for example 14 files, they will be shown in “filelist”.

How to limit that, or where to put “return false”;

Thanks for any help 🙂

Answer 1

Expand if(up.files.length > 4 || uploader.files.length > 4) to if(up.files.length > 4 || uploader.files.length > 4 || files.length > 4).