ICEfaces
  1. ICEfaces
  2. ICE-8000

PanelPositioned renders a visible xml comment containing a random number

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: EE-1.8.2.GA_P03
    • Fix Version/s: EE-1.8.2.GA_P04
    • Component/s: ICE-Components
    • Labels:
      None
    • Environment:
      Chrome, IE

      Description

      The PanelPositioned component renders a visible xml comment containing a random number. The rendered comment only appears visible on some browsers, such as Chrome and IE, but not others, such as Firefox. The xml comment is being rendered in the following section of PanelPositionedRenderer:

                   if (isChanged(facesContext)) {
                          // Force the re rendering of the entire component. This is due to a strange quick with positioned
                          // panel. When an element is moved in the same list then it container element moves with it
                          // When the update occurs the elements are replaced but because the containers have moved
                          // then the result looks the same. (But a refresh shows otherwise)
                          Node node = domContext.createTextNode(
                                  "<!-- " + (new Random().nextInt(1000)) + "-->");
                          root.appendChild(node);
                      }

        Issue Links

          Activity

          Repository Revision Date User Message
          ICEsoft Public SVN Repository #28650 Mon Apr 09 12:14:32 MDT 2012 art.zambrano ICE-8000 unescaped text node containing XML comment with random number in order to prevent it from being displayed on the browser
          Files Changed
          Commit graph MODIFY /icefaces/trunk/icefaces/component/src/com/icesoft/faces/component/panelpositioned/PanelPositionedRenderer.java

            People

            • Assignee:
              Arturo Zambrano
              Reporter:
              Philip Breau
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: