From 0a09760aec86825f9746ed9e1e3e003b6f51862f Mon Sep 17 00:00:00 2001
From: Will Stranathan <will@thestranathans.com>
Date: Fri, 5 Jun 2015 22:03:13 -0400
Subject: [PATCH] Fixed the Xpath expression to be more accurate and to find
 the getLogger statement

Former-commit-id: 2ecf96fe54d6f417f1fbf4b51092b73ed0d8a015
---
 src/main/config/dcrules.xml | 28 +++++++++++++++-------------
 1 file changed, 15 insertions(+), 13 deletions(-)

diff --git a/src/main/config/dcrules.xml b/src/main/config/dcrules.xml
index b66e6a29b..29de5cf0c 100644
--- a/src/main/config/dcrules.xml
+++ b/src/main/config/dcrules.xml
@@ -37,19 +37,21 @@ the work on the Logger.
         <properties>
             <property name="xpath">
                 <value><![CDATA[
-//ClassOrInterfaceDeclaration[.//PrimaryExpression[PrimaryPrefix/Name[@Image = 'Logger.getLogger']]
-and not(@Image = .//PrimaryExpression[PrimaryPrefix/Name[@Image = 'Logger.getLogger']]
-  /PrimarySuffix
-  /Arguments
-  /ArgumentList
-  /Expression
-  /PrimaryExpression
-  /PrimaryPrefix
-  /ResultType
-  /Type
-  /ReferenceType
-  /ClassOrInterfaceType/@Image
-)]        ]]></value>
+//PrimaryExpression[
+  PrimaryPrefix/Name/@Image = 'Logger.getLogger'
+  and
+  not(PrimarySuffix/
+    Arguments/
+    ArgumentList/
+    Expression/
+    PrimaryExpression/
+    PrimaryPrefix/
+    ResultType/
+    Type/
+    ReferenceType/
+    ClassOrInterfaceType/
+    @Image = ancestor::ClassOrInterfaceDeclaration/@Image)
+]               ]]></value>
             </property>
         </properties>
         <example><![CDATA[