ICEfaces
  1. ICEfaces
  2. ICE-3154

Clean up improper 'encodeChildren' implementations in a few components

          Details

          • Type: Bug Bug
          • Status: Closed
          • Priority: Minor Minor
          • Resolution: Fixed
          • Affects Version/s: 1.7
          • Fix Version/s: 1.7.1
          • Component/s: ICE-Components
          • Labels:
            None
          • Environment:
            All

            Description

            Because of ICE-3147, I did a little impromptu audit of encodeChildren() methods in our components. The following classes use non-standard implementations of encodeChildren(-), which may cause problems:

            com.icesoft.faces.component.gmap.GMapLatLngs
            com.icesoft.faces.component.gmap.GMapMarker
            com.icesoft.faces.component.gmap.GMapRenderer
            com.icesoft.faces.component.menubar.MenuItemRenderer
               Lines 658, 704 :: Link rendering. The way we hard-code the links, it
            should be ok
            com.icesoft.faces.component.menubar.MenuItemRendererBase
               Omits check for rendered that MenuItemsRenderer does
            com.icesoft.faces.component.panellayout.PanelLayoutRenderer

              Activity

              Ascending order - Click to sort in descending order
              Mark Collette created issue -
              Ken Fyten made changes -
              Field Original Value New Value
              Summary Improper encodeChildren Clean up improper 'encodeChildren' implementations in a few components
              Fix Version/s 1.7.1 [ 10122 ]
              Security Private [ 10001 ]
              Priority Major [ 3 ] Minor [ 4 ]
              Ken Fyten made changes -
              Affects Version/s 1.7 [ 10080 ]
              Affects Version/s 1.7.1 [ 10122 ]
              Ken Fyten made changes -
              Status Open [ 1 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Ken Fyten made changes -
              Resolution Fixed [ 1 ]
              Status Resolved [ 5 ] Reopened [ 4 ]
              Assignee Priority P1
              Assignee Yip Ng [ yip.ng ]
              Ken Fyten made changes -
              Status Reopened [ 4 ] Resolved [ 5 ]
              Assignee Priority P1
              Resolution Fixed [ 1 ]
              Assignee Yip Ng [ yip.ng ]
              Ken Fyten made changes -
              Status Resolved [ 5 ] Closed [ 6 ]

                People

                • Assignee:
                  Unassigned
                  Reporter:
                  Mark Collette
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  0 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: