Ext 4.1.1 RC1 tree node deep copy is not deep

[campersau1]

The NodeInterface.copy method doesn't copy the childnodes which are deeper than level 1.

Here is the fix:

NodeInterface.js

                copy: function(newId, deep) {
                    var me = this,
                        //result = me.callOverridden(arguments),
                        result = me.createNode(me.callOverridden(arguments)),
                        len = me.childNodes ? me.childNodes.length : 0,
                        i;


                    // Move child nodes across to the copy if required
                    if (deep) {
                        for (i = 0; i < len; i++) {
//                            result.appendChild(me.childNodes[i].copy(true));
                            result.appendChild(me.childNodes[i].copy(newId, deep));
                        }
                    }
                    return result;
                },

This bug was metioned before here:
http://www.sencha.com/forum/showthre...ight=node+copy

[mitchellsimoens]

Thanks for the report.

[campersau1]

any news on this?

it is really annoying to change that code in every release.

[cdomigan]

Note that this method also has to be changed in NodeInterface.getPrototypeBody() if you are going to be decorating models as Nodes!