Investigated issue where the panelTabSet was used in two different configurations, and in one, an ice:commandLink in the label facet of a non-selected panelTab could be clicked, and have its action method called, and in the other configuration the action method would not be called. The scenario that worked was when using the value property of the panelTabSet, to create a dynamic list of panelTabs, using the panelTabSet's UIData mechanism. The scenario that did not work was using a c:forEach to create a dynamic list of panelTabs, but that used the panelTabSet's static mechanism.

Investigated differences in the generated markup, but didn't find anything inexplicably different. Investigated the javascript POST, and determined that server calls were happening in both cases. Investigated the lifecycle of the commandLink, and determined that it was not receiving phases. Auditing of the panelTabSet code indicated that there were two distinct code paths for passing phases to children, which would affect non-selected tab label facets. Normalised the code on the technique that is correct in all cases.

icefaces 1.8.x trunk
Subversion 22190

icefaces 2 trunk
Subversion 22189

Deryk pointed out that I had accidentally committed the wrong file to icefaces 2, for this fix. Turns out that I had committed a test file, for reproducing the issue, and not even committed the actual code fix. First commit here is to roll back the damage, and the second commit is the actual fix.

icefaces 2 trunk
Subversion 22214
Subversion 22215