Skip to content

Commit 75f29ad

Browse files
committed
Fix FileCollector#addSourceFile calls
1 parent 5915835 commit 75f29ad

File tree

4 files changed

+7
-6
lines changed

4 files changed

+7
-6
lines changed

net.sourceforge.pmd.eclipse.plugin.test/src/main/java/net/sourceforge/pmd/eclipse/BasicPMDTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -76,7 +76,7 @@ private void runPmd(String javaVersion) {
7676
final String sourceCode = "public class Foo {\n public void foo() {\nreturn;\n}}";
7777

7878
try (PmdAnalysis pmd = PmdAnalysis.create(configuration)) {
79-
pmd.files().addSourceFile("Foo.java", sourceCode);
79+
pmd.files().addSourceFile(sourceCode, "Foo.java");
8080
Report result = pmd.performAnalysisAndCollectReport();
8181

8282
Assert.assertFalse("There should be at least one violation", result.getViolations().isEmpty());

net.sourceforge.pmd.eclipse.plugin/src/main/java/net/sourceforge/pmd/eclipse/runtime/cmd/BaseVisitor.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -308,7 +308,7 @@ protected final void reviewResource(IResource resource) {
308308
PmdAnalysis pmdAnalysis = PmdAnalysis.create(configuration());) {
309309

310310
String sourceContents = IOUtil.toString(input);
311-
pmdAnalysis.files().addSourceFile(sourceCodeFile.getAbsolutePath(), sourceContents);
311+
pmdAnalysis.files().addSourceFile(sourceContents, sourceCodeFile.getAbsolutePath());
312312

313313
pmdAnalysis.addRuleSets(getRuleSetList());
314314

net.sourceforge.pmd.eclipse.plugin/src/main/java/net/sourceforge/pmd/eclipse/runtime/cmd/ReviewResourceForRuleCommand.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -94,7 +94,7 @@ public void execute() {
9494
PmdAnalysis pmdAnalysis = PmdAnalysis.create(configuration)) {
9595

9696
pmdAnalysis.addRuleSet(ruleSet);
97-
pmdAnalysis.files().addSourceFile(sourceCodeFile.getAbsolutePath(), IOUtil.toString(input));
97+
pmdAnalysis.files().addSourceFile(IOUtil.toString(input), sourceCodeFile.getAbsolutePath());
9898

9999
report = pmdAnalysis.performAnalysisAndCollectReport();
100100
} catch (Exception e) {

net.sourceforge.pmd.eclipse.plugin/src/main/java/net/sourceforge/pmd/eclipse/ui/views/ast/XPathEvaluator.java

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -25,6 +25,7 @@
2525
* @author Brian Remedios
2626
*/
2727
public final class XPathEvaluator {
28+
private static final String SNIPPET_FILENAME = "snippet.java";
2829

2930
public static final XPathEvaluator INSTANCE = new XPathEvaluator();
3031

@@ -42,15 +43,15 @@ public Node getCompilationUnit(String source) {
4243

4344
try (PmdAnalysis pmd = PmdAnalysis.create(configuration)) {
4445
pmd.addRuleSet(ruleset);
45-
pmd.files().addSourceFile("snippet.java", source);
46+
pmd.files().addSourceFile(source, SNIPPET_FILENAME);
4647
pmd.performAnalysis();
4748
}
4849

4950
return rule.getRootNode();
5051
}
5152

5253
private LanguageVersion getLanguageVersion() {
53-
return LanguageRegistry.getLanguage(JavaLanguageModule.NAME).getDefaultVersion();
54+
return LanguageRegistry.PMD.getLanguageByFullName(JavaLanguageModule.NAME).getDefaultVersion();
5455
}
5556

5657
/**
@@ -72,7 +73,7 @@ public List<RuleViolation> evaluate(String source, String xpathQuery, String xpa
7273
configuration.setForceLanguageVersion(getLanguageVersion());
7374
try (PmdAnalysis pmd = PmdAnalysis.create(configuration)) {
7475
pmd.addRuleSet(ruleSet);
75-
pmd.files().addSourceFile("snippet.java", source);
76+
pmd.files().addSourceFile(source, SNIPPET_FILENAME);
7677
Report report = pmd.performAnalysisAndCollectReport();
7778
return report.getViolations();
7879
}

0 commit comments

Comments
 (0)