null binder element npe

This commit is contained in:
Keith Donald
2008-09-16 19:42:32 +00:00
parent 8dc8537171
commit 24bb3a894e
2 changed files with 17 additions and 22 deletions

View File

@@ -628,23 +628,25 @@ public class FlowModelFlowBuilder extends AbstractFlowBuilder {
}
private BinderConfiguration createBinderConfiguration(BinderModel binderModel) {
if (binderModel == null) {
if (binderModel != null && binderModel.getBindings() != null) {
BinderConfiguration binderConfiguration = new BinderConfiguration();
List bindings = binderModel.getBindings();
for (Iterator it = bindings.iterator(); it.hasNext();) {
BindingModel bindingModel = (BindingModel) it.next();
boolean required;
if (StringUtils.hasText(bindingModel.getRequired())) {
required = ((Boolean) fromStringTo(Boolean.class).execute(bindingModel.getRequired()))
.booleanValue();
} else {
required = false;
}
Binding binding = new Binding(bindingModel.getProperty(), bindingModel.getConverter(), required);
binderConfiguration.addBinding(binding);
}
return binderConfiguration;
} else {
return null;
}
BinderConfiguration binderConfiguration = new BinderConfiguration();
List bindings = binderModel.getBindings();
for (Iterator it = bindings.iterator(); it.hasNext();) {
BindingModel bindingModel = (BindingModel) it.next();
boolean required;
if (StringUtils.hasText(bindingModel.getRequired())) {
required = ((Boolean) fromStringTo(Boolean.class).execute(bindingModel.getRequired())).booleanValue();
} else {
required = false;
}
Binding binding = new Binding(bindingModel.getProperty(), bindingModel.getConverter(), required);
binderConfiguration.addBinding(binding);
}
return binderConfiguration;
}
private ViewVariable[] parseViewVariables(List vars) {

View File

@@ -25,13 +25,6 @@ public class BinderModel extends AbstractModel {
private LinkedList bindings;
public void addBinding(BindingModel bindingModel) {
if (bindings == null) {
bindings = new LinkedList();
}
bindings.add(bindingModel);
}
public LinkedList getBindings() {
return bindings;
}