EDIT: Just in case this reply shows up twice, when I originally posted it, it went missing entirely. So, I'm posting it again. Thankfully, my browser back button enabled me to recover my reply content, else I would not have had the energy to write this all over again.
Summary: You cannot have spaces in the name.
To learn why, let's take a closer look at what's going on under the hood. Let's use a filename with a space in the name: test prompt.wav
When you select a file on your computer, with the browse button, the browser exposes the filename to the javascript like this: C:\\fakepath\\test prompt.wav
In the code, it grabs a reference to this value with the following line of code:
var srcFile = document.FORM1.file1.value;
From here, the code tries to strip off the prefix with the fake path with this line of code, so that all we're left with is the filename itself: test prompt.wav
srcFile = srcFile.substring(srcFile.lastIndexOf("\\")+1,srcFile.length);
Then, the code tries to normalize the data for transfer to the server, by running the filename through an encoder, like so:
srcFile = encodeURIComponent(srcFile);
The details of the function encodeURIComponent are unimportant, but the outcome is. The space in the filename gets converted to %20, the ASCII Hex value for the Space character.
So, at this point, our filename looks like this, to the code: test%20prompt.wav
Now, there's some other things going on in the code at this point, but they are not important for our understanding of why filenames with spaces in them are breaking your upload. E.g., Trimming leading and trailing white space off the name.
Last but not least, the star of the show, the error message you are seeing is produced by the outcome of the following function call:
isInvalidString(srcFile,filenameSplChars)
The details of this function, again, are unimportant, but the outcome is. The results of this call are true, because the filename is compared against filenameSplChars, which holds the following value:
"&%*?\"<>|'"
Notice anything familiar in there? You may have noticed the percent sign (%) is in that list. So, when this section of code in the function isInvalidString is ran, we return true for the percent sign (%).
for (var i = 0; i < mask.length; i++) {
if (s.indexOf(mask.charAt(i)) > -1)
return true; // a character was found that is not valid
}
And that's the story of how a Voice Engineer identified a Cisco software defect. Either the alert message should contain the space character, or the code should check isInvalidString before running encodeURIComponent.
