Call to getData() on TreeGrid creates ID collision warning

[beckyo]

SmartGWT 4.1p 2014-05-06

Running into a warning in dev mode when calling getData inside a TreeGrid DataArrivedHandler.
Verified we're not calling setId. Pulled out miscellaneous code:

public class CustomTree extends TreeGrid
{
    public CustomTree()
    {
        init();
    }

    private void init()
    {
        addDataArrivedHandler(new DataArrivedHandler()
        {
            @Override
            public void onDataArrived(final DataArrivedEvent event)
            {
                if(getData() != null)   //warning occurs here
                {
                }
            }
        });
    }
}

Stack Info:

17:31:36.279 [ERROR] [cloudview1] 17:31:36.278:WARN:Log:Specified ID: isc_ResultTree_0 collides with the ID for an existing SmartGWT component or object. The existing object will be destroyed and the ID bound to the new object.
com.smartgwt.client.core.JsObject$SGWT_WARN: 17:31:36.278:WARN:Log:Specified ID: isc_ResultTree_0 collides with the ID for an existing SmartGWT component or object. The existing object will be destroyed and the ID bound to the new object.
    at sun.reflect.NativeConstructorAccessorImpl.newInstance0(Native Method)
    at sun.reflect.NativeConstructorAccessorImpl.newInstance(NativeConstructorAccessorImpl.java:57)
    at sun.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:45)
    at java.lang.reflect.Constructor.newInstance(Constructor.java:526)
    at com.google.gwt.dev.shell.MethodAdaptor.invoke(MethodAdaptor.java:105)
    at com.google.gwt.dev.shell.MethodDispatch.invoke(MethodDispatch.java:71)
    at com.google.gwt.dev.shell.OophmSessionHandler.invoke(OophmSessionHandler.java:172)
    at com.google.gwt.dev.shell.BrowserChannelServer.reactToMessagesWhileWaitingForReturn(BrowserChannelServer.java:338)
    at com.google.gwt.dev.shell.BrowserChannelServer.invokeJavascript(BrowserChannelServer.java:219)
    at com.google.gwt.dev.shell.ModuleSpaceOOPHM.doInvoke(ModuleSpaceOOPHM.java:136)
    at com.google.gwt.dev.shell.ModuleSpace.invokeNative(ModuleSpace.java:576)
    at com.google.gwt.dev.shell.ModuleSpace.invokeNativeVoid(ModuleSpace.java:304)
    at com.google.gwt.dev.shell.JavaScriptHost.invokeNativeVoid(JavaScriptHost.java:107)
    at com.smartgwt.client.util.SC.logWarn(SC.java)
    at com.smartgwt.client.util.IDManager.validateID(IDManager.java:75)
    at com.smartgwt.client.util.IDManager.registerID(IDManager.java:124)
    at com.smartgwt.client.core.BaseClass.registerID(BaseClass.java:77)
    at com.smartgwt.client.core.BaseClass.internalSetID(BaseClass.java:92)
    at com.smartgwt.client.widgets.tree.Tree.setJavaScriptObject(Tree.java:99)
    at com.smartgwt.client.widgets.tree.ResultTree.<init>(ResultTree.java:97)
    at com.smartgwt.client.widgets.tree.ResultTree.getOrCreateRef(ResultTree.java:86)
    at com.smartgwt.client.widgets.tree.TreeGrid.getData(TreeGrid.java:2656)
    at com.cloudcruiser.client.components.common.CustomTree$1.onDataArrived(CustomTree.java:167)
    at com.smartgwt.client.widgets.tree.events.DataArrivedEvent.dispatch(DataArrivedEvent.java:110)
    at com.smartgwt.client.widgets.tree.events.DataArrivedEvent.dispatch(DataArrivedEvent.java:1)
    at com.google.gwt.event.shared.GwtEvent.dispatch(GwtEvent.java:1)
    .......

[Isomorphic]

We would need a complete, ready-to-run test case in order to investigate this as a possible framework issue.

[beckyo]

I don't think this is your issue. I found a lot of manual firing of DataArrivedEvent and an override to invalidateCache that set the data to an empty Tree array. I cleared that override (it was totally unnecessary) and everything's looking good.