Skip to content

Commit 667aea5

Browse files
committed
Fix PSUseDeclaredVarsMoreThanAssignments when variable is assigned more than once to still give a warning.
Issue 833: #833
1 parent 2056fe0 commit 667aea5

File tree

1 file changed

+11
-1
lines changed

1 file changed

+11
-1
lines changed

Rules/UseDeclaredVarsMoreThanAssignments.cs

Lines changed: 11 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -166,7 +166,17 @@ private IEnumerable<DiagnosticRecord> AnalyzeScriptBlockAst(ScriptBlockAst scrip
166166
// Checks if this variableAst is part of the logged assignment
167167
foreach (VariableExpressionAst varInAssignment in varsInAssignment)
168168
{
169-
inAssignment |= varInAssignment.Equals(varAst);
169+
// Try casting to AssignmentStatementAst to be able to catch case where a variable is assigned more than once (https://github.com/PowerShell/PSScriptAnalyzer/issues/833)
170+
var varInAssignmentAsStatementAst = varInAssignment.Parent as AssignmentStatementAst;
171+
var varAstAsAssignmentStatementAst = varAst.Parent as AssignmentStatementAst;
172+
if (varInAssignmentAsStatementAst != null && varAstAsAssignmentStatementAst != null)
173+
{
174+
inAssignment = varInAssignmentAsStatementAst.Left.Extent.Text.Equals(varAstAsAssignmentStatementAst.Left.Extent.Text, StringComparison.OrdinalIgnoreCase);
175+
}
176+
else
177+
{
178+
inAssignment = varInAssignment.Equals(varAst);
179+
}
170180
}
171181

172182
if (!inAssignment)

0 commit comments

Comments
 (0)