tree node copy(deep) not working (4.0.1)

[christophe.geiser]

Problem still present under B3.
C.

[cyborat]

Bump! This feature will be a huge selling point to my manager and our clients!

[christophe.geiser]

In general, I really appreciate performance and code improvement of 4.1 (to be honest I felt really worried at some point...). So thanks for the good work and effort.

Now, on this one.
It has been almost one year that this bug has been reported and regularly bumped, and still nothing (not even a bug-id or acknowledgement).

Since the problem lies in a static function being applied while decorating tree nodes, a simple override will not work, hence the code below. But please, take this into account into the code base.

Cheers,
C.

(function() {
	var initBody = Ext.data.NodeInterface.getPrototypeBody();
	Ext.data.NodeInterface.addStatics({getPrototypeBody : function() {
				return Ext.apply(
					  initBody, {
					 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;
			                    if (deep) {
			                        for (i = 0; i < len; i++) {
			                       //     result.appendChild(me.childNodes[i].copy(true));
			                            result.appendChild(me.childNodes[i].copy(null, true));
			                        }
			                    }
			                    return result;
			            }
				})
			}
	})
})();

[andrevio]

Thanks for posting the workaround! I do hope that it'd be incorporated into the main product, at least in 4.1.

[bizbox]

Bug still present in 4.1.1
Any plan to fix it when working workaround provided from community already exists?

[christophe.geiser]

Bumping, and bouncing.
Cheers,
C.

[evant]

Resolved in EXTJSIV-7261.

[christophe.geiser]

Excellent, thanks .