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

Removing PrinterCommunication from ActiveMaterialKey

Browse source 
issue: MatterHackers/MCCentral#4631
Remove PrinterConnection dependency in PrinterSettings.SetMaterialPreset
This commit is contained in:
Lars Brubaker 2018-11-26 14:23:03 -08:00
parent 5b5cfdff62
commit 04211ec613
3 changed files with 11 additions and 30 deletions
Download patch file Download diff file Expand all files Collapse all files

37
MatterControlLib/SlicerConfiguration/Settings/PrinterSettings.cs
View file

@ -339,37 +339,18 @@ namespace MatterHackers.MatterControl.SlicerConfiguration
}
internal set
{
SetMaterialPreset(0, value);
if (MaterialSettingsKeys.Count == 0 || value != MaterialSettingsKeys[0])
{
MaterialSettingsKeys.Clear();
MaterialSettingsKeys.Add(value);
MaterialLayer = GetMaterialLayer(value);
this.OnMaterialPresetChanged();
Save();
}
}
}
private void SetMaterialPreset(int extruderIndex, string materialKey)
{
if (extruderIndex >= PrinterCommunication.PrinterConnection.MAX_EXTRUDERS)
{
throw new ArgumentOutOfRangeException("Requested extruder index is outside of bounds: " + extruderIndex);
}
// TODO: This should really be in PrinterProfile and should be run when the extruder count changes
if (MaterialSettingsKeys.Count <= extruderIndex)
{
var resizedArray = new string[extruderIndex + 1];
MaterialSettingsKeys.CopyTo(resizedArray);
MaterialSettingsKeys = new List<string>(resizedArray);
}
MaterialSettingsKeys[extruderIndex] = materialKey;
if (extruderIndex == 0)
{
MaterialLayer = GetMaterialLayer(materialKey);
this.OnMaterialPresetChanged();
}
Save();
}
public List<string> MaterialSettingsKeys { get; set; } = new List<string>();
[JsonIgnore]