Ext.each(menu.items, myFunction) not working?

[dncpax]

Hello.

I'm starting with ExtJS. While trying to cycle a split button's items I get undefined errors with:
Ext.each(menu.items, myFunction)

While the following will work fine:
Ext.each(menu.items.items, myFunction)

I don't know if this is by design, or I'm doing something wrong, but I feel there must be a more elegant way of cycling menu items. I don't use menu.items.each(), because I need to recursivelly cycle all buttons/menu items in a toolbar...
So I guess this would be question #2??

Thanks,
Duarte

[Condor]

Read the docs! Ext.each iterates over an Array. Ext.menu.Menu.items is a MixedCollection and not an Array.

For a normal container structure you would use cascade(), but the menuitem/button-menu relation isn't a container-child relation, so that won't work.

[dncpax]

Condor, hmm that doesn't seem very nice. I'm reading the docs, believe me. It seems that particular nuance eluded me.

I'm not using items.each() since, as I said, I need to recursively access all items in a toolbar.
So I'm doing:

toolbar.items.each(
  function(item) {
    //operate on this item
    //then go through its children
    if(item.menu)
      item.menu.items.each(arguments.callee(item));
  }
);

And this does not work, since item is already defined when I reach the last .each call... and that's why I tried Ext.each instead... This is working but I feel this is not correct.

Any more suggestions? Anyone?

Thanks,
Duarte

[evant]

Just declare the fn:

var m, fn = function(item){
    // ...
    if((m = item.menu)){
        m.items.each(fn);
    }
};

toolbar.items.each(fn);

[dncpax]

evant, that is so simple it makes me smile! Works a charm.

Thanks.
Duarte