Fixes for SCH_GROUPs.

eeschema/sch_group.cpp

@@ -427,8 +427,8 @@ void SCH_GROUP::RunOnChildren( const std::function<void( SCH_ITEM* )>& aFunction
         {
             aFunction( static_cast<SCH_ITEM*>( item ) );
 
-            if( item->Type() == SCH_GROUP_T )
-                static_cast<SCH_ITEM*>( item )->RunOnChildren( aFunction, RECURSE_MODE::RECURSE );
+            if( item->Type() == SCH_GROUP_T && aMode == RECURSE_MODE::RECURSE )
+                static_cast<SCH_GROUP*>( item )->RunOnChildren( aFunction, RECURSE_MODE::RECURSE );
         }
     }
     catch( std::bad_function_call& )

eeschema/tools/sch_line_wire_bus_tool.cpp

@@ -53,6 +53,7 @@
 #include <sch_line.h>
 #include <sch_screen.h>
 #include <sch_sheet.h>
+#include <sch_group.h>
 #include <sch_sheet_pin.h>
 #include <schematic.h>
 #include <sch_commit.h>
@@ -1321,9 +1322,24 @@ int SCH_LINE_WIRE_BUS_TOOL::TrimOverLappingWires( SCH_COMMIT* aCommit, SCH_SELEC
 
 int SCH_LINE_WIRE_BUS_TOOL::AddJunctionsIfNeeded( SCH_COMMIT* aCommit, SCH_SELECTION* aSelection )
 {
-    SCH_SCREEN* screen = m_frame->GetScreen();
+    SCH_SCREEN*           screen = m_frame->GetScreen();
+    std::deque<EDA_ITEM*> allItems;
 
-    for( const VECTOR2I& point : screen->GetNeededJunctions( aSelection->Items() ) )
+    for( EDA_ITEM* item : aSelection->Items() )
+    {
+        allItems.push_back( item );
+
+        if( item->Type() == SCH_GROUP_T )
+        {
+            static_cast<SCH_GROUP*>( item )->RunOnChildren(
+                    [&]( SCH_ITEM* child )
+                    {
+                        allItems.push_back( child );
+                    }, RECURSE_MODE::RECURSE );
+        }
+    }
+
+    for( const VECTOR2I& point : screen->GetNeededJunctions( allItems ) )
         m_frame->AddJunction( aCommit, m_frame->GetScreen(), point );
 
     return 0;