Merge pull request #2 from fartem/1.2.0

2021-04-03 Version 1.2.0: Updated checks

Author: fartem

README.md

@@ -42,7 +42,7 @@ Add to `TreeWalker` module:
 <module name="com.smlnskgmail.jaman.checkstyle.checks.AndroidViewFieldNameCheck">
     <property name="id" value="AndroidViewFieldNameCheck" />
 </module>
-<module name="com.smlnskgmail.jaman.checkstyle.checks.ContextFirstParameterCheck">
+<module name="com.smlnskgmail.jaman.checkstyle.checks.ContextAsFirstParameterCheck">
     <property name="id" value="ContextFirstParameterCheck" />
 </module>
 <module name="com.smlnskgmail.jaman.checkstyle.checks.MethodObjectReturnAnnotationCheck">

build.gradle

@@ -5,7 +5,7 @@ plugins {
 }
 
 group 'com.smlnskgmail.jaman.checkstyle'
-version '1.1.0'
+version '1.2.0'
 
 checkstyle {
     toolVersion '8.38'

src/main/java/com/smlnskgmail/jaman/checkstyle/checks/AndroidViewFieldNameCheck.java

@@ -14,6 +14,36 @@ public void visitToken(DetailAST ast) {
         if (identifier != null) {
             String fieldClassName = identifier.getText();
             switch (fieldClassName) {
+                case "LinearLayout":
+                    if (!ast.findFirstToken(TokenTypes.IDENT).getText().startsWith("lv")) {
+                        log(ast);
+                    }
+                    break;
+                case "RelativeLayout":
+                    if (!ast.findFirstToken(TokenTypes.IDENT).getText().startsWith("rl")) {
+                        log(ast);
+                    }
+                    break;
+                case "ConstraintLayout":
+                    if (!ast.findFirstToken(TokenTypes.IDENT).getText().startsWith("cl")) {
+                        log(ast);
+                    }
+                    break;
+                case "FrameLayout":
+                    if (!ast.findFirstToken(TokenTypes.IDENT).getText().startsWith("fl")) {
+                        log(ast);
+                    }
+                    break;
+                case "ScrollView":
+                    if (!ast.findFirstToken(TokenTypes.IDENT).getText().startsWith("sv")) {
+                        log(ast);
+                    }
+                    break;
+                case "HorizontalScrollView":
+                    if (!ast.findFirstToken(TokenTypes.IDENT).getText().startsWith("hsv")) {
+                        log(ast);
+                    }
+                    break;
                 case "TextView":
                     if (!ast.findFirstToken(TokenTypes.IDENT).getText().startsWith("tv")) {
                         log(ast);

src/main/java/com/smlnskgmail/jaman/checkstyle/checks/ContextFirstParameterCheck.java renamed to src/main/java/com/smlnskgmail/jaman/checkstyle/checks/ContextAsFirstParameterCheck.java

@@ -4,9 +4,9 @@
 import com.puppycrawl.tools.checkstyle.api.DetailAST;
 import com.puppycrawl.tools.checkstyle.api.TokenTypes;
 
-public class ContextFirstParameterCheck extends AbstractCheck {
+public class ContextAsFirstParameterCheck extends AbstractCheck {
 
-    private static final String MESSAGE_KEY = "ContextFirstParameterCheck";
+    private static final String MESSAGE_KEY = "ContextAsFirstParameterCheck";
 
     @Override
     public void visitToken(DetailAST ast) {