| | | 1 | | using System.Text.RegularExpressions; |
| | | 2 | | |
| | | 3 | | namespace AspxLint.Core.Rules; |
| | | 4 | | |
| | | 5 | | public sealed class Attr002MixedQuotes : IRule |
| | | 6 | | { |
| | 207 | 7 | | public string Id => "ATTR-002"; |
| | 28 | 8 | | public string Name => "Melange guillemets simples / doubles"; |
| | 25 | 9 | | public Severity Severity => Severity.Info; |
| | | 10 | | public string Description => |
| | 25 | 11 | | "Pour la coherence du code, utilisez systematiquement les guillemets doubles (\") pour les attributs HTML/ASP.NE |
| | 46 | 12 | | public bool HasFix => true; |
| | | 13 | | |
| | 5 | 14 | | private static readonly Regex SingleQuotedAttr = |
| | 5 | 15 | | new(@"\s([a-zA-Z][a-zA-Z0-9\-:_]*)='([^']*)'", RegexOptions.Compiled); |
| | | 16 | | |
| | 5 | 17 | | private static readonly Regex AspBlockOnLine = |
| | 5 | 18 | | new(@"<%[\s\S]*?%>", RegexOptions.Compiled); |
| | | 19 | | |
| | 5 | 20 | | private static readonly Regex TagWithAttrs = |
| | 5 | 21 | | new(@"(<[a-zA-Z][a-zA-Z0-9:_\-]*\b[^>]*?)>", RegexOptions.Compiled); |
| | | 22 | | |
| | | 23 | | public IEnumerable<Issue> Detect(string content, string[] lines, RuleContext ctx) |
| | | 24 | | { |
| | | 25 | | for (int i = 0; i < lines.Length; i++) |
| | | 26 | | { |
| | | 27 | | var line = lines[i]; |
| | | 28 | | // Si la ligne contient du code serveur, c'est trop ambigu (les ' peuvent etre du C#) — on saute. |
| | | 29 | | if (AspBlockOnLine.IsMatch(line)) continue; |
| | | 30 | | |
| | | 31 | | foreach (Match m in SingleQuotedAttr.Matches(line)) |
| | | 32 | | { |
| | | 33 | | var before = line[..m.Index]; |
| | | 34 | | var lastOpen = before.LastIndexOf('<'); |
| | | 35 | | var lastClose = before.LastIndexOf('>'); |
| | | 36 | | if (lastOpen <= lastClose) continue; |
| | | 37 | | |
| | | 38 | | yield return new Issue(Id, Name, Severity, |
| | | 39 | | i + 1, m.Index + 1, m.Value.Trim(), |
| | | 40 | | $"Preferer \"{m.Groups[1].Value}=\\\"{m.Groups[2].Value}\\\"\""); |
| | | 41 | | } |
| | | 42 | | } |
| | | 43 | | } |
| | | 44 | | |
| | | 45 | | public string? Fix(string content, RuleContext ctx) => |
| | 21 | 46 | | TagWithAttrs.Replace(content, m => |
| | 21 | 47 | | { |
| | 21 | 48 | | var body = m.Groups[1].Value; |
| | 21 | 49 | | var fixedBody = SingleQuotedAttr.Replace(body, mm => |
| | 21 | 50 | | $" {mm.Groups[1].Value}=\"{mm.Groups[2].Value.Replace("\"", """)}\""); |
| | 21 | 51 | | return fixedBody + ">"; |
| | 21 | 52 | | }); |
| | | 53 | | } |