ICEfaces
  1. ICEfaces
  2. ICE-6284

PanelTab is not correctly resolving URL for icon attribute in portlets

    Details

      Description

      In the portlet version of component showase, the static panel tabset dumps the following to the server log:

      19:07:32,883 WARN [PortalImpl:2948] Current URL /web/guest/images/menu/recent.gif generates exception: null

      This is caused by the icon attribute of the panelTab component. The markup looks like this:


              <ice:panelTab rendered="#{staticTabbedPaneExample.tabbedPane3Visible}"
                            label="#{msgs['page.panelTabSet.tab3']}"
                            icon="images/menu/recent.gif"
                            iconAlignRight="true"
                            title="#{msgs['page.panelTabSet.static.accessibility.tabThreeTitle']}'">

      The resolution of the final URL for the recent.gif image is not valid in the portlet context.

        Activity

        Hide
        Deryk Sinotte added a comment -

        Checked in changes to both the ICEfaces 1.8 trunk and the ICEfaces 2 compat section.

        For the panetTabSet-static.jspx, panetTabSet-static.xhtml file, I set the path of the icon to absolute. This allows the path to be resolved to include the web-app context.

        For the PanelTab, I changed the the getIcon() call so that it runs through the CoreUtils.resolveResourceURL to ensure the resource is properly encoded for the environment that it's running in.

        Show
        Deryk Sinotte added a comment - Checked in changes to both the ICEfaces 1.8 trunk and the ICEfaces 2 compat section. For the panetTabSet-static.jspx, panetTabSet-static.xhtml file, I set the path of the icon to absolute. This allows the path to be resolved to include the web-app context. For the PanelTab, I changed the the getIcon() call so that it runs through the CoreUtils.resolveResourceURL to ensure the resource is properly encoded for the environment that it's running in.

          People

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

            Dates

            • Created:
              Updated:
              Resolved: