kicad-developers team mailing list archive
-
kicad-developers team
-
Mailing list archive
-
Message #42876
Fix lp:1849376
Hi,
Here attached you'll find a patch to correct issue 1849376 on launchpad.
When a component is placed, action addNeededJunctions is now triggered.
If some of you have some spare time, I also have two branches with
patches you can review and/or test. The first allows the hierarchy
navigator to stay open:
https://github.com/fjullien/kicad/tree/navigator_v6
The second branch as a patch adding a function to insert intersheets
references attached to global labels:
https://github.com/fjullien/kicad/tree/iref
Thanks in advance.
Franck.
From 6e2788c5153eaab4e312decd361b4fa9914b1eb5 Mon Sep 17 00:00:00 2001
From: Franck Jullien <franck.jullien@xxxxxxxxx>
Date: Tue, 26 Nov 2019 22:03:19 +0100
Subject: [PATCH] eechema: add junction if needed when component is placed
Fixes: lp:1849376
* https://bugs.launchpad.net/kicad/+bug/1849376
---
eeschema/tools/sch_drawing_tools.cpp | 3 +++
1 file changed, 3 insertions(+)
diff --git a/eeschema/tools/sch_drawing_tools.cpp b/eeschema/tools/sch_drawing_tools.cpp
index 05d6a0a04..35a3db363 100644
--- a/eeschema/tools/sch_drawing_tools.cpp
+++ b/eeschema/tools/sch_drawing_tools.cpp
@@ -191,6 +191,9 @@ int SCH_DRAWING_TOOLS::PlaceComponent( const TOOL_EVENT& aEvent )
m_view->ClearPreview();
m_frame->AddItemToScreenAndUndoList( component );
+ m_toolMgr->RunAction( EE_ACTIONS::addNeededJunctions, true, component );
+ m_frame->OnModify();
+
if( m_frame->GetUseAllUnits() || m_frame->GetRepeatComponent() )
{
int new_unit = component->GetUnit();
--
2.17.1
Follow ups