Counting positional parameters fix.

Author: kalgiz

Engine/Helper.cs

@@ -629,16 +629,21 @@ public bool PositionalParameterUsed(CommandAst cmdAst, bool moreThanThreePositio
                 return false;
             }
 
-            int parameters = 0;
             // Because of the way we count, we will also count the cmdlet as an argument so we have to -1
-            int arguments = -1;
+            int argumentsWithoutParameters = -1;
+            bool wasParameter = false;
 
             foreach (CommandElementAst ceAst in cmdAst.CommandElements)
             {
                 var cmdParamAst = ceAst as CommandParameterAst;
-                if (cmdParamAst == null)
+                if (cmdParamAst != null)
                 {
-                    arguments += 1;
+                    wasParameter = true;
+                } else {
+                    if (!wasParameter) {
+                        argumentsWithoutParameters += 1;
+                    }
+                    wasParameter = false;
                 }
             }
 
@@ -647,16 +652,10 @@ public bool PositionalParameterUsed(CommandAst cmdAst, bool moreThanThreePositio
 
             if (parent != null && parent.PipelineElements.Count > 1 && parent.PipelineElements[0] != cmdAst)
             {
-                arguments += 1;
-            }
-
-            // if we are only checking for 3 or more positional parameters, check that arguments < parameters + 3
-            if (moreThanThreePositional && (arguments - parameters) < 3)
-            {
-                return false;
+                argumentsWithoutParameters += 1;
             }
 
-            return arguments > parameters;
+            return moreThanThreePositional ? argumentsWithoutParameters >= 3 : argumentsWithoutParameters > 0;
         }
 
 

Rules/AvoidPositionalParameters.cs

@@ -52,7 +52,7 @@ public IEnumerable<DiagnosticRecord> AnalyzeScript(Ast ast, string fileName)
                 if (cmdAst.GetCommandName() == null) continue;
 
                 if ((Helper.Instance.IsCmdlet(cmdAst) || declaredFunctionNames.Contains(cmdAst.GetCommandName())) &&
-                    (Helper.Instance.PositionalParameterUsed(cmdAst)))
+                    (Helper.Instance.PositionalParameterUsed(cmdAst, true)))
                 {
                     PipelineAst parent = cmdAst.Parent as PipelineAst;