Fix rules

pmd-javascript/src/main/resources/category/ecmascript/codestyle.xml

@@ -190,7 +190,7 @@ See also: <http://eslint.org/docs/rules/no-else-return>
             <property name="xpath">
                 <value>
 <![CDATA[
-//IfStatement[@Else="true"][Scope[1]/ReturnStatement]
+//IfStatement[@Else=true()][Scope[1]/ReturnStatement]
 ]]>
                 </value>
             </property>

pmd-javascript/src/main/resources/category/ecmascript/errorprone.xml

@@ -62,11 +62,10 @@ same type. The === operator avoids the casting.
             <property name="xpath">
                 <value>
 <![CDATA[
-//InfixExpression[(@Image = "==" or @Image = "!=")
+//InfixExpression[
+  (@Image = "==" or @Image = "!=")
   and
- (child::KeywordLiteral[@Image = true() or @Image = false()]
- or
- child::NumberLiteral)
+  (KeywordLiteral[@Image='true' or @Image = 'false'] or NumberLiteral)
 ]
 ]]>
                 </value>
@@ -110,7 +109,7 @@ precision in a floating point number.  This may result in numeric calculations b
             <property name="xpath">
                 <value>
 <![CDATA[
-//NumberLiteral[@NormalizedImage != @Number]
+//NumberLiteral[@NormalizedImage != string(@Number)]
 ]]>
                 </value>
             </property>