ICEfaces
  1. ICEfaces
  2. ICE-2864

iframe fix for selectInputText has extra slash in the iframe URL

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.6.2, 1.7Beta1
    • Fix Version/s: 1.6.3, 1.7RC1, 1.7
    • Component/s: ICE-Components
    • Labels:
      None
    • Environment:
      IE 6

      Description

      From Ryan Dawson:

      ICE-2065 had been marked as closed but the fix arrived at was not adequate. selectInputText is still causing the secure and nonsecure items warning in IE6. This is because it uses an iframe with a src attribute set to an invalid url (it contains a double forward-slash - '//').

      The fix is a small change to line 4931 of ice-extras.js.
      "src=\""+configuration.connection.context+"/xmlhttp/blank\"
      Should be:
      "src=\""+configuration.connection.context+"xmlhttp/blank\"

        Activity

        Hide
        yip.ng added a comment -

        Screenshot showing the fix.

        Show
        yip.ng added a comment - Screenshot showing the fix.
        Hide
        yip.ng added a comment -

        Only one instance in 1.6. Trunk version doesn't use configuration.connection.context and uses about:blank as the URL.

        Show
        yip.ng added a comment - Only one instance in 1.6. Trunk version doesn't use configuration.connection.context and uses about:blank as the URL.
        Hide
        yip.ng added a comment -

        about:blank doesn't seem to work, but the configuration.connection.context variable is gone from trunk version. Need to figure out how to get it back, or else figure out where in the renderer to resolve the URL on the server side and pass it to the client side.

        Show
        yip.ng added a comment - about:blank doesn't seem to work, but the configuration.connection.context variable is gone from trunk version. Need to figure out how to get it back, or else figure out where in the renderer to resolve the URL on the server side and pass it to the client side.
        Hide
        yip.ng added a comment -

        Screenshot showing the fix in the trunk version.

        Show
        yip.ng added a comment - Screenshot showing the fix in the trunk version.

          People

          • Assignee:
            Unassigned
            Reporter:
            yip.ng
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: