ICEfaces-EE
  1. ICEfaces-EE
  2. IPCK-58

<ice-cc:dragAndDropTreeTable> tag incorrectly references xmlhttp images using a relative path

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.8.2.GA
    • Fix Version/s: 1.8.2.GA_P01
    • Component/s: Facelet Components
    • Labels:
      None
    • Environment:
      RHEL 5.0, Glassfish V2.1.1
    • Workaround Description:
      Add additional servlet-mapping's for each additional path from which the tag is referenced.

      Description

      The <ice-cc:dragAndDropTreeTable> tag includes image references using a relative path. This is difference from the rest of the icefaces components that use a "absolute relative path", meaning one that starts with a "/". Since the xmlhttp servlet needs to be mapped, it is essential that links to these images start with a "/", when the referencing page is not in the root folder.

        Issue Links

          Activity

          Hide
          Brad Kroeger added a comment -

          Revision #22111 - Change references to ICEfaces resources in xmlhttp directory to be preceded with ./ rather than / or nothing. This is an ICEfaces best practice.

          Talked with the component team about this, they said the relative/absolute method of using just the / is less predictable in it's behavior.

          Show
          Brad Kroeger added a comment - Revision #22111 - Change references to ICEfaces resources in xmlhttp directory to be preceded with ./ rather than / or nothing. This is an ICEfaces best practice. Talked with the component team about this, they said the relative/absolute method of using just the / is less predictable in it's behavior.
          Hide
          Brad Kroeger added a comment -

          Upon further investigation the issue here is that some of the images are represented with <img> tags and some with <ice:graphicImage> tags. With ICEfaces components, it is correct to use the / when the referencing page is not in the root folder. However, the <img> tags are treated differently and their bindings will be modified so the images can be accessed from outside the root folder.

          Show
          Brad Kroeger added a comment - Upon further investigation the issue here is that some of the images are represented with <img> tags and some with <ice:graphicImage> tags. With ICEfaces components, it is correct to use the / when the referencing page is not in the root folder. However, the <img> tags are treated differently and their bindings will be modified so the images can be accessed from outside the root folder.
          Hide
          Brad Kroeger added a comment -

          Revision #22945 'Absolute Relative' references standard in ICEfaces component images. This means you precede image urls with a / ie: /xmlhttp/css/rime/css-images/tree_line_vertical.gif. This results in our servlet adding the context appropriately and images being accessible whether the source page is in the root of the web application or another folder nested in the root.

          HTML <img> tags have been replaced with ice:graphicImage tags for consistency. The <img> tag urls were not processed by our servlet to add the appropriate context to the absolute relative url.

          Show
          Brad Kroeger added a comment - Revision #22945 'Absolute Relative' references standard in ICEfaces component images. This means you precede image urls with a / ie: /xmlhttp/css/rime/css-images/tree_line_vertical.gif. This results in our servlet adding the context appropriately and images being accessible whether the source page is in the root of the web application or another folder nested in the root. HTML <img> tags have been replaced with ice:graphicImage tags for consistency. The <img> tag urls were not processed by our servlet to add the appropriate context to the absolute relative url.

            People

            • Assignee:
              Brad Kroeger
              Reporter:
              Brian Leathem
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: