From 83a0a343d718c58d432bb218e412527de049070f Mon Sep 17 00:00:00 2001 From: Amaury Levé Date: Tue, 26 Jun 2018 14:03:10 +0200 Subject: Rule S4662: at-rules should be valid --- .../main/java/org/sonar/css/plugin/CssRules.java | 2 ++ .../sonar/css/plugin/rules/AtRuleNoUnknown.java | 31 ++++++++++++++++++++++ .../org/sonar/l10n/css/rules/css/S4662.html | 11 ++++++++ .../org/sonar/l10n/css/rules/css/S4662.json | 16 +++++++++++ .../l10n/css/rules/css/Sonar_way_profile.json | 1 + 5 files changed, 61 insertions(+) create mode 100644 sonar-css-plugin/src/main/java/org/sonar/css/plugin/rules/AtRuleNoUnknown.java create mode 100644 sonar-css-plugin/src/main/resources/org/sonar/l10n/css/rules/css/S4662.html create mode 100644 sonar-css-plugin/src/main/resources/org/sonar/l10n/css/rules/css/S4662.json (limited to 'sonar-css-plugin/src') diff --git a/sonar-css-plugin/src/main/java/org/sonar/css/plugin/CssRules.java b/sonar-css-plugin/src/main/java/org/sonar/css/plugin/CssRules.java index c463622..5248ff0 100644 --- a/sonar-css-plugin/src/main/java/org/sonar/css/plugin/CssRules.java +++ b/sonar-css-plugin/src/main/java/org/sonar/css/plugin/CssRules.java @@ -29,6 +29,7 @@ import javax.annotation.Nullable; import org.sonar.api.batch.rule.CheckFactory; import org.sonar.api.batch.rule.Checks; import org.sonar.api.rule.RuleKey; +import org.sonar.css.plugin.rules.AtRuleNoUnknown; import org.sonar.css.plugin.rules.BlockNoEmpty; import org.sonar.css.plugin.rules.ColorNoInvalidHex; import org.sonar.css.plugin.rules.CommentNoEmpty; @@ -67,6 +68,7 @@ public class CssRules { public static List getRuleClasses() { return Collections.unmodifiableList(Arrays.asList( + AtRuleNoUnknown.class, BlockNoEmpty.class, ColorNoInvalidHex.class, CommentNoEmpty.class, diff --git a/sonar-css-plugin/src/main/java/org/sonar/css/plugin/rules/AtRuleNoUnknown.java b/sonar-css-plugin/src/main/java/org/sonar/css/plugin/rules/AtRuleNoUnknown.java new file mode 100644 index 0000000..3a2a821 --- /dev/null +++ b/sonar-css-plugin/src/main/java/org/sonar/css/plugin/rules/AtRuleNoUnknown.java @@ -0,0 +1,31 @@ +/* + * SonarCSS + * Copyright (C) 2018-2018 SonarSource SA + * mailto:info AT sonarsource DOT com + * + * This program is free software; you can redistribute it and/or + * modify it under the terms of the GNU Lesser General Public + * License as published by the Free Software Foundation; either + * version 3 of the License, or (at your option) any later version. + * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU + * Lesser General Public License for more details. + * + * You should have received a copy of the GNU Lesser General Public License + * along with this program; if not, write to the Free Software Foundation, + * Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA. + */ +package org.sonar.css.plugin.rules; + +import org.sonar.check.Rule; + +@Rule(key = "S4662") +public class AtRuleNoUnknown implements CssRule { + + @Override + public String stylelintKey() { + return "at-rule-no-unknown"; + } +} diff --git a/sonar-css-plugin/src/main/resources/org/sonar/l10n/css/rules/css/S4662.html b/sonar-css-plugin/src/main/resources/org/sonar/l10n/css/rules/css/S4662.html new file mode 100644 index 0000000..96e4255 --- /dev/null +++ b/sonar-css-plugin/src/main/resources/org/sonar/l10n/css/rules/css/S4662.html @@ -0,0 +1,11 @@ +

The W3C specifications define the valid at-rules. Only the official and browser-specific at-rules should be used to get +the expected impact in the final rendering.

+

Noncompliant Code Example

+
+@unknown {}
+
+

Compliant Solution

+
+@media (max-width: 960px) {}
+
+ diff --git a/sonar-css-plugin/src/main/resources/org/sonar/l10n/css/rules/css/S4662.json b/sonar-css-plugin/src/main/resources/org/sonar/l10n/css/rules/css/S4662.json new file mode 100644 index 0000000..d2cfb03 --- /dev/null +++ b/sonar-css-plugin/src/main/resources/org/sonar/l10n/css/rules/css/S4662.json @@ -0,0 +1,16 @@ +{ + "title": "\"at-rules\" should be valid", + "type": "BUG", + "status": "ready", + "remediation": { + "func": "Constant\/Issue", + "constantCost": "1min" + }, + "tags": [ + + ], + "defaultSeverity": "Major", + "ruleSpecification": "RSPEC-4662", + "sqKey": "S4662", + "scope": "Main" +} diff --git a/sonar-css-plugin/src/main/resources/org/sonar/l10n/css/rules/css/Sonar_way_profile.json b/sonar-css-plugin/src/main/resources/org/sonar/l10n/css/rules/css/Sonar_way_profile.json index 7e9fe85..73fa72d 100644 --- a/sonar-css-plugin/src/main/resources/org/sonar/l10n/css/rules/css/Sonar_way_profile.json +++ b/sonar-css-plugin/src/main/resources/org/sonar/l10n/css/rules/css/Sonar_way_profile.json @@ -15,6 +15,7 @@ "S4658", "S4659", "S4660", + "S4662", "S4663", "S4666", "S4667", -- cgit v1.2.3