ICEfaces
  1. ICEfaces
  2. ICE-1267

Portlets: rationalize resource URLs for portlet compatibility

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.5.2
    • Fix Version/s: 1.6DR#5, 1.6
    • Component/s: Framework
    • Labels:
      None
    • Environment:
      Operating System: Mac OS X 10.0
      Platform: Macintosh

      Description

      Resources are frequently referred to by hard-coded paths. These paths are not necessarily valid
      in a portlet context and they should not be hard-coded. For instance:

      src/com/icesoft/faces/renderkit/dom_html_basic/TableRenderer.java:

      spacerImg.setAttribute(HTML.SRC_ATTR, base + "xmlhttp/css/xp/css-images/selection_spacer.gif");

      Initially assigning to Deryk to determine the correct path mechanism. The follow-on task will
      likely lie with the component team to uniformly apply that mechanism to all the components.

        Issue Links

          Activity

          Ted Goddard created issue -
          Icefaces Administrator made changes -
          Field Original Value New Value
          issue.field.bugzillaimportkey 1340 12536
          Ken Fyten made changes -
          Summary rationalize resource URLs for portlet compatibility Portlets: rationalize resource URLs for portlet compatibility
          Ken Fyten made changes -
          Estimated Complexity High
          Assignee Priority P1
          Ken Fyten made changes -
          Assignee Priority P1
          Hide
          Adnan Durrani added a comment -

          Tree component is using the new mechanism to resolving path added by Deryk. -revision 13913

          Show
          Adnan Durrani added a comment - Tree component is using the new mechanism to resolving path added by Deryk. -revision 13913
          Hide
          Adnan Durrani added a comment -

          selectInputDate component is using the new path mechanism -revision 13914

          Show
          Adnan Durrani added a comment - selectInputDate component is using the new path mechanism -revision 13914
          Hide
          Adnan Durrani added a comment -

          Menu component is using the new path mechanism -revision 13916

          Show
          Adnan Durrani added a comment - Menu component is using the new path mechanism -revision 13916
          Hide
          Adnan Durrani added a comment -

          PanelPopup and PanelTabSet components are using the new path mechanism -revision 13917

          Show
          Adnan Durrani added a comment - PanelPopup and PanelTabSet components are using the new path mechanism -revision 13917
          Repository Revision Date User Message
          ICEsoft Public SVN Repository #13927 Fri May 18 17:26:24 MDT 2007 deryk.sinotte ICE-1267 extra logic for resolving absolute paths for portlet resources
          Files Changed
          Commit graph MODIFY /icefaces/trunk/icefaces/core/src/com/icesoft/faces/application/D2DViewHandler.java
          Hide
          Deryk Sinotte added a comment -

          Marking as FIXED.

          We've refactored the code so that resource location resolution works in both portlet and servlet environments using a common API ViewHander.getResourceURL. Then changes were checked in and referred under the general portlet improvement case:

          http://jira.icefaces.org/browse/ICE-1478

          Show
          Deryk Sinotte added a comment - Marking as FIXED. We've refactored the code so that resource location resolution works in both portlet and servlet environments using a common API ViewHander.getResourceURL. Then changes were checked in and referred under the general portlet improvement case: http://jira.icefaces.org/browse/ICE-1478
          Deryk Sinotte made changes -
          Status Open [ 1 ] Resolved [ 5 ]
          Fix Version/s 1.6DR#5 [ 10071 ]
          Fix Version/s 1.6 [ 10031 ]
          Resolution Fixed [ 1 ]
          Ken Fyten made changes -
          Fix Version/s 1.6 [ 10031 ]
          Ken Fyten made changes -
          Link This issue blocks ICE-1809 [ ICE-1809 ]
          Ken Fyten made changes -
          Status Resolved [ 5 ] Closed [ 6 ]
          Assignee Deryk Sinotte [ deryk.sinotte ]

            People

            • Assignee:
              Unassigned
              Reporter:
              Ted Goddard
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: