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

              depends on

              Improvement - An improvement or enhancement to an existing feature or task. ICE-11017 Update CKEditor to version 4.5.8 to support MS Edge Browser

              • Major - Major loss of function.
              • Closed

                Activity

                Ascending order - Click to sort in descending order
                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
                Permalink
                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
                Permalink
                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
                Permalink
                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
                Permalink
                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: