ICEfaces
  1. ICEfaces
  2. ICE-9013

ice:panelSeries corrupts order of div's and span's

          Details

          • Type: Bug Bug
          • Status: Closed
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: 3.2
          • Fix Version/s: 3.3
          • Component/s: ICE-Components
          • Labels:
            None
          • Environment:
            Tomcat 7, Chrome, Firefox, IE

            Description

            For full description of the problem please see a forum link above.

            To reproduce use attached source code.

              Activity

              Ascending order - Click to sort in descending order
              Evgheni Sadovoi created issue -
              Evgheni Sadovoi made changes -
              Field Original Value New Value
              Attachment ForumTestApp.zip [ 15473 ]
              Evgheni Sadovoi made changes -
              Attachment ForumTestApp.zip [ 15473 ]
              Evgheni Sadovoi made changes -
              Attachment ForumTestApp.zip [ 15474 ]
              Hide
              Permalink
              Ted Goddard added a comment -

              Removing a call to setCursorParent() appears to resolve this issue. setCursorParent would have the effect of placing each child at the top of the output, resulting in incorrect order.

              +++ compat/components/src/main/java/com/icesoft/faces/component/panelseries/PanelSeriesRenderer.java (working copy)
              @@ -78,8 +78,9 @@
              childs = list.getChildren().iterator();
              while (childs.hasNext()) {
              UIComponent nextChild = (UIComponent) childs.next();
              +System.out.println("PanelSeries encoding " + nextChild);
              if (nextChild.isRendered())

              { - domContext.setCursorParent(root); +// domContext.setCursorParent(root); encodeParentAndChildren(facesContext, nextChild); }

              }

              Show
              Ted Goddard added a comment - Removing a call to setCursorParent() appears to resolve this issue. setCursorParent would have the effect of placing each child at the top of the output, resulting in incorrect order. +++ compat/components/src/main/java/com/icesoft/faces/component/panelseries/PanelSeriesRenderer.java (working copy) @@ -78,8 +78,9 @@ childs = list.getChildren().iterator(); while (childs.hasNext()) { UIComponent nextChild = (UIComponent) childs.next(); +System.out.println("PanelSeries encoding " + nextChild); if (nextChild.isRendered()) { - domContext.setCursorParent(root); +// domContext.setCursorParent(root); encodeParentAndChildren(facesContext, nextChild); } }
              Ken Fyten made changes -
              Fix Version/s 3.3 [ 10370 ]
              Ted Goddard made changes -
              Assignee Ted Goddard [ ted.goddard ]
              Ted Goddard made changes -
              Status Open [ 1 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Ken Fyten made changes -
              Status Resolved [ 5 ] Closed [ 6 ]

                People

                • Assignee:
                  Ted Goddard
                  Reporter:
                  Evgheni Sadovoi
                • Votes:
                  1 Vote for this issue
                  Watchers:
                  3 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: