aboutsummaryrefslogtreecommitdiffstats
path: root/sonar-css-plugin/src/main/java/org/sonar
diff options
context:
space:
mode:
authorElena Vilchik2018-06-27 15:22:56 +0200
committerGitHub2018-06-27 15:22:56 +0200
commitcea9681a811f5bced88e443d5fda14ba15396343 (patch)
tree3746b822ead44cf60de9336e6454770710c76960 /sonar-css-plugin/src/main/java/org/sonar
parentbde3e646ffe1b1ba1b513cac042f5c1e379da4ff (diff)
downloadsonar-css-cea9681a811f5bced88e443d5fda14ba15396343.tar.bz2
Exclude minified files (#93)
Diffstat (limited to 'sonar-css-plugin/src/main/java/org/sonar')
-rw-r--r--sonar-css-plugin/src/main/java/org/sonar/css/plugin/CssPlugin.java1
-rw-r--r--sonar-css-plugin/src/main/java/org/sonar/css/plugin/MinifiedFilesFilter.java62
2 files changed, 63 insertions, 0 deletions
diff --git a/sonar-css-plugin/src/main/java/org/sonar/css/plugin/CssPlugin.java b/sonar-css-plugin/src/main/java/org/sonar/css/plugin/CssPlugin.java
index cd98f25..5300947 100644
--- a/sonar-css-plugin/src/main/java/org/sonar/css/plugin/CssPlugin.java
+++ b/sonar-css-plugin/src/main/java/org/sonar/css/plugin/CssPlugin.java
@@ -53,6 +53,7 @@ public class CssPlugin implements Plugin {
CssRuleSensor.class,
StylelintCommandProvider.class,
StylelintReportSensor.class,
+ MinifiedFilesFilter.class,
PropertyDefinition.builder(FILE_SUFFIXES_KEY)
.defaultValue(FILE_SUFFIXES_DEFVALUE)
diff --git a/sonar-css-plugin/src/main/java/org/sonar/css/plugin/MinifiedFilesFilter.java b/sonar-css-plugin/src/main/java/org/sonar/css/plugin/MinifiedFilesFilter.java
new file mode 100644
index 0000000..87dff1a
--- /dev/null
+++ b/sonar-css-plugin/src/main/java/org/sonar/css/plugin/MinifiedFilesFilter.java
@@ -0,0 +1,62 @@
+/*
+ * 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;
+
+import java.io.IOException;
+import org.sonar.api.batch.fs.InputFile;
+import org.sonar.api.batch.fs.InputFileFilter;
+import org.sonar.api.utils.log.Logger;
+import org.sonar.api.utils.log.Loggers;
+
+public class MinifiedFilesFilter implements InputFileFilter {
+
+ private static final int AVERAGE_LINE_LENGTH_THRESHOLD = 200;
+
+ private static final Logger LOG = Loggers.get(MinifiedFilesFilter.class);
+
+ @Override
+ public boolean accept(InputFile file) {
+ if (!CssLanguage.KEY.equals(file.language())) {
+ return true;
+ }
+
+ try {
+ boolean isMinified = hasMinifiedFileName(file) || hasExcessiveAverageLineLength(file);
+ if (isMinified) {
+ LOG.debug("File [" + file.uri() + "] looks like a minified file and will not be analyzed");
+ }
+ return !isMinified;
+
+ } catch (IOException e) {
+ throw new IllegalStateException("Failed to read input file", e);
+ }
+ }
+
+ private static boolean hasMinifiedFileName(InputFile file) {
+ String fileName = file.filename();
+ return fileName.endsWith("-min.css") || fileName.endsWith(".min.css");
+ }
+
+ private static boolean hasExcessiveAverageLineLength(InputFile file) throws IOException {
+ int averageLineLength = file.contents().length() / file.lines();
+ return averageLineLength > AVERAGE_LINE_LENGTH_THRESHOLD;
+ }
+
+}