Learning/mRemoteNG
1
0
Fork 0
mirror of https://github.com/mRemoteNG/mRemoteNG.git synced 2026-02-26 12:08:37 +08:00
Code Issues Packages Projects Releases Wiki Activity

fixes #971

Browse Source 
Remove the "machineNode" XML node so that settings are portable to all systems
This commit is contained in:
Sean Kaim
2018-07-25 13:02:30 -04:00
parent 930579d983
commit 7e24e2dcfb
1 changed files with 2 additions and 17 deletions
Download Patch File Download Diff File Expand all files Collapse all files

19
mRemoteV1/Config/Settings/Providers/PortableSettingsProvider.cs
View File

@@ -45,20 +45,7 @@ namespace mRemoteNG.Config.Settings.Providers
private string _filePath => Path.Combine(Path.GetDirectoryName(Application.ExecutablePath) ?? throw new InvalidOperationException(), $"{ApplicationName}.settings");
private XmlNode _localSettingsNode
{
get
{
var settingsNode = GetSettingsNode(_localSettingsNodeName);
var machineNode = settingsNode.SelectSingleNode(Environment.MachineName.ToLowerInvariant());
if (machineNode != null) return machineNode;
machineNode = _rootDocument.CreateElement(Environment.MachineName.ToLowerInvariant());
settingsNode.AppendChild(machineNode);
return machineNode;
}
}
private XmlNode _localSettingsNode => GetSettingsNode(_localSettingsNodeName);
private XmlNode _globalSettingsNode => GetSettingsNode(_globalSettingsNodeName);
@@ -138,9 +125,7 @@ namespace mRemoteNG.Config.Settings.Providers
private void SetValue(SettingsPropertyValue propertyValue)
{
var targetNode = IsGlobal(propertyValue.Property)
? _globalSettingsNode
: _localSettingsNode;
var targetNode = IsGlobal(propertyValue.Property) ? _globalSettingsNode : _localSettingsNode;
var settingNode = targetNode.SelectSingleNode($"setting[@name='{propertyValue.Name}']");