To make it clear: this "throws BindingException" bubbles up to the Binding.java, so it's in fact a interface change.

Joerg

On 02.02.2004 21:51, Joerg Heinicke wrote:

Regularly I have a problem with NPEs in binding when the binding references an ID which does not exist in the definition. The NPE is thrown in ValueJXPathBinding.java, line 129. What I want to add is to throw an Exception before depending on a null test on the widget. Therefore the method should declare that it throws a checked exception.

Has anybody any objections against the change of
public abstract void doLoad(Widget frmModel, JXPathContext jxpc);
to
public abstract void doLoad(Widget frmModel, JXPathContext jxpc) throws BindingException;
in org.apache.cocoon.woody.binding.JXPathBindingBase.java, line 149?


Joerg

Reply via email to