Seam has problem with component-binding unless the bean is session-scoped. (not practical). So to use value binding, the selectedIndex has to have value binding available to selectedIndex property.

would be nice to have the option to use valueBinding for this property for ease of use with seam (as well as MVC)

Mark's adhoc fixes worked great and simplified the Bean & xhtml page immensely when he modified the getters and setters of com.icesoft.faces.component.paneltabset as follows:-

/**

• @param selectedIndex
  */
  void setSelectedIndex(Integer selectedIndex) { System.out.println("PanelTabSet.setSelectedIndex(I="+selectedIndex+")"); if(selectedIndex == null) _selectedIndex = selectedIndex; else setSelectedIndex(selectedIndex.intValue()); }

/**

• @param selectedIndex
  */
  public void setSelectedIndex(int selectedIndex) {
  System.out.println("PanelTabSet.setSelectedIndex(i="selectedIndex")");
  ValueBinding vb = getValueBinding("selectedIndex");
  boolean worked = false;
  if(vb != null)
  Unknown macro: { try { vb.setValue(getFacesContext(), new Integer(selectedIndex)); System.out.println("PanelTabSet.setSelectedIndex() vb.setValue"); worked = true; } catch(Exception e) { worked = false; System.out.println("Problem: " + e); } }

  if(!worked)

  { _selectedIndex = new Integer(selectedIndex); System.out.println("PanelTabSet.setSelectedIndex() field set"); }

  }

/**

• @return the value of selectedIndex
  */
  public int getSelectedIndex()
  Unknown macro: { if (_selectedIndex != null) { System.out.println("PanelTabSet.getSelectedIndex() field get"); return _selectedIndex.intValue(); } ValueBinding vb = getValueBinding("selectedIndex"); System.out.println("PanelTabSet.getSelectedIndex() vb}

I moved the behaviour of the settable property out into its own class, called UpdatableProperty, which encalsulates the behaviour of getting the value locally and through a ValueBinding, as well as setting the value locally, and setting it on the ValueBinding in the UPDATE_MODEL phase.

For ICEfaces 1.7 DR3 I've updated the component-showcase sample to use the simplified method without any listener glue code. We've left the ICEfaces 1.6.2 component-showcase alone, although you could do the exact same technique there as well. In fact, Judy is updating the Seam component-showcase application, that relies on ICEfaces 1.6.2, to use this technique, so we won't need any component bindings.

ICE-2124 : PanelTabSet doesn't sync bound selectedIndex on tabChange events
Subversion 15045
icefaces\component\src\com\icesoft\faces\utils\MessageUtils.java
icefaces\component\src\com\icesoft\faces\utils\UpdatableProperty.java
icefaces\component\src\com\icesoft\faces\component\paneltabset\PanelTabSet.java
icefaces\component\src\com\icesoft\faces\component\paneltabset\PanelTabSetRenderer.java
icefaces\component\src\com\icesoft\faces\resources\messages.properties
icefaces\component\src\com\icesoft\faces\resources\messages_fr.properties

ICE-2124 : PanelTabSet doesn't sync bound selectedIndex on tabChange events (update component-showcase)
Subversion 15046
icefaces\samples\component-showcase\web\inc\layoutPanels\panelTabSet.jspx
icefaces\samples\component-showcase\src\com\icesoft\icefaces\samples\showcase\layoutPanels\tabSetPanel\StaticTabSetBean.java
icefaces\samples\component-showcase\src\com\icesoft\icefaces\samples\showcase\layoutPanels\tabSetPanel\DynamicTabSetBean.java

Backport to 1.6.2 : ICE-2124
svn merge -r 15044:15045 http://server.ice:8888/svn/ossrepo/icefaces/trunk/icefaces
icefaces\component\src\com\icesoft\faces\utils\MessageUtils.java
icefaces\component\src\com\icesoft\faces\utils\UpdatableProperty.java
icefaces\component\src\com\icesoft\faces\component\paneltabset\PanelTabSet.java
icefaces\component\src\com\icesoft\faces\component\paneltabset\PanelTabSetRenderer.java
icefaces\component\src\com\icesoft\faces\resources\messages.properties
icefaces\component\src\com\icesoft\faces\resources\messages_fr.properties
Committed revision 15047

Found an issue where we need to keep the pre-decode selectedIndex available throughout all the pre-render phases, while the new selectedIndex is being properly updated, so that the phase notification forwarding, that's supposed to be recursed into the panelTabSet's children, will still go to the proper ones, in the pre-decode selectedIndex tab.

ICEfaces 1.6.2 branch commit: Subversion 15085

icefaces\component\src\com\icesoft\faces\component\paneltabset\PanelTabSet.java
icefaces\component\src\com\icesoft\faces\component\paneltabset\PanelTabSetRenderer.java
icefaces\component\src\com\icesoft\faces\utils\UpdatableProperty.java

Trunk commit: Subversion 15090.

icefaces\component\src\com\icesoft\faces\component\paneltabset\PanelTabSet.java
icefaces\component\src\com\icesoft\faces\component\paneltabset\PanelTabSetRenderer.java
icefaces\component\src\com\icesoft\faces\utils\UpdatableProperty.java

I committed some more changes to this, in ICE-2187, to make it work in UISeries (panelSeries, dataTable) containers.