nmaitland / smartgwt

Automatically exported from code.google.com/p/smartgwt
0 stars 0 forks source link

tabSet.setTabs(new Tab[]{}) is not removing all tabs #721

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
Here is my testcase:

  public void onModuleLoad() {

        VLayout layout = new VLayout();
        layout.setHeight100();
        layout.setWidth100();

        TabSet tabSet = new TabSet();

        if(tabSet.getTabs().length==0)
            System.err.println("test1 ok");
        else
            System.err.println("test1 non ok");

        tabSet.addTab(new Tab("tab1"));
        tabSet.addTab(new Tab("tab2"));
        tabSet.addTab(new Tab("tab3"));

        if(tabSet.getTabs().length==3)
            System.err.println("test2 ok");
        else
            System.err.println("test2 non ok");

        tabSet.setTabs(new Tab[]{});

        if(tabSet.getTabs().length==0)
            System.err.println("test3 ok");
        else
            System.err.println("test3 non ok");

        layout.addMember(tabSet);
        layout.draw();
  }

What steps will reproduce the problem?
1. Run the above example code
2. see the messages at the console

What is the expected output? What do you see instead?
The tabSet is expected to have zero tabs, the same behaviour as when we do 
dynamicForm.setFields(new FormItem[]{}) to remove all fieldItems of a 
DynamicForm. Instead, the tabSet still have 3 tabs.

What version of the product are you using? On what operating system?
I use SmartGWT 4.1. My OS is Windows7.

What browser(s) does this happen in?  Are there any browsers where the issue 
does not occur?
I see in debug mode and in all browser. I think that the implementation is not 
aligned to the others implementations, like dynamicForm.setFields

Original issue reported on code.google.com by diego.a...@gmail.com on 27 Aug 2013 at 10:02