From e1b5717675646779553cebdfa510dbca17e2a30f Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Cl=C3=A9ment=20Fournier?= <clement.fournier76@gmail.com>
Date: Fri, 2 Nov 2018 09:09:13 +0100
Subject: [PATCH] Fix #1372

---
 .../resources/category/java/codestyle.xml     |  6 +++--
 .../codestyle/xml/UselessQualifiedThis.xml    | 23 +++++++++++++++++++
 2 files changed, 27 insertions(+), 2 deletions(-)

diff --git a/pmd-java/src/main/resources/category/java/codestyle.xml b/pmd-java/src/main/resources/category/java/codestyle.xml
index 3329ce7284..483d2e3a05 100644
--- a/pmd-java/src/main/resources/category/java/codestyle.xml
+++ b/pmd-java/src/main/resources/category/java/codestyle.xml
@@ -1950,7 +1950,9 @@ public class Foo {
           message="Useless qualified this usage in the same class."
           class="net.sourceforge.pmd.lang.rule.XPathRule"
           externalInfoUrl="${pmd.website.baseurl}/pmd_rules_java_codestyle.html#uselessqualifiedthis">
-        <description>Look for qualified this usages in the same class.</description>
+        <description>
+            Reports qualified this usages in the same class.
+        </description>
         <priority>3</priority>
         <properties>
             <property name="xpath">
@@ -1958,7 +1960,7 @@ public class Foo {
 <![CDATA[
 //PrimaryExpression
 [PrimaryPrefix/Name[@Image]]
-[PrimarySuffix[@Arguments='false']]
+[PrimarySuffix[@Arguments='false' and @ArrayDereference = 'false']]
 [not(PrimarySuffix/MemberSelector)]
 [ancestor::ClassOrInterfaceBodyDeclaration[1][@AnonymousInnerClass='false']]
 /PrimaryPrefix/Name[@Image = ancestor::ClassOrInterfaceDeclaration[1]/@Image]
diff --git a/pmd-java/src/test/resources/net/sourceforge/pmd/lang/java/rule/codestyle/xml/UselessQualifiedThis.xml b/pmd-java/src/test/resources/net/sourceforge/pmd/lang/java/rule/codestyle/xml/UselessQualifiedThis.xml
index 2c8ba82356..f9044564af 100644
--- a/pmd-java/src/test/resources/net/sourceforge/pmd/lang/java/rule/codestyle/xml/UselessQualifiedThis.xml
+++ b/pmd-java/src/test/resources/net/sourceforge/pmd/lang/java/rule/codestyle/xml/UselessQualifiedThis.xml
@@ -152,4 +152,27 @@ public class ApiExceptionCtrlAdvice {
 }
         ]]></code>
     </test-code>
+
+    <test-code>
+        <description>False positive for UselessQualifiedThis #1372</description>
+        <expected-problems>0</expected-problems>
+        <code>
+            <![CDATA[
+            public class Board {
+                private int[] Board;
+
+
+                public Board(int[] b) {
+                    for (int i = 0; i < b.length; i++)
+                        Board[i] = b[i];
+                }
+
+
+                public int size() {
+                    return Board.length;
+                }
+            }
+            ]]>
+        </code>
+    </test-code>
 </test-data>
\ No newline at end of file