diff --git a/OpenNest.IO/CadImportResult.cs b/OpenNest.IO/CadImportResult.cs index d040ecd..9b0cf39 100644 --- a/OpenNest.IO/CadImportResult.cs +++ b/OpenNest.IO/CadImportResult.cs @@ -1,5 +1,4 @@ using System.Collections.Generic; -using ACadSharp; using OpenNest.Bending; using OpenNest.Geometry; @@ -30,12 +29,6 @@ namespace OpenNest.IO /// public Box Bounds { get; set; } - /// - /// Underlying CAD document for callers that need to run additional - /// analysis (e.g., MText extraction). May be null for stream imports. - /// - public CadDocument Document { get; set; } - /// /// Absolute path to the source file. /// diff --git a/OpenNest.IO/CadImporter.cs b/OpenNest.IO/CadImporter.cs index ef989e9..aa2ed4a 100644 --- a/OpenNest.IO/CadImporter.cs +++ b/OpenNest.IO/CadImporter.cs @@ -42,7 +42,6 @@ namespace OpenNest.IO Entities = dxf.Entities, Bends = bends, Bounds = dxf.Entities.GetBoundingBox(), - Document = dxf.Document, SourcePath = path, Name = options.Name ?? Path.GetFileNameWithoutExtension(path), }; diff --git a/OpenNest.Tests/IO/CadImporterTests.cs b/OpenNest.Tests/IO/CadImporterTests.cs index 7a52338..bceb883 100644 --- a/OpenNest.Tests/IO/CadImporterTests.cs +++ b/OpenNest.Tests/IO/CadImporterTests.cs @@ -39,6 +39,18 @@ namespace OpenNest.Tests.IO Assert.Equal("custom", result.Name); } + [Fact] + public void Import_WhenNamedDetectorDoesNotExist_ReturnsEmptyBends() + { + // Exercises the named-detector branch: when BendDetectorName doesn't + // match any registered detector, bends should be an empty list + // (not a crash, and no fall-through to auto-detect). + var result = CadImporter.Import(TestDxf, + new CadImportOptions { BendDetectorName = "__nonexistent__" }); + + Assert.Empty(result.Bends); + } + [Fact] public void BuildDrawing_ProducesDrawingWithProgramAndMetadata() { diff --git a/OpenNest/Forms/CadConverterForm.cs b/OpenNest/Forms/CadConverterForm.cs index 3242d14..53fff87 100644 --- a/OpenNest/Forms/CadConverterForm.cs +++ b/OpenNest/Forms/CadConverterForm.cs @@ -5,7 +5,6 @@ using OpenNest.Converters; using OpenNest.Geometry; using OpenNest.IO; using OpenNest.IO.Bending; -using OpenNest.Properties; using System; using System.Collections.Generic; using System.Drawing; @@ -356,7 +355,6 @@ namespace OpenNest.Forms : Path.GetTempPath(); var index = fileList.SelectedIndex; - var newItems = new List(); var splitWriter = new SplitDxfWriter(); var splitItems = new List(); @@ -369,7 +367,6 @@ namespace OpenNest.Forms var splitPath = GetUniquePath(Path.Combine(writableDir, splitName)); splitWriter.Write(splitPath, splitDrawing); - newItems.Add(splitPath); // Re-import geometry but keep bends from the split drawing var result = Dxf.Import(splitPath); @@ -750,9 +747,6 @@ namespace OpenNest.Forms item.SuppressedEntityIds = null; } - - private static Color GetNextColor() => Drawing.GetNextColor(); - private static bool IsDirectoryWritable(string path) { try