nettika/mattercontrol
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Fix SaveAsWindow regression exposed by failing test

Browse source
This commit is contained in:
John Lewin 2017-09-16 10:29:52 -07:00
parent 20315077bf
commit 8cfb1e4b7f
2 changed files with 1 additions and 14 deletions
Download patch file Download diff file Expand all files Collapse all files

12
PartPreviewWindow/View3D/View3DWidget.cs
View file

@ -2314,16 +2314,6 @@ namespace MatterHackers.MatterControl.PartPreviewWindow
sceneContext,
async (returnInfo) =>
{
// TODO: The PrintItemWrapper seems unnecessary in the new LibraryContainer model. Couldn't we just pass the scene to the LibraryContainer via it's add function, no need to perist to disk?
// Create a new PrintItemWrapper
var printItemWrapper = new PrintItemWrapper(
new PrintItem()
{
Name = returnInfo.newName,
FileLocation = Path.ChangeExtension(returnInfo.fileNameAndPath, ".mcx")
},
returnInfo.DestinationContainer);
// Save the scene to disk
await this.SaveChanges();
@ -2335,7 +2325,7 @@ namespace MatterHackers.MatterControl.PartPreviewWindow
{
writableContainer.Add(new[]
{
new FileSystemFileItem(printItemWrapper.FileLocation)
new FileSystemFileItem(sceneContext.printItem.FileLocation)
{
Name = returnInfo.newName
}