diff --git a/lib/javascript-static.js b/lib/javascript-static.js index bf5ecf21e72..84bf7bbceb5 100644 --- a/lib/javascript-static.js +++ b/lib/javascript-static.js @@ -1212,9 +1212,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()');