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

          Carmen Cristurean created issue -
          Carmen Cristurean made changes -
          Field Original Value New Value
          Fix Version/s EE-4.1.0.GA [ 12171 ]
          Ken Fyten made changes -
          Fix Version/s 4.2 [ 12870 ]
          Fix Version/s EE-4.1.0.GA [ 12171 ]
          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.
          Ken Fyten made changes -
          Assignee Arturo Zambrano [ artzambrano ]
          Fix Version/s EE-4.2.0.GA [ 13071 ]
          Fix Version/s 4.2 [ 12870 ]
          Assignee Priority P2 [ 10011 ]
          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.
          Arturo Zambrano made changes -
          Status Open [ 1 ] Resolved [ 5 ]
          Resolution Fixed [ 1 ]
          Arturo Zambrano made changes -
          Link This issue depends on ICE-11017 [ ICE-11017 ]
          Repository Revision Date User Message
          ICEsoft Public SVN Repository #48691 Tue May 03 15:18:43 MDT 2016 art.zambrano ICE-10988 fix to only add our custom plugin if it wasn't already added
          Files Changed
          Commit graph MODIFY /icefaces4/trunk/icefaces/ace/component/resources/icefaces.ace/richtextentry/richtextentry.js
          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,
          Ken Fyten made changes -
          Security Private [ 10001 ]
          Ken Fyten made changes -
          Status Resolved [ 5 ] Closed [ 6 ]

            People

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

              Dates

              • Created:
                Updated:
                Resolved: