Using on a List doesn't update model values

Question

I've the below data table:

<h:dataTable var="row" value="#{myBean.listOfStrings}">
    <h:column> 
         <h:inputText value="#{row}" />
    </h:column>
</h:dataTable>

Which is tied to a List<String>:

private List<String> listOfStrings = new ArrayList<String>();

public List<String> getListOfStrings() {
    return listOfStrings;
}

public void setListOfStrings(List<String> listOfStrings) {
    this.listOfStrings = listOfStrings;
}

When I enter a value in the field and save the form it is not passing the value to the field in the list, it is setting null, what am I doing wrong here?

We need full bean class declaration. My wild guess: check the scope of the bean. — Stefano Sanfilippo, Jan 20 '14 at 14:15

score 5 · Accepted Answer · edited Jun 20 '20 at 09:12

The String class is immutable. It doesn't have a setter for the instance value. The getter is in this construct basically the Object#toString() method as implicitly called by EL, which coincidentally returns the string value itself.

You need to set the changed value as a new list item instead. You can do this via the brace notation on the list whereby you pass the list index: #{myBean.listOfStrings[index]}.

So, this should do, making use of UIData#getRowIndex() as list index:

<h:dataTable binding="#{table}" value="#{myBean.listOfStrings}" var="row">
    <h:column> 
         <h:inputText value="#{myBean.listOfStrings[table.rowIndex]}" />
    </h:column>
</h:dataTable>

(note: the value expression of binding is as-is! don't bind it to a bean property)

Using on a List doesn't update model values

1 Answers1

See also:

Linked

Related