ICEfaces
  1. ICEfaces
  2. ICE-4983

PanelPositionedEvent's phase not changed by setEventPhase

          Details

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

            Description

            Neil Griffin ran into an issue where this wouldn't change the event to being broadcast in INVOKE_APPLICATION. It was still being broadcast in UPDATE_MODEL.

            <ice:setEventPhase events="com.icesoft.faces.component.panelpositioned.PanelPositionedEvent" phase="INVOKE_APPLICATION">
                <ice:panelPositioned constraint="vertical" listener="#{wikiBackingBean.lineRepositioned}" value="#{wikiPageModelBean.creoleLines}" var="creoleLine">
                    ...
                </ice:panelPositioned>
            </ice:setEventPhase >

              Activity

              Ascending order - Click to sort in descending order
              Hide
              Permalink
              Mark Collette added a comment -

              I looked through SetEventPhase, PanelPositioned, PanelPositionedRenderer, and nothing jumped out as being wrong that would explain this. Will have to develop a test case, and investigate further.

              Show
              Mark Collette added a comment - I looked through SetEventPhase, PanelPositioned, PanelPositionedRenderer, and nothing jumped out as being wrong that would explain this. Will have to develop a test case, and investigate further.
              Hide
              Permalink
              Michel Z added a comment -

              I have observed the same problem for both panelSeries and panelPositioned and the ValueChangeEvent.
              In both cases the broadcasted event's phaseId is "ANY 0".

              Workaround seems to be: adding another setEventPhase element inside the panelSeries/panelPositioned element.

              Show
              Michel Z added a comment - I have observed the same problem for both panelSeries and panelPositioned and the ValueChangeEvent. In both cases the broadcasted event's phaseId is "ANY 0". Workaround seems to be: adding another setEventPhase element inside the panelSeries/panelPositioned element.
              Hide
              Permalink
              Mark Collette added a comment -

              UIData containers wrap events in index aware events, which should copy the desired broadcast phase info. Perhaps this one does not. If that's it, quick simple one line fix.

              Show
              Mark Collette added a comment - UIData containers wrap events in index aware events, which should copy the desired broadcast phase info. Perhaps this one does not. If that's it, quick simple one line fix.
              Hide
              Permalink
              Adnan Durrani added a comment -

              Confirmed with dataTable. Should be able to work with nested UIData as well.

              Command: Commit
              Modified: D:\work\development\head\svn\ossrepo\icefaces\trunk\icefaces\component\src\com\icesoft\faces\component\seteventphase\SetEventPhase.java
              Sending content: D:\work\development\head\svn\ossrepo\icefaces\trunk\icefaces\component\src\com\icesoft\faces\component\seteventphase\SetEventPhase.java
              Completed: At revision: 25805

              Show
              Adnan Durrani added a comment - Confirmed with dataTable. Should be able to work with nested UIData as well. Command: Commit Modified: D:\work\development\head\svn\ossrepo\icefaces\trunk\icefaces\component\src\com\icesoft\faces\component\seteventphase\SetEventPhase.java Sending content: D:\work\development\head\svn\ossrepo\icefaces\trunk\icefaces\component\src\com\icesoft\faces\component\seteventphase\SetEventPhase.java Completed: At revision: 25805

                People

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

                  Dates

                  • Created:
                    Updated:
                    Resolved: