Multi file upload validaiton javascript if statement not working

Question

Trying to add validation for my files to see if the files being uploaded exceed 5MB, however it doesnt alert if the file is over 5MB, Can anyone see what is wrong with my validaiton in my IF statement? This is a multi file upload which works fine but the validation doesnt

function makeProgress(number){   
          var url = getRelativeURL("web/fileUpload");   
          var formData = new FormData();
          formData.append('number', number);
          fls = document.getElementById("attachmentFileUploadInput").files; //number of files... 
          console.log(fls);
          for(j=0;j<fls.length;j++){
              if (fls.size > 5000000) //5MB
            {
                  console.log('error');
                  alert('file to big');
            }
              else
                  {
              formData.append('files[]', fls[j]);  //note files[] not files
              $.ajax({
                  url : url,
                  data : formData,

fls is an array of files? so, to get size of each indivial file:fls[j].size — yezzz
– yezzz, Commented Apr 15, 2017 at 12:19

elixenide · Accepted Answer · 2017-08-07 02:52:19Z

1

You don't check the size of the current file. Change

if (fls.size > 5000000) //5MB

to

if (fls[j].size > 5000000) //5MB

One more think. You don't declare j in your function and therefore it is in the global scope which can break things because it can be manipulated in some other parts of your code. You can declare it like this:

var j;
for(j=0;j<fls.length;j++){

edited Aug 7, 2017 at 2:52

elixenide

44.9k16 gold badges79 silver badges103 bronze badges

answered Apr 15, 2017 at 12:25

Johannes Filter

1,9313 gold badges21 silver badges25 bronze badges

Sign up to request clarification or add additional context in comments.

Collectives™ on Stack Overflow

Multi file upload validaiton javascript if statement not working

1 Answer 1

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

Comments

Your Answer

Sign up or log in

Post as a guest

Related