From 4d8a8a091949bb22bae81b7f60644e1a9928f80d Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=E6=98=B6=E4=B9=90?= <changle.lq@alibaba-inc.com>
Date: Tue, 6 Nov 2018 19:50:32 +0800
Subject: [PATCH] =?UTF-8?q?=E6=B3=A8=E8=A7=A3=E4=B8=8D=E8=BF=9B=E8=A1=8C?=
 =?UTF-8?q?=E5=B0=8F=E5=86=99=E9=A9=BC=E5=B3=B0=E5=91=BD=E5=90=8D=E6=A0=A1?=
 =?UTF-8?q?=E9=AA=8C?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 .../LowerCamelCaseVariableNamingRule.java     |  6 +++
 .../xml/LowerCamelCaseVariableNamingRule.xml  | 54 +++++++++++++++++++
 2 files changed, 60 insertions(+)

diff --git a/p3c-pmd/src/main/java/com/alibaba/p3c/pmd/lang/java/rule/naming/LowerCamelCaseVariableNamingRule.java b/p3c-pmd/src/main/java/com/alibaba/p3c/pmd/lang/java/rule/naming/LowerCamelCaseVariableNamingRule.java
index bb414af..1ae3c0e 100644
--- a/p3c-pmd/src/main/java/com/alibaba/p3c/pmd/lang/java/rule/naming/LowerCamelCaseVariableNamingRule.java
+++ b/p3c-pmd/src/main/java/com/alibaba/p3c/pmd/lang/java/rule/naming/LowerCamelCaseVariableNamingRule.java
@@ -73,4 +73,10 @@ public class LowerCamelCaseVariableNamingRule extends AbstractAliRule {
         }
         return super.visit(node, data);
     }
+
+    @Override
+    public Object visit(ASTAnnotationTypeDeclaration node, Object data) {
+        //对所有注解内的内容不做检查
+        return null;
+    }
 }
diff --git a/p3c-pmd/src/test/resources/com/alibaba/p3c/pmd/lang/java/rule/naming/xml/LowerCamelCaseVariableNamingRule.xml b/p3c-pmd/src/test/resources/com/alibaba/p3c/pmd/lang/java/rule/naming/xml/LowerCamelCaseVariableNamingRule.xml
index a010529..effc1b6 100644
--- a/p3c-pmd/src/test/resources/com/alibaba/p3c/pmd/lang/java/rule/naming/xml/LowerCamelCaseVariableNamingRule.xml
+++ b/p3c-pmd/src/test/resources/com/alibaba/p3c/pmd/lang/java/rule/naming/xml/LowerCamelCaseVariableNamingRule.xml
@@ -140,6 +140,60 @@ public interface BizConstants {
         <expected-problems>0</expected-problems>
         <code-ref id="LowerCamelCaseVariableNamingRuleTest8" />
     </test-code>
+
+    <code-fragment id="LowerCamelCaseVariableNamingRuleTest8">
+        <![CDATA[
+        public @interface TYPE {
+            int DO_NO_THING = 0;
+            int DO_ONE_START_TO_END_ROUTE = 1;
+            int DO_ONE_CAR_TO_START_ROUTE = 2;
+            int DO_TWO_ROUTE = 3;
+        }
+    ]]>
+    </code-fragment>
+    <test-code>
+        <description>Variable name should be lowerCamelCase8</description>
+        <expected-problems>0</expected-problems>
+        <code-ref id="LowerCamelCaseVariableNamingRuleTest8" />
+    </test-code>
+
+    <code-fragment id="LowerCamelCaseVariableNamingRuleTest9">
+        <![CDATA[
+        @Document
+        @Retention(RetentionPolicy.RUNTIME)
+        @Target(ElementType.ANNOTATION_TYPE)
+        public @interface TYPE {
+            int DO_NO_THING = 0;
+            int DO_ONE_START_TO_END_ROUTE = 1;
+            int DO_ONE_CAR_TO_START_ROUTE = 2;
+            int DO_TWO_ROUTE = 3;
+        }
+    ]]>
+    </code-fragment>
+    <test-code>
+        <description>Variable name should be lowerCamelCase9</description>
+        <expected-problems>0</expected-problems>
+        <code-ref id="LowerCamelCaseVariableNamingRuleTest9" />
+    </test-code>
+
+    <code-fragment id="LowerCamelCaseVariableNamingRuleTest10">
+        <![CDATA[
+        @Document
+        @Retention(RetentionPolicy.RUNTIME)
+        @Target(ElementType.ANNOTATION_TYPE)
+        public class Type {
+            int DO_NO_THING = 0;
+            int DO_ONE_START_TO_END_ROUTE = 1;
+            int DO_ONE_CAR_TO_START_ROUTE = 2;
+            int DO_TWO_ROUTE = 3;
+        }
+    ]]>
+    </code-fragment>
+    <test-code>
+        <description>Variable name should be lowerCamelCase10</description>
+        <expected-problems>4</expected-problems>
+        <code-ref id="LowerCamelCaseVariableNamingRuleTest10" />
+    </test-code>
 </test-data>