Ignore open brace in a one line if else statement

Author: Kapil Borle

Rules/PlaceOpenBrace.cs

@@ -45,6 +45,9 @@ public class PlaceOpenBrace : ConfigurableRule
         [ConfigurableRuleProperty(defaultValue: true)]
         public bool NewLineAfter { get; protected set; }
 
+        [ConfigurableRuleProperty(defaultValue: true)]
+        public bool IgnoreOneLineIf { get; protected set; }
+
         private List<Func<Token[], Ast, string, IEnumerable<DiagnosticRecord>>> violationFinders
             = new List<Func<Token[], Ast, string, IEnumerable<DiagnosticRecord>>>();
 
@@ -96,12 +99,21 @@ public override IEnumerable<DiagnosticRecord> AnalyzeScript(Ast ast, string file
             var tokens = Helper.Instance.Tokens;
 
             // Ignore open braces that are part of arguments to a command
-            // * E.g. get-process | % { "blah }
+            // * E.g. get-process | % { "blah }f
             // In the above case even if OnSameLine == false, we should not
             // flag the open brace as it would move the brace to the next line
             // and will invalidate the command
-            tokensToIgnore = new HashSet<Token>(
-                new TokenOperations(tokens, ast).GetOpenBracesInCommandElements());
+            var tokenOps = new TokenOperations(tokens, ast);
+            tokensToIgnore = new HashSet<Token>(tokenOps.GetOpenBracesInCommandElements());
+
+            if (IgnoreOneLineIf)
+            {
+                foreach (var openBraceToken in tokenOps.GetOpenBraceInOneLineIfStatement())
+                {
+                    tokensToIgnore.Add(openBraceToken);
+                }
+            }
+
             foreach (var violationFinder in violationFinders)
             {
                 diagnosticRecords.AddRange(violationFinder(tokens, ast, fileName));