Don't raise PSAvoidUninitalizedVariables for parameters outside a param block

GoodOlClint · GoodOlClint · commit fdd11985bef3 · 2015-05-17T17:07:43.000-05:00
diff --git a/Rules/AvoidUninitializedVariable.cs b/Rules/AvoidUninitializedVariable.cs
@@ -70,6 +70,12 @@ public IEnumerable<DiagnosticRecord> AnalyzeScript(Ast ast, string fileName)
                 {
                     paramVariables.UnionWith(funcAst.Body.ParamBlock.Parameters.Select(paramAst => paramAst.Name.VariablePath.UserPath));
                 }
+                
+                //don't raise the rules for parameters outside the param block
+                if(funcAst.Parameters != null)
+                {
+                    paramVariables.UnionWith(funcAst.Parameters.Select(paramAst => paramAst.Name.VariablePath.UserPath));
+                }
 
                 // Iterates all VariableExpressionAst and check the command name.
                 foreach (VariableExpressionAst varAst in varAsts)

Original file line number	Diff line number	Diff line change
`@@ -70,6 +70,12 @@ public IEnumerable<DiagnosticRecord> AnalyzeScript(Ast ast, string fileName)`
`70`	`70`	`{`
`71`	`71`	`paramVariables.UnionWith(funcAst.Body.ParamBlock.Parameters.Select(paramAst => paramAst.Name.VariablePath.UserPath));`
`72`	`72`	`}`
	`73`	`+`
	`74`	`+ //don't raise the rules for parameters outside the param block`
	`75`	`+ if(funcAst.Parameters != null)`
	`76`	`+ {`
	`77`	`+ paramVariables.UnionWith(funcAst.Parameters.Select(paramAst => paramAst.Name.VariablePath.UserPath));`
	`78`	`+ }`
`73`	`79`
`74`	`80`	`// Iterates all VariableExpressionAst and check the command name.`
`75`	`81`	`foreach (VariableExpressionAst varAst in varAsts)`