Skip to content

Commit 9d4e2ad

Browse files
committed
Fixes #29: Processing errors without cause
1 parent 439b541 commit 9d4e2ad

File tree

2 files changed

+5
-2
lines changed

2 files changed

+5
-2
lines changed

ReleaseNotes.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,7 @@ Eclipse Update Site:
1717
### Fixed Issues
1818

1919
* [#20](https://github.com/pmd/pmd-eclipse-plugin/issues/20): category.xml maybe broken
20+
* [#29](https://github.com/pmd/pmd-eclipse-plugin/issues/29): Processing errors without cause
2021
* [#32](https://github.com/pmd/pmd-eclipse-plugin/issues/32): Upgrade PMD to 6.2.0
2122
* [#43](https://github.com/pmd/pmd-eclipse-plugin/issues/43): Update unit tests to use new ruleset categories
2223

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

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -374,10 +374,12 @@ public String defaultFileExtension() {
374374

375375
if (collectingReport.hasErrors()) {
376376
StringBuilder message = new StringBuilder("There were processing errors!\n");
377-
Iterator<ProcessingError> errors = context.getReport().errors();
377+
Iterator<ProcessingError> errors = collectingReport.errors();
378378
while (errors.hasNext()) {
379379
ProcessingError error = errors.next();
380-
message.append(error.getFile()).append(": ").append(error.getMsg()).append("\n");
380+
message.append(error.getFile()).append(": ").append(error.getMsg()).append(' ')
381+
.append(error.getDetail())
382+
.append("\n");
381383
}
382384
PMDPlugin.getDefault().logWarn(message.toString());
383385
throw new PMDException(message.toString());

0 commit comments

Comments
 (0)