developed a patch which should solve this problem.
The BlockingServlet is the patch for ICEFaces 1.5.x and ReceiveSendUpdates is for ICEFaces 1.6.0.

patch for 1.6

patch for 1.5.3

relevant changes to ReceiveSendUpdates:

— ReceiveSendUpdates_updated.java 2007-04-11 09:05:43.000000000 -0600
@@ -56,49 +56,63 @@
}
}

private void renderCyclePartial(FacesContext context,
UIComponent component) {
synchronized (context)

}

• private void setRequiredTrue(List requiredComponents) {
• Iterator i = requiredComponents.iterator();
  + private void setRequiredTrue(Map<UIInput,ValueBinding> requiredComponents)
  Unknown macro: {+ Iterator<Entry<UIInput, ValueBinding>> i = requiredComponents.entrySet().iterator();+ UIInput next = null; while (i.hasNext()) { - next = (UIInput) i.next(); - ((UIInput) next).setRequired(true); + Entry<UIInput, ValueBinding> entry = i.next(); + next = entry.getKey(); + ValueBinding valueBinding = entry.getValue(); + if(valueBinding==null) + next.setRequired(true); + else + next.setValueBinding("required", valueBinding); } }

• private List setRequiredFalseInFormContaining(UIComponent component) {
• List alteredComponents = new ArrayList();
  + private Map<UIInput,ValueBinding> setRequiredFalseInFormContaining(UIComponent component) { + Map<UIInput,ValueBinding> alteredComponents = new HashMap<UIInput,ValueBinding>(); UIComponent form = getContainingForm(component); setRequiredFalseOnAllChildrenExceptOne(form, component, alteredComponents); return alteredComponents; }

private void setRequiredFalseOnAllChildrenExceptOne(UIComponent parent,
UIComponent componentToAvoid,

• List alteredComponents) {
  + Map<UIInput,ValueBinding> alteredComponents) {
  int length = parent.getChildCount();
  UIComponent next = null;
  for (int i = 0; i < length; i++) {
  next = (UIComponent) parent.getChildren().get;
  if (next instanceof UIInput && next != componentToAvoid) {
• if (((UIInput) next).isRequired()) {
• ((UIInput) next).setRequired(false);
• alteredComponents.add(next);
  + UIInput input = (UIInput) next;
  + if (input.isRequired()) {
  + ValueBinding valueBinding = input.getValueBinding("required");
  + if(valueBinding==null)
  + input.setRequired(false);
  + else
  Unknown macro: {+ ValueBinding falseValueBinding = FacesContext.getCurrentInstance().getApplication().createValueBinding("#{false}");+ input.setValueBinding("required", falseValueBinding);+ }

  +
  + alteredComponents.put(input,valueBinding);
  }
  }
  setRequiredFalseOnAllChildrenExceptOne(next, componentToAvoid,
  alteredComponents);
  }
  }
  @@ -113,9 +127,7 @@
  break;
  }
  parent = parent.getParent();
  }
  return (UIForm) parent;
  }
  -
  -
  }

We'll have to modify this fix a bit so that it doesn't require generics (that would break 1.4 compatibility), but overall it looks good.

Is it necessary to create the #

or #

once and use it for all components.

The following syntax

if (conditional)

else

is preferred over

if (conditional)
function();
else
otherFunction();

as the second form is fragile under code maintenance.

Hi, I'm the patch author.
I think it's necessary create the #

valueBinding.
In fact, after the constant value is set ( using setRequired(false)), assigning a valueBinding has no effect and it's not possible assigning a valueBinding any more.
Please, look at the javax.faces.component.UIInput code:

-------------------------------------------------------------------
public void setRequired(boolean required)

public boolean isRequired() {

if (this.requiredSet)

}
------------------------------------------------------

As you can see, after the setRequired() method is called, the property "requiredSet" is set and the subsequent calls to isRequired() ignore the valueBinding.

I see what you mean about #

.

Thanks for your contribution. A modification of the provided fix has been checked in and should propagate to the public repository later today.

The required attribute is not working when contained in a UIData component. Please see https://www.icesoft.ca:4443/supportilla/show_bug.cgi?id=4030 for more details.

The proposed change in the suportilla case is a major re-work of this part of the implementation so will require some consideration.

I'll be glad to talk about it, if you like. We've already made in our enviroment and it works fine (untill now, of course ).

Marking this fixed for the non-UIData component cases.

There is a follow-up issue for the UIData components: ICE-1654.