lievendoclo / Valkyrie-RCP

A fork from http://www.gitorious.org/valkyrie-rcp
Apache License 2.0
23 stars 10 forks source link

NPE in RefreshableValueHolder ctor #8

Closed cmadsen closed 10 years ago

cmadsen commented 11 years ago
import java.util.Collections;
import org.valkyriercp.binding.value.support.RefreshableValueHolder;
import org.valkyriercp.rules.closure.Closure;

public class TestRefreshableValueHolder {
    public static void main(String[] args) {
        RefreshableValueHolder refreshableAssetTypeValueHolder = new RefreshableValueHolder(
                new Closure() {
                    @Override
                    public Object call(Object object) {
                        return Collections.EMPTY_LIST;
                    }
                }, true, false);
    }
}

Exception in thread "main" java.lang.NullPointerException at org.valkyriercp.binding.value.support.AbstractValueModel.hasValueChanged(AbstractValueModel.java:141) at org.valkyriercp.binding.value.support.ValueHolder.setValue(ValueHolder.java:40) at org.valkyriercp.binding.value.support.RefreshableValueHolder.refresh(RefreshableValueHolder.java:75) at org.valkyriercp.binding.value.support.RefreshableValueHolder.(RefreshableValueHolder.java:50) at dk.navicon.valkyrie.test.TestRefreshableValueHolder.main(TestRefreshableValueHolder.java:12)

cmadsen commented 11 years ago

AbstractValueModel is @Configurable and valueChangeDetector is @Autowired and used in constructor ie NPE

lievendoclo commented 11 years ago

Have you tried using the preConstruct=true flag on @Configurable?

lievendoclo commented 10 years ago

fixed