Node.Clone() doesn't set the correct parents for children.

spyzhov / ajson

Abstract JSON for Golang with JSONPath support

MIT License

246 stars 23 forks source link

Node.Clone() doesn't set the correct parents for children. #73

Closed yevgenin closed 2 months ago

yevgenin commented 2 months ago

The parent fields of children after a clone, still point to the original nodes, not the cloned ones. I wonder if this was intended? Anyway I propose a fix. See a PR I opened. #74

Also thank you for this great lib! Very useful.

spyzhov commented 2 months ago

Thanks! Looks catch, it was a bug :-) merged :+1: