Attached test case that shows the issue with f:setPropertyActionListener. The f:attribute implementation is commented out.

Steps to reproduce:

• Load welcomeICEfaces.jsf
• Click on the view link to show the row info.
• In the first columns filter input enter in a 'p'.
• The data is dfiltered to two rows. Clicking on the view links will show the row info for the previous data set.

My suspicion is that this is due to the data model being generated (via getDataModel()) with a list that is not the filtered list(via getValue()), even though we have a filtered list in our state we should be exposing.

This is usually due to our state saving breaking because the table clientId contains iterative indexes. When getFilteredData() is called is called in getValue(), null is returned (because the state doesn't exist for the incorrect clientId) and the UIData implementation of getValue() supplies the backing for the data model.

Up until now, our solution for this issue has been putting off fixing it at large and simply avoiding it and not calling methods that access the component state (as held by the meta class generated properties) during an iterative context. However this appears to be a framework initiated instance where we need that state during iteration, and I'm not sure how we're to work around it.

The implementation of f:setPropertyActionListener is very simple: for this page we have

<ace:dataTable value="#

" var="entity">

and setPropertyActionListener evaluates #

at

at com.sun.faces.facelets.tag.jsf.core.SetPropertyActionListenerHandler$SetPropertyListener.processAction(SetPropertyActionListenerHandler.java:209)
at javax.faces.event.ActionEvent.processListener(ActionEvent.java:88)
at javax.faces.component.UIComponentBase.broadcast(UIComponentBase.java:769)
at javax.faces.component.UICommand.broadcast(UICommand.java:300)
at javax.faces.component.UIData.broadcast(UIData.java:1093)
at org.icefaces.ace.component.datatable.DataTable.broadcast(DataTable.java:222)
at javax.faces.component.UIViewRoot.broadcastEvents(UIViewRoot.java:794)
at javax.faces.component.UIViewRoot.processApplication(UIViewRoot.java:1259)
at com.sun.faces.lifecycle.InvokeApplicationPhase.execute(InvokeApplicationPhase.java:81)

The relevant code in UIData is:

FacesContext context = FacesContext.getCurrentInstance();
// Set up the correct context and fire our wrapped event
WrapperEvent revent = (WrapperEvent) event;
if (isNestedWithinUIData())

int oldRowIndex = getRowIndex();
setRowIndex(revent.getRowIndex());
FacesEvent rowEvent = revent.getFacesEvent();
UIComponent source = rowEvent.getComponent();
UIComponent compositeParent = null;
try {
if (!UIComponent.isCompositeComponent(source))

if (compositeParent != null)

source.pushComponentToEL(context, null);
source.broadcast(rowEvent);

It looks like UIData calls getRowIndex() – does our DataTable return the rowIndex into the filtered data or into the unfiltered data? It also looks like we have an opportunity during broadcast() to filter the data rather than calling

setDataModel(null);

Judging from:

int oldRowIndex = getRowIndex();
setRowIndex(revent.getRowIndex());

The data table probably has an iterative ID because it has a pre-existing row index that needs saving. When an iterative ID is set, the generator properties don't work correctly, so the unfiltered dataset is being referenced.

I suppose the only fixes are to override the UIData broadcast behaviour or fix generator properties.

Revision #29269
Committed by nils.lundquist
A minute ago
ICE-8098 - Overrode UIData.broadcast() in ace:datatable to correct clientId before accessing filtered state.