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

Revise naming to clarify behavior

Browse source
This commit is contained in:
John Lewin 2018-11-21 09:48:43 -08:00
parent be1062e7d1
commit ba862715c7
2 changed files with 5 additions and 5 deletions
Download patch file Download diff file Expand all files Collapse all files

4
MatterControlLib/ApplicationView/ApplicationController.cs
View file

@ -390,7 +390,7 @@ namespace MatterHackers.MatterControl
public void ClosePrinter(PrinterConfig printer, bool allowChangedEvent = true)
{
// Actually clear printer
ProfileManager.Instance.RemoveOpenPrinter(printer.Settings.ID);
ProfileManager.Instance.ClosePrinter(printer.Settings.ID);
_activePrinters.Remove(printer);
@ -1756,7 +1756,7 @@ namespace MatterHackers.MatterControl
{
if (!_activePrinters.Any(p => p.Settings.ID == printerID))
{
ProfileManager.Instance.AddOpenPrinter(printerID);
ProfileManager.Instance.OpenPrinter(printerID);
var printer = new PrinterConfig(await ProfileManager.LoadSettingsAsync(printerID));

6
MatterControlLib/SlicerConfiguration/Settings/ProfileManager.cs
View file

@ -125,7 +125,7 @@ namespace MatterHackers.MatterControl.SlicerConfiguration
if (openedPrinter != null)
{
// Clear selected printer state
ProfileManager.Instance.RemoveOpenPrinter(printerID);
ProfileManager.Instance.ClosePrinter(printerID);
}
UiThread.RunOnIdle(() =>
@ -245,7 +245,7 @@ namespace MatterHackers.MatterControl.SlicerConfiguration
[JsonIgnore]
public IEnumerable<string> OpenPrinterIDs => _activeProfileIDs;
public void AddOpenPrinter(string printerID)
public void OpenPrinter(string printerID)
{
try
{
@ -260,7 +260,7 @@ namespace MatterHackers.MatterControl.SlicerConfiguration
}
}
public void RemoveOpenPrinter(string printerID)
public void ClosePrinter(string printerID)
{
try
{