AccordionContainer

[Jean-Pascal Laux]

I found a bug in AccordionContainer. When I destroy an AccordionContainer and recreate later a new AccordionContainer with the same Id, I get the content of each child container at the same time and no headers have been displayed. This bug still exists in the version 1.3.2. I have looked at the code and have found the error (I changed the code and it is OK)

The function in cause is :

destroy:function(){
dojo.forEach(this.getChildren(),function(_b){
_b._buttonWidget.destroy();
});

the code is right except that this function should be named destroyRecursive to work correctly

[bill]

Please attach a test case using the attach file button.

The destroy() code you quoted is correct. AccordionContainer.destroy(), as specified in the definition of destroy() in _Widget.js, is supposed to destroy the AccordionContainer but not the children, where "children" means the underlying ContentPanes. The buttons, however, are part of the AccordionContainer and should be removed in the destroy() call.