Ticket #1344: 1344_3.patch

_whatsnew.html

@@ -146 +146 @@
                         FCKConfig.RemoveAttributes.</li>
                 <li>[<a target="_blank" href="http://dev.fckeditor.net/ticket/1596">#1596</a>] On Safari,
                         dialogs have now rigth-to-left layout when it runs a RTL language, like Arabic.</li>
+                <li>[<a target="_blank" href="http://dev.fckeditor.net/ticket/1344">#1344</a>] Added warning message on
+                        Copy and Cut operation failure on IE due to paste permission settings.</li>
         </ul>
         <p>
                 <a href="_whatsnew_history.html">See previous versions history</a>

editor/_source/commandclasses/fck_othercommands.js

@@ -396 +396 @@
         }
 } ;
 
-// FCKCopyCommand
-var FCKCopyCommand = function()
+// FCKCutCopyCommand
+var FCKCutCopyCommand = function( isCut )
 {
-        this.Name = 'Copy' ;
+        this.Name = isCut ? 'Cut' : 'Copy' ;
 }
 
-FCKCopyCommand.prototype =
+FCKCutCopyCommand.prototype =
 {
         Execute : function()
         {
-                FCK.ExecuteNamedCommand( this.Name ) ;
+                var enabled = false ;
+                
+                if ( FCKBrowserInfo.IsIE )
+                {
+                        // The following seems to be the only reliable way to detect that
+                        // cut/copy is enabled in IE. It will fire the oncut/oncopy event
+                        // only if the security settings enabled the command to execute.
+
+                        var onEvent = function() 
+                        { 
+                                enabled = true ; 
+                        } ;
+
+                        var eventName = 'on' + this.Name.toLowerCase() ;
+                        
+                        FCK.EditorDocument.body.attachEvent( eventName, onEvent ) ;
+                        FCK.ExecuteNamedCommand( this.Name ) ;
+                        FCK.EditorDocument.body.detachEvent( eventName, onEvent ) ;
+                }
+                else
+                {
+                        try                     
+                        {
+                                // Other browsers throw an error if the command is disabled.
+                                FCK.ExecuteNamedCommand( this.Name ) ;
+                                enabled = true ;
+                        }
+                        catch(e){}
+                }
+
+                if ( !enabled )
+                        alert( FCKLang[ 'PasteError' + this.Name ] ) ;
         },
 
         GetState : function()
         {
-                if ( FCK.EditMode != FCK_EDITMODE_WYSIWYG )
-                        return FCK_TRISTATE_DISABLED ;
-                // Strangely, the cut command happens to have the correct states for both Copy and Cut in all browsers.
-                return FCK.GetNamedCommandState( 'Cut' ) ;
+                // Strangely, the Cut command happens to have the correct states for
+                // both Copy and Cut in all browsers.
+                return FCK.EditMode != FCK_EDITMODE_WYSIWYG ?
+                                FCK_TRISTATE_DISABLED :
+                                FCK.GetNamedCommandState( 'Cut' ) ;
         }
 };
 

editor/_source/internals/fck_gecko.js

@@ -301 +301 @@
 FCK.RedirectNamedCommands =
 {
         Print   : true,
-        Paste   : true,
-
-        Cut     : true,
-        Copy    : true
+        Paste   : true
 } ;
 
 // ExecuteNamedCommand overload for Gecko.
@@ -327 +324 @@
                         }
                         catch (e)       { FCKDialog.OpenDialog( 'FCKDialog_Paste', FCKLang.Paste, 'dialog/fck_paste.html', 400, 330, 'Security' ) ; }
                         break ;
-                case 'Cut' :
-                        try                     { FCK.ExecuteNamedCommand( 'Cut', null, true ) ; }
-                        catch (e)       { alert(FCKLang.PasteErrorCut) ; }
-                        break ;
-                case 'Copy' :
-                        try                     { FCK.ExecuteNamedCommand( 'Copy', null, true ) ; }
-                        catch (e)       { alert(FCKLang.PasteErrorCopy) ; }
-                        break ;
                 default :
                         FCK.ExecuteNamedCommand( commandName, commandParameter ) ;
         }

editor/_source/internals/fckcommands.js

@@ -126 +126 @@
 
                 case 'Undo'     : oCommand = new FCKUndoCommand() ; break ;
                 case 'Redo'     : oCommand = new FCKRedoCommand() ; break ;
-                case 'Copy'     : oCommand = new FCKCopyCommand() ; break ;
+                case 'Copy'     : oCommand = new FCKCutCopyCommand( false ) ; break ;
+                case 'Cut'      : oCommand = new FCKCutCopyCommand( true ) ; break ;
 
                 case 'SelectAll'                        : oCommand = new FCKSelectAllCommand() ; break ;
                 case 'InsertOrderedList'        : oCommand = new FCKListCommand( 'insertorderedlist', 'ol' ) ; break ;