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

              Ascending order - Click to sort in descending order
              Hide
              Permalink
              yip.ng added a comment -

              Screenshot showing the fix.

              Show
              yip.ng added a comment - Screenshot showing the fix.
              Hide
              Permalink
              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
              Permalink
              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
              Permalink
              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: