The strategy for using the StateHelper had a slight wrinkle. If a row level component had a default value, the default value is set on the component during the time of execution when the defaults are being defined. After this default setting phase has passed, the framework 'marks' the component and any subsequent puts on the StateHelper are stored in the deltaMap. The problem with looking for our map and instantiating it if missing and adding it to the StateHelper during initialization is that it wont get persisted by state saving.

The solution was to create a map for default values and adding it to the component at default setup time, and then creating a separate instance of the values map and adding it to the stateHelper during the lifecycle if necessary for the row level values so it winds up in the delta map and gets state saved.

One last wrinkle is that JSF moves the value map into the default map after a state saving cycle in the restoreView phase. This means that to continually persist state row values, the value map needs to be re-added to the stateHelper each time a row value changes otherwise it wont be persisted even though we're changing the contents of the map.

I have changes to the generator code checked in. Testing should determine if there are any regressions.

I think that the default should be for properties to do this. Either they will always do this, or maybe we could optionally to do it the old/basic way. It would be nice to know the impact on partial state saving, doing it the new way, versus the pre-existing way, to see if we need make the old way optional, or just discard it.

My recommendation is that we use a combination of settable properties and the Map<String clientId, Object value>. This all has to work with JSF 2's StateHelper class, which our properties use for storing their state in. The implementation of which is in: mojarra2\jsf-api\src\main\java\javax\faces\component\ComponentStateHelper.java

In the setter method, we could see if there's a ValueExpression, and if so use it. Otherwise we fall through to using the StateHelper. If we then use the PropertyKeys field as the key, we would get a Map<String clientId, Object value>. If that doesn't exist, create it and put it back into the StateHelper. Then put the value into the map using the clientId as the key.

In the getter method, first try seeing if the StateHelper has the Map, using the PropertyKeys field as the key, if so, see if there's a value in the Map, using the clientId as the key. If not, then fall back to seeing if there's a ValueExpression, and evaluating it. If not, return the default value.