ICEfaces
  1. ICEfaces
  2. ICE-10988

showcase ace:richTextEntry 'coalesced' error

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: EE-4.1.0.GA
    • Fix Version/s: EE-4.2.0.GA
    • Component/s: ACE-Components, Sample Apps
    • Labels:
      None
    • Environment:
      ICEfaces4 trunk r.48527
      Browsers: all
      Server: Tomcat7.0.42
    • Assignee Priority:
      P2

      Description

      showcase > richTextEntry (all browsers including MS Edge, IF4 trunk r.48527)
      A browser console error randomly occurred when saving text in the richTextEntry, or when navigating between the richTextEntry demo and a maskedEntry demo. Cannot be reproduced with the showcase public demo.

      SCRIPT5022: [CKEDITOR.resourceManager.add] The resource name "aceSave" is already registered.
      coalesced.js.jsf (3044,77)

        Issue Links

          Activity

          Hide
          Carmen Cristurean added a comment - - edited

          EE4-trunk r45005/48674/ MS Edge:
          showcase > RichTextEntry: on MS Edge/Windows10 test PC, a browser console error occurs when clicking the icon to save text in the richTextEntry component, and the text is not saved.
          SCRIPT5022: [CKEDITOR.resourceManager.add] The resource name "aceSave" is already registered.
          eval code (17) (218,77)
          This error was seen only on MS Edge, when testing the 'Save' function first time after server restart, in the cache-cleared browser.

          This could not be reproduced with ICEfaces 4.1.1 release libraries.

          Show
          Carmen Cristurean added a comment - - edited EE4-trunk r45005/48674/ MS Edge: showcase > RichTextEntry: on MS Edge/Windows10 test PC, a browser console error occurs when clicking the icon to save text in the richTextEntry component, and the text is not saved. SCRIPT5022: [CKEDITOR.resourceManager.add] The resource name "aceSave" is already registered. eval code (17) (218,77) This error was seen only on MS Edge, when testing the 'Save' function first time after server restart, in the cache-cleared browser. This could not be reproduced with ICEfaces 4.1.1 release libraries.
          Hide
          Arturo Zambrano added a comment -

          I found a fix in our code to avoid that specific error. However, errors keep happening at different parts of the CKEditor code, specifically, when trying to destroy the previous editor instance. I've tried to fix some of those errors in the CKEditor code, but more errors keep occurring at other points in the code. It seems like MS Edge is not very compatible with our current CKEditor version (4.3.3). Newer CKEditor versions have resolved various compatibility issues with MS Edge (http://ckeditor.com/blog/CKEditor-4.5.2-Released), so it seems like the only solution would be to update our CKEditor code to the newest version.

          Show
          Arturo Zambrano added a comment - I found a fix in our code to avoid that specific error. However, errors keep happening at different parts of the CKEditor code, specifically, when trying to destroy the previous editor instance. I've tried to fix some of those errors in the CKEditor code, but more errors keep occurring at other points in the code. It seems like MS Edge is not very compatible with our current CKEditor version (4.3.3). Newer CKEditor versions have resolved various compatibility issues with MS Edge ( http://ckeditor.com/blog/CKEditor-4.5.2-Released ), so it seems like the only solution would be to update our CKEditor code to the newest version.
          Hide
          Arturo Zambrano added a comment - - edited

          r48691: fix to only add our custom plugin if it wasn't already added

          This fix and the CKEditor update (ICE-11017) solve this problem.

          Show
          Arturo Zambrano added a comment - - edited r48691: fix to only add our custom plugin if it wasn't already added This fix and the CKEditor update ( ICE-11017 ) solve this problem.
          Hide
          Liana Munroe added a comment -

          Verified ICEfaces EE-4.1.0, Tomcat 8, Chrome 50, FF 43, IE 11, 10, 9, 8,

          Show
          Liana Munroe added a comment - Verified ICEfaces EE-4.1.0, Tomcat 8, Chrome 50, FF 43, IE 11, 10, 9, 8,

            People

            • Assignee:
              Arturo Zambrano
              Reporter:
              Carmen Cristurean
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: