diff --git a/lib/javascript-static.js b/lib/javascript-static.js index c9d2efd19e6..91b75f04e15 100644 --- a/lib/javascript-static.js +++ b/lib/javascript-static.js @@ -1109,9 +1109,12 @@ function openpopup(event, args) { } } - // Cleans window name because IE does not support funky ones. + // Make sure the name argument is set and valid. var nameregex = /[^a-z0-9_]/i; - if (args.name.match(nameregex)) { + if (typeof args.name !== 'string') { + args.name = '_blank'; + } else if (args.name.match(nameregex)) { + // Cleans window name because IE does not support funky ones. args.name = args.name.replace(nameregex, '_'); if (M.cfg.developerdebug) { alert('DEVELOPER NOTICE: Invalid \'name\' passed to openpopup()');