ICEfaces
  1. ICEfaces
  2. ICE-8715

Cached resources for ace:richTextEntry can lead to problems with portlets

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 3.2
    • Fix Version/s: EE-3.2.0.GA, 3.3
    • Component/s: ACE-Components
    • Labels:
      None
    • Environment:
      Portlets, ACE comps.
    • Assignee Priority:
      P1
    • Affects:
      Compatibility/Configuration

      Description

      In doing work for ICE-8710, I noticed that the strategy of caching resource URLs during the initial creation of the RichTextEntryResourceHandler can lead to problems when running in portlets. Specifically, when the RichTextEntryResourceHandler is first created, it will try to determine and use the URL of the first portal page that loads with an ace:richTextEntry component. For example, a Liferay URL might look like:

           http://localhost:8080/web/guest/page1....

       It will then try to create mappings for all the resources used by the CKEditor using that URL. If you add another ace:richTextEntry instance to another page:
           http://localhost:8080/web/guest/page2....
       it will attempt to load the resources from the original page. This may work or it may fail depending on the current state of the application. However, we should not be relying on this strategy anyway for portlets as it can either use the incorrect URL or it may fail completely in environments (e.g. WebSphere) were the URLs might be more opaque.

        Activity

        Repository Revision Date User Message
        ICEsoft Public SVN Repository #37242 Fri Jul 26 10:34:51 MDT 2013 art.zambrano ICE-9447 applied fix ICE-8715, updated readme.txt
        Files Changed
        Commit graph MODIFY /icefaces3/trunk/icefaces/compat/components/src/main/resources/META-INF/resources/inputrichtext/readme.txt
        Commit graph MODIFY /icefaces3/trunk/icefaces/compat/components/src/main/resources/META-INF/resources/inputrichtext/ckeditor/ckeditor.mapping.js
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #33254 Mon Jan 28 16:13:46 MST 2013 art.zambrano ICE-8715 removed parameter containing timestamp in order to be able to map resources correctly in the cases where this parameter is included
        Files Changed
        Commit graph MODIFY /icefaces3/trunk/icefaces/ace/component/resources/icefaces.ace/richtextentry/ckeditor/ckeditor.mapping.js
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #33191 Wed Jan 23 18:19:21 MST 2013 art.zambrano ICE-8715 fixed issue with ckeditor.mapping.js resource URL only working on the machine that loaded it first
        Files Changed
        Commit graph MODIFY /icefaces3/trunk/icefaces/ace/component/src/org/icefaces/ace/component/richtextentry/RichTextEntryResourceHandler.java
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #33045 Fri Jan 11 16:50:18 MST 2013 art.zambrano ICE-8715 removed calls to ace:richTextEntry ant script
        Files Changed
        Commit graph MODIFY /icefaces3/trunk/icefaces/ace/component/build.xml
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #33044 Fri Jan 11 16:38:19 MST 2013 art.zambrano ICE-8715 added tool to modify CKEditor CSS resources to use EL resource expressions instead of relative paths in url() expressions and to create a Javascript file with a mapping from the canonical resource names to EL resource expressions
        Files Changed
        Commit graph ADD /icefaces3/trunk/icefaces/ace/tools/ckeditorurlmapper/src
        Commit graph ADD /icefaces3/trunk/icefaces/ace/tools/ckeditorurlmapper
        Commit graph ADD /icefaces3/trunk/icefaces/ace/tools/ckeditorurlmapper/src/org/icefaces/ace
        Commit graph ADD /icefaces3/trunk/icefaces/ace/tools/ckeditorurlmapper/build.xml
        Commit graph DEL /icefaces3/trunk/icefaces/ace/component/src/org/icefaces/ace/component/richtextentry/build.xml
        Commit graph ADD /icefaces3/trunk/icefaces/ace/tools/ckeditorurlmapper/src/org
        Commit graph ADD /icefaces3/trunk/icefaces/ace/tools/ckeditorurlmapper/src/org/icefaces/ace/util
        Commit graph ADD /icefaces3/trunk/icefaces/ace/tools/ckeditorurlmapper/src/org/icefaces/ace/util/ckeditorurlmapper/CKEditorUrlMapper.java
        Commit graph ADD /icefaces3/trunk/icefaces/ace/tools/ckeditorurlmapper/readme.txt
        Commit graph ADD /icefaces3/trunk/icefaces/ace/tools/ckeditorurlmapper/src/org/icefaces
        Commit graph ADD /icefaces3/trunk/icefaces/ace/tools/ckeditorurlmapper/src/org/icefaces/ace/util/ckeditorurlmapper
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #32921 Fri Dec 21 17:05:04 MST 2012 art.zambrano ICE-8715 modified RichTextResourceHandler to only take the existing ckeditor resource mapping script and serve it by means of a special Resource implementation that processes the contents and converts EL resource expressions to correct JSF URLs in portlets and non-portlets environments; also added this pre-processed ckeditor resource mapping script file and pre-processed CSS files with EL resource expressions instead of simple relative paths to images
        Files Changed
        Commit graph MODIFY /icefaces3/trunk/icefaces/ace/component/resources/icefaces.ace/richtextentry/ckeditor/plugins/wsc/dialogs/wsc.css
        Commit graph MODIFY /icefaces3/trunk/icefaces/ace/component/resources/icefaces.ace/richtextentry/ckeditor/skins/v2/templates.css
        Commit graph ADD /icefaces3/trunk/icefaces/ace/component/resources/icefaces.ace/richtextentry/ckeditor/ckeditor.mapping.js
        Commit graph MODIFY /icefaces3/trunk/icefaces/ace/component/resources/icefaces.ace/richtextentry/ckeditor/contents.css
        Commit graph MODIFY /icefaces3/trunk/icefaces/ace/component/resources/icefaces.ace/richtextentry/ckeditor/plugins/scayt/dialogs/toolbar.css
        Commit graph MODIFY /icefaces3/trunk/icefaces/ace/component/src/org/icefaces/ace/component/richtextentry/RichTextEntryResourceHandler.java
        Commit graph MODIFY /icefaces3/trunk/icefaces/ace/component/resources/icefaces.ace/richtextentry/ckeditor/skins/kama/editor.css
        Commit graph MODIFY /icefaces3/trunk/icefaces/ace/component/resources/icefaces.ace/richtextentry/ckeditor/skins/office2003/templates.css
        Commit graph MODIFY /icefaces3/trunk/icefaces/ace/component/resources/icefaces.ace/richtextentry/ckeditor/skins/office2003/dialog.css
        Commit graph MODIFY /icefaces3/trunk/icefaces/ace/component/resources/icefaces.ace/richtextentry/ckeditor/skins/v2/dialog.css
        Commit graph MODIFY /icefaces3/trunk/icefaces/ace/component/resources/icefaces.ace/richtextentry/ckeditor/plugins/uicolor/yui/assets/yui.css
        Commit graph MODIFY /icefaces3/trunk/icefaces/ace/component/resources/icefaces.ace/richtextentry/ckeditor/skins/kama/templates.css
        Commit graph MODIFY /icefaces3/trunk/icefaces/ace/component/resources/icefaces.ace/richtextentry/ckeditor/skins/kama/dialog.css
        Commit graph MODIFY /icefaces3/trunk/icefaces/ace/component/resources/icefaces.ace/richtextentry/ckeditor/skins/v2/editor.css
        Commit graph MODIFY /icefaces3/trunk/icefaces/ace/component/resources/icefaces.ace/richtextentry/readme.txt
        Commit graph MODIFY /icefaces3/trunk/icefaces/ace/component/resources/icefaces.ace/richtextentry/ckeditor/skins/office2003/editor.css

          People

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

            Dates

            • Created:
              Updated:
              Resolved: