FrigaT/SQLLint
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases 12 Wiki Activity

Поправлены правила. Исправлен параметр формирования деталировки
All checks were successful
CI / build-test (push) Successful in 31s
Details
Release / pack-and-publish (release) Successful in 34s
Details

Browse Source
This commit is contained in:
FrigaT
2025-12-29 02:19:30 +03:00
parent 507c466b5d
commit edc4d5c087
4 changed files with 6 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
SQLLinter.CLI/Program.cs
View File

@@ -45,7 +45,7 @@ namespace SQLLinter.CLI
["SetVariable"] = Common.RuleViolationSeverity.Critical, ["SetVariable"] = Common.RuleViolationSeverity.Critical,
["CreateProcedureInDbo"] = Common.RuleViolationSeverity.Warning, ["CreateProcedureInDbo"] = Common.RuleViolationSeverity.Warning,
}, },
GenerateDetails = true, GenerateDetails = false,
}; };
//var linter = new Linter(con, rep); //var linter = new Linter(con, rep);
@@ -63,7 +63,7 @@ namespace SQLLinter.CLI
Dictionary<string, Stream> files = new(); Dictionary<string, Stream> files = new();
for (int i = 0; i < 15; i++) for (int i = 0; i < 5; i++)
{ {
files[name + i + ".sql"] = reader.BaseStream; files[name + i + ".sql"] = reader.BaseStream;
} }

2
SQLLinter/Infrastructure/Parser/SqlRuleVisitor.cs
View File

@@ -39,7 +39,7 @@ public class SqlRuleVisitor : IRuleVisitor
if (sqlFragment == null) return; if (sqlFragment == null) return;
//Dictionary<TSqlFragment, TSqlFragment?>? parentMap = generateDetails ? ParentMapBuilder.Build(sqlFragment) : null; //Dictionary<TSqlFragment, TSqlFragment?>? parentMap = generateDetails ? ParentMapBuilder.Build(sqlFragment) : null;
Dictionary<TSqlFragment, TSqlFragment?>? parentMap = new Dictionary<TSqlFragment, TSqlFragment?>(); Dictionary<TSqlFragment, TSqlFragment?>? parentMap = generateDetails ? new Dictionary<TSqlFragment, TSqlFragment?>() : null;
var ruleExceptions = ignoredRules as IRuleException[] ?? ignoredRules.ToArray(); var ruleExceptions = ignoredRules as IRuleException[] ?? ignoredRules.ToArray();
if (errors.Any()) if (errors.Any())

4
SQLLinter/Infrastructure/Rules/ConditionalBeginEndRule.cs
View File

@@ -17,7 +17,7 @@ public class ConditionalBeginEndRule : BaseRuleVisitor, IRule
public override void Visit(IfStatement node) public override void Visit(IfStatement node)
{ {
if (node.ThenStatement != null && node.ThenStatement is not BeginEndBlockStatement) if (node.ThenStatement != null && node.ThenStatement is not BeginEndBlockStatement && node.ThenStatement is not TryCatchStatement)
{ {
if (node.ThenStatement.StartLine != node.StartLine || node.ScriptTokenStream.Where(t => t.Offset <= node.ThenStatement.StartOffset + node.ThenStatement.FragmentLength).Max(t => t.Line) != node.StartLine) if (node.ThenStatement.StartLine != node.StartLine || node.ScriptTokenStream.Where(t => t.Offset <= node.ThenStatement.StartOffset + node.ThenStatement.FragmentLength).Max(t => t.Line) != node.StartLine)
{ {
@@ -25,7 +25,7 @@ public class ConditionalBeginEndRule : BaseRuleVisitor, IRule
} }
} }
if (node.ElseStatement != null && node.ElseStatement is not BeginEndBlockStatement && node.ElseStatement is not IfStatement) if (node.ElseStatement != null && node.ElseStatement is not BeginEndBlockStatement && node.ElseStatement is not TryCatchStatement && node.ElseStatement is not IfStatement)
{ {
if (node.ElseStatement.StartLine != node.StartLine || node.ScriptTokenStream.Where(t => t.Offset <= node.ElseStatement.StartOffset + node.ElseStatement.FragmentLength).Max(t => t.Line) != node.StartLine) if (node.ElseStatement.StartLine != node.StartLine || node.ScriptTokenStream.Where(t => t.Offset <= node.ElseStatement.StartOffset + node.ElseStatement.FragmentLength).Max(t => t.Line) != node.StartLine)
{ {

2
SQLLinter/Infrastructure/Rules/DeleteWhereRule.cs
View File

@@ -20,6 +20,6 @@ public class DeleteWhereRule : BaseRuleVisitor, IRule
.Select(t => t.Text) .Select(t => t.Text)
); );
AddViolation(node, name); AddViolation(node.Target, name);
} }
} }