Ticket #5278: 5278.patch

_source/lang/en.js

@@ -169 +169 @@
                 emailBody               : 'Message Body',
                 noAnchors               : '(No anchors available in the document)',
                 noUrl                   : 'Please type the link URL',
-                noEmail                 : 'Please type the e-mail address'
+                noEmail                 : 'Please type the e-mail address',
+                validatePopupName       : 'Please remove white space in Popup Window Name'      
         },
 
         // Anchor dialog

_source/plugins/link/dialogs/link.js

@@ -91 +91 @@
                 /\s*window.open\(\s*this\.href\s*,\s*(?:'([^']*)'|null)\s*,\s*'([^']*)'\s*\)\s*;\s*return\s*false;*\s*/;
         var popupFeaturesRegex = /(?:^|,)([^=]+)=(\d+|yes|no)/gi;
 
+        var targetFrameRegex = /\s+/g;
+        
         var parseLink = function( editor, element )
         {
                 var href = element ? ( element.getAttribute( '_cke_saved_href' ) || element.getAttribute( 'href' ) ) : '',
@@ -776 +778 @@
                                                                         if ( data.target )
                                                                                 this.setValue( data.target.name );
                                                                 },
+                                                                validate : function( data )
+                                                                {
+                                                                        // Popup Window Name cannot contain white space #5278.
+                                                                        var aMatch = this.getValue().match( targetFrameRegex );
+                                                                        if ( aMatch )
+                                                                                alert( editor.lang.link.validatePopupName );
+                                                                        return !aMatch;
+                                                                },
                                                                 commit : function( data )
                                                                 {
                                                                         if ( !data.target )