aj/OpenNest
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

fix: use Part.Rotate() in PlateView to avoid mutating shared Programs

Browse Source 
RotateSelectedParts was calling Program.Rotate() directly on shared
Program instances, bypassing Part's copy-on-write (EnsureOwnedProgram).
Parts created via CloneAtOffset share the same Program, so rotating one
part would rotate all parts with the same reference.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This commit is contained in:
AJ Isaacs
2026-03-21 23:08:47 -04:00
parent 8ffdacd6c0
commit b6ee04f038
1 changed files with 3 additions and 10 deletions
Download Patch File Download Diff File Expand all files Collapse all files
OpenNest/Controls/PlateView.cs
+3 -10
View File
@@ -1098,23 +1098,16 @@ namespace OpenNest.Controls
var bounds = parts.GetBoundingBox(); var bounds = parts.GetBoundingBox();
var center = bounds.Center; var center = bounds.Center;
var anchor = bounds.Location; var anchor = bounds.Location;
var rotatedPrograms = new HashSet<Program>();
for (int i = 0; i < SelectedParts.Count; ++i) for (var i = 0; i < SelectedParts.Count; ++i)
{ {
var part = SelectedParts[i]; var part = SelectedParts[i];
var basePart = part.BasePart; part.BasePart.Rotate(angle, center);
if (rotatedPrograms.Add(basePart.Program))
basePart.Program.Rotate(angle);
part.Location = part.Location.Rotate(angle, center);
basePart.UpdateBounds();
} }
var diff = anchor - parts.GetBoundingBox().Location; var diff = anchor - parts.GetBoundingBox().Location;
for (int i = 0; i < SelectedParts.Count; ++i) for (var i = 0; i < SelectedParts.Count; ++i)
SelectedParts[i].Offset(diff); SelectedParts[i].Offset(diff);
} }