Bug: LeadInOut dressup created on Cancel

Here's the place for discussion related to CAM/CNC and the development of the Path module.
Forum rules
Be nice to others! Respect the FreeCAD code of conduct!
Post Reply
Russ4262
Posts: 952
Joined: Sat Jun 30, 2018 3:22 pm
Location: Oklahoma
Contact:

Bug: LeadInOut dressup created on Cancel

Post by Russ4262 »

Evening,
I was working on another related issue and discovered that the LeadInOut Dressup creates the dressup upon activation even when the Cancel button is pressed in its task panel.

Steps to reproduce:
  1. Open a model file, and open Path workbench.
  2. Select the model, and create a Job object with defaults accepted.
  3. Create Profile object with defaults accepted, no feature selection (Base Geometry).
  4. Right-click on Profile in object tree and select LeadInOut Dressup. (Or, select Profile in object tree and select the dressup from the Path menu.)
  5. Click the Cancel button in the LeadInOut task panel.
  6. The dressup is still created and added to the object tree as if OK was pressed.
Can anyone confirm? Sample model file attached.

Thanks,
Russell

Code: Select all

OS: Windows 10 Version 2009
Word size of FreeCAD: 64-bit
Version: 0.20.28793 (Git)
Build type: Release
Branch: master
Hash: 72e91dbbca9094d9bd56b1d47a1eeaaeb7266849
Python 3.8.6+, Qt 5.15.2, Coin 4.0.1, OCC 7.5.0
Locale: English/United States (en_US)
Installed mods: 
  * FC_SU
  * Z_MacroStartup
Attachments
PartDesignExample_2.FCStd
Sample model file for testing bug
(56.48 KiB) Downloaded 17 times
jescombe
Posts: 90
Joined: Tue Mar 09, 2021 4:19 pm

Re: Bug: LeadInOut dressup created on Cancel

Post by jescombe »

Russ4262 wrote: Fri May 06, 2022 11:36 pm Can anyone confirm? Sample model file attached.
Yes, can confirm I see the same behaviour (28671 appimage)..

Regards,
Jon
Russ4262
Posts: 952
Joined: Sat Jun 30, 2018 3:22 pm
Location: Oklahoma
Contact:

Re: Bug: LeadInOut dressup created on Cancel

Post by Russ4262 »

Afternoon,
With LuzPaz request, Path: Fixes #6849 - Properly cancel new Lead In/Out dressup is now active as a possible solution.

Russell

Edit: PR closed 14 July
Post Reply