Group not updated correctly when changing groups

[fidelthomet]

I tried changing the group of an object, at first sight it seems to work just fine, as the DOM is updated just fine. However looking in the group object of the original group the element is still listed there. Therefore the Object is not in sync with the actual SVG Structure.

[nickdoesdesign]

The way it looks to me, without having direct access to your code, is you didn't remove the object from the original group. If you are attempting to reassign an object to a new group, you need to remember to remove the original reference to it, or else it will stay as a reference within the original object in memory.

To think of it another way, think of a row of books, and you somehow make a copy of one of them and put it in another row. If you look up that book, it would then have two locations in which it existed, unless you remove or destroy the original book.

If you look at the source, Paper.g() is defined as such:

proto.group = proto.g = function (first) {
        var el = make("g", this.node);
        el.add = add2group;
        for (var method in proto) if (proto[has](method)) {
            el[method] = proto[method];
        }
        if (arguments.length == 1 && first && !first.type) {
            el.attr(first);
        } else if (arguments.length) {
            el.add(Array.prototype.slice.call(arguments, 0));
        }
        return el;
    };

This calls an add method on the el, but does not do any destruction of the original reference. So, when calling group on an element that exists in a previous group, and you do not wish to have it remain in there, you are going to need to remove the original reference. Now, if you look at the API docs, there is a Set method that can probably do more of what you need it to do, in terms of instantiating and removing objects, reference of which is here: http://snapsvg.io/docs/#Set

I hope this helps!