hiifong/pmd
1
0
Fork 0
forked from phoedos/pmd
Code Pull Requests Activity

Dead code removal

Browse Source
This commit is contained in:
Sergey
2017-01-16 11:35:49 -08:00
committed by Juan Martín Sotuyo Dodero
parent 549274f852
commit ba325eb8a2
1 changed files with 5 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
pmd-apex/src/main/java/net/sourceforge/pmd/lang/apex/rule/security/ApexOpenRedirectRule.java
View File

@ -101,9 +101,11 @@ public class ApexOpenRedirectRule extends AbstractApexRule {
} }
} else { } else {
if (node instanceof ASTField) { if (node instanceof ASTField) {
/* sergey.gorbaty: /*
* Apex Jorje parser is returning a null from Field.getFieldInfo(), but the info is available from an inner field. * sergey.gorbaty: Apex Jorje parser is returning a null from
* DO NOT attempt to optimize this block without checking that Jorje parser actually fixed its bug. * Field.getFieldInfo(), but the info is available from an inner
* field. DO NOT attempt to optimize this block without checking
* that Jorje parser actually fixed its bug.
* *
*/ */
try { try {
@ -143,13 +145,6 @@ public class ApexOpenRedirectRule extends AbstractApexRule {
} }
} }
private void addVariable(AbstractApexNode<?> node) {
ASTVariableExpression variable = node.getFirstChildOfType(ASTVariableExpression.class);
if (variable != null) {
listOfStringLiteralVariables.add(Helper.getFQVariableName(variable));
}
}
/** /**
* Traverses all new declarations to find PageReferences * Traverses all new declarations to find PageReferences
* *