Use Settings class to parse input settings

Kapil Borle · Kapil Borle · commit f371b9879794 · 2017-02-22T22:00:16.000-08:00
diff --git a/Engine/Commands/InvokeScriptAnalyzerCommand.cs b/Engine/Commands/InvokeScriptAnalyzerCommand.cs
@@ -273,12 +273,16 @@ protected override void BeginProcessing()
 
             if (settingsMode != SettingsMode.None)
             {
-                var settingFileHasErrors = !ScriptAnalyzer.Instance.ParseProfile(settingsFound, this.SessionState.Path, this);
-                if (settingFileHasErrors)
+                try
                 {
-                    this.WriteWarning("Cannot parse settings. Will abort the invocation.");
-                    stopProcessing = true;
-                    return;
+                    var settingsObj = new Settings(settingsFound);
+                    ScriptAnalyzer.Instance.UpdateSettings(settingsObj);
+                }
+                catch
+                {
+                        this.WriteWarning("Cannot parse settings. Will abort the invocation.");
+                        stopProcessing = true;
+                        return;
                 }
             }
 

Original file line number	Diff line number	Diff line change
`@@ -273,12 +273,16 @@ protected override void BeginProcessing()`
`273`	`273`
`274`	`274`	`if (settingsMode != SettingsMode.None)`
`275`	`275`	`{`
`276`		`- var settingFileHasErrors = !ScriptAnalyzer.Instance.ParseProfile(settingsFound, this.SessionState.Path, this);`
`277`		`- if (settingFileHasErrors)`
	`276`	`+ try`
`278`	`277`	`{`
`279`		`- this.WriteWarning("Cannot parse settings. Will abort the invocation.");`
`280`		`- stopProcessing = true;`
`281`		`- return;`
	`278`	`+ var settingsObj = new Settings(settingsFound);`
	`279`	`+ ScriptAnalyzer.Instance.UpdateSettings(settingsObj);`
	`280`	`+ }`
	`281`	`+ catch`
	`282`	`+ {`
	`283`	`+ this.WriteWarning("Cannot parse settings. Will abort the invocation.");`
	`284`	`+ stopProcessing = true;`
	`285`	`+ return;`
`282`	`286`	`}`
`283`	`287`	`}`
`284`	`288`