Merge pull request #317 from TouchInstinct/ubrir_linters_fix

Ubrir linters fix

gradle/plugins/src/main/java/static_analysis/linters/AndroidLinter.kt

@@ -14,23 +14,30 @@ class AndroidLinter : Linter {
 
     override val name: String = "Android lint"
 
-    override fun getErrors(project: Project): List<StaticAnalysisError> = xmlParser(project.getLintReportFile())
-            .typedChildren()
-            .filter { it.name() == "issue" && (it.attribute("severity") as String) == "Error" }
-            .map { errorNode ->
-                errorNode
-                        .typedChildren()
-                        .filter { it.name() == "location" }
-                        .map { locationNode ->
-                            AndroidLintError(
-                                    filePath = locationNode.attribute("file") as String,
-                                    fileLine = locationNode.attribute("line") as String?,
-                                    errorId = errorNode.attribute("id") as String,
-                                    description = errorNode.attribute("message") as String
-                            )
-                        }
-            }
-            .flatten()
+    override fun getErrors(project: Project): List<StaticAnalysisError> {
+        val lintRepostFile = project.getLintReportFile()
+        return if (lintRepostFile.exists()) {
+            xmlParser(lintRepostFile)
+                    .typedChildren()
+                    .filter { it.name() == "issue" && (it.attribute("severity") as String) == "Error" }
+                    .map { errorNode ->
+                        errorNode
+                                .typedChildren()
+                                .filter { it.name() == "location" }
+                                .map { locationNode ->
+                                    AndroidLintError(
+                                            filePath = locationNode.attribute("file") as String,
+                                            fileLine = locationNode.attribute("line") as String?,
+                                            errorId = errorNode.attribute("id") as String,
+                                            description = errorNode.attribute("message") as String
+                                    )
+                                }
+                    }
+                    .flatten()
+        } else {
+            emptyList()
+        }
+    }
 
     override fun setupForProject(project: Project, extension: StaticAnalysisExtension) {
         project.beforeEvaluate {
@@ -62,7 +69,11 @@ class AndroidLinter : Linter {
                 .mapNotNull { subproject: Project ->
                     subproject
                             .tasks
-                            .find { task -> task.name.contains(buildType, ignoreCase = true) && task.name.contains("lint") }
+                            .find { task ->
+                                task.name.contains(buildType, ignoreCase = true)
+                                        && task.name.contains("lint")
+                                        && !task.name.contains("lintVital")
+                            }
                             ?.path
                 }
     }

gradle/plugins/src/main/java/static_analysis/linters/CpdLinter.kt

@@ -39,6 +39,7 @@ class CpdLinter : Linter {
                 isSkipLexicalErrors = true
                 language = "kotlin"
                 minimumTokenCount = 60
+                encoding = "UTF-8"
             }
             tasks.withType<Cpd> {
                 reports.xml.required.set(true)

gradle/plugins/src/main/java/static_analysis/plugins/StaticAnalysisAndroidPlugin.kt

@@ -32,9 +32,9 @@ class StaticAnalysisAndroidPlugin : StaticAnalysisPlugin() {
         }
     }
 
+    //TODO: return CpdLinter after finding better way to disable it
     override fun createLinters(): List<Linter> = listOf(
             DetektLinter(),
-            CpdLinter(),
             AndroidLinter()
     )