[richfaces-issues] [JBoss JIRA] Resolved: (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.5 (was 3.2.4.1) where the bug has been fixed.

[Pavel Yaschenko (JIRA)]

     [ https://jira.jboss.org/jira/browse/RF-7290?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Pavel Yaschenko resolved RF-7290.
---------------------------------

    Resolution: Done


> A bug is identified in tiny_mce_src.js packaged within Richfaces. Richfaces future release needs to update tiny mce to version 3.2.5 (was 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
>          Components: component-input
>    Affects Versions: 3.2.2, 3.3.0, 3.3.1
>            Reporter: Gary Hu
>            Assignee: Pavel Yaschenko
>            Priority: Blocker
>             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