[richfaces-issues] [JBoss JIRA] Created: (RF-7290) A bug is identified in tiny_mce_src.js packaged within Richfaces. Richfaces future release needs to update tiny mce to version 3.2.4.1 where the bug has been fixed.

[Gary Hu (JIRA)]

A bug is identified in tiny_mce_src.js packaged within Richfaces. Richfaces future release needs to update tiny mce to version 3.2.4.1 where the bug has been fixed.
--------------------------------------------------------------------------------------------------------------------------------------------------------------------

                 Key: RF-7290
                 URL: https://jira.jboss.org/jira/browse/RF-7290
             Project: RichFaces
          Issue Type: Thirdparty Change
    Affects Versions: 3.3.1, 3.3.0, 3.2.2
            Reporter: Gary Hu
             Fix For: 3.3.2.CR1


A bug is identified in tiny_mce_src.js packaged within Richfaces. The version of tiny mce coming with Richfaces is 3.2. The following is the problematic code in tiny_mce_src.js:

function fakeUnload() {
   var d = document;
   debugger;
   if (d.readyState == "interactive") {
        function stop() {
            d.detachEvent("onstop", stop);
            unload();
            d = null;
         }
       *** d.attachEvent("onstop", stop);
       window.setTimeout(function () {
       *** d.detachEvent("onstop", stop);
       }, 0);
   }
}

The lines with "***" are problematic. The code needs to check if "d" is null for those lines. 

The latest Tiny mce release 3.2.4.1 has fixed the above issue.

The following is the code in 3.2.4.1:

function fakeUnload() {
         var d = document;
         // Is there things still loading, then do some magic
        if (d.readyState == 'interactive') {
               function stop() {
                   // Prevent memory leak
                  d.detachEvent('onstop', stop);
                  // Call unload handler
                 if (unload)
                     unload();
                 d = 0;
             };
           // Fire unload when the currently loading page is stopped
          if (d)
             d.attachEvent('onstop', stop);
          // Remove onstop listener after a while to prevent the unload function
          // to execute if the user presses cancel in an onbeforeunload
          // confirm dialog and then presses the browser stop button
              window.setTimeout(function() {
                  if (d)
                     d.detachEvent('onstop', stop);
               }, 0);
         }
       };

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: https://jira.jboss.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira