Body problem in assembly when using shape binder about freecad

[Pendelf]

If you drag and drop bodies, everything works fine

[maxwxyz]

Does it work when you manually drag both into the assembly object in the tree view?
@PaddleStroke FYI

[PaddleStroke]

This is the same bug as #13780
The issue is that getRootObject is returning the root of dependency graph and not root of the tree. I made this getRootObjectsIgnoreLinks to solve the initial problem. But I have not taken into account the shape binders which are same as links.
And there are also Draft and PartDesign Clones.

We could add

            return obj->isDerivedFrom<App::Link>() 
                || obj->isDerivedFrom(Base::Type::fromName("PartDesign::SubShapeBinder"));

to fix the case of subShapeBinder, but that cannot solve Draft clone case for instance, as those objects are Part::FeaturePython. Also there might be a lot of case of other objects referencing tree root objects.

@wwmayer is there not a clean way to get the tree-root-level objects? The tree is being build so there must be a way to get that.
See Pendelf video where he shows the problem : https://youtu.be/f-PxEIDXzF4

[wwmayer]

If you use shape binder to create one of the bodies, the first body will not be available when adding bodies to the assembly

It doesn't work with the sub-shape binder (green icon) but works without problems with the shape binder (blue icon).

is there not a clean way to get the tree-root-level objects?

The method claimChildren() of a view provider shows the referenced objects as child items in the tree view. So, all objects that are not claimed by a view provider must be top-level objects.

But does this really solve the problem? What happens if the two bodies are inside a Part container?

[PaddleStroke]

So to get the tree root object you iterate through all the document objects, build the list of objects that are within at least one of the object VP claimChildren return function, then the root is the rest of the objects right?

If so I will make a 'getTreeRootObject' in Gui.

Yes it solves the problem because 'insert component' is building a tree starting from root objects. So if Part container is a root object, it will be picked up. Then it is iteratively browsed to build the tree so the bodies within are found.