Skip to content

Commit de04036

Browse files
committed
Fix pmd problems
1 parent bd38d6f commit de04036

File tree

5 files changed

+10
-10
lines changed

5 files changed

+10
-10
lines changed

src/main/java/software/xdev/saveactions/processors/java/inspection/CustomAccessCanBeTightenedInspection.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -86,7 +86,7 @@
8686
*
8787
* @see com.intellij.codeInspection.visibility.AccessCanBeTightenedInspection
8888
*/
89-
@SuppressWarnings({"java:S6201", "java:S3776", "java:S125", "java:S3398", "java:S2637", "java:S120", "java:S6541", "java:S135"})
89+
@SuppressWarnings({"java:S6201", "java:S3776", "java:S125", "java:S3398", "java:S2637", "java:S120", "java:S6541", "java:S135", "PMD"})
9090
public class CustomAccessCanBeTightenedInspection extends AbstractBaseJavaLocalInspectionTool {
9191
private static final Logger LOG = Logger.getInstance(CustomAccessCanBeTightenedInspection.class);
9292
private final VisibilityInspection myVisibilityInspection;

src/main/java/software/xdev/saveactions/processors/java/inspection/style/CustomUnqualifiedStaticUsageInspection.java

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -85,6 +85,7 @@ public void visitReferenceExpression(@NotNull final PsiReferenceExpression expre
8585
this.registerError(expression, expression);
8686
}
8787

88+
@SuppressWarnings("PMD.NPathComplexity")
8889
private boolean isUnqualifiedStaticAccess(final PsiReferenceExpression expression)
8990
{
9091
if(CustomUnqualifiedStaticUsageInspection.this.m_ignoreStaticAccessFromStaticContext)

src/main/java/software/xdev/saveactions/ui/BuildPanel.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -123,7 +123,7 @@ private boolean hasId(final String id)
123123

124124
private String getId()
125125
{
126-
return this.quickList.hashCode() + "";
126+
return String.valueOf(this.quickList.hashCode());
127127
}
128128

129129
@Override

src/main/java/software/xdev/saveactions/ui/FileMaskPanel.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -116,7 +116,7 @@ private InputValidator getRegexInputValidator()
116116
@Override
117117
public boolean checkInput(final String string)
118118
{
119-
if(string == null || string.trim().isEmpty())
119+
if(string == null || string.isBlank())
120120
{
121121
// do not allow null or blank entries
122122
return false;

src/test/java/software/xdev/saveactions/junit/JUnit5Utils.java

Lines changed: 6 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -15,15 +15,14 @@ private JUnit5Utils()
1515

1616
public static void rethrowAsJunit5Error(final AssertionError error)
1717
{
18-
if(error.getCause() instanceof final InvocationTargetException intellijInternal)
18+
if(error.getCause() instanceof final InvocationTargetException intellijInternal
19+
&& intellijInternal.getCause() instanceof final FileComparisonFailure fileComparisonFailure)
1920
{
20-
if(intellijInternal.getCause() instanceof final FileComparisonFailure fileComparisonFailure)
21-
{
22-
final String expected = fileComparisonFailure.getExpected();
23-
final String actual = fileComparisonFailure.getActual();
24-
throw new AssertionFailedError("Expected file do not match actual file", expected, actual);
25-
}
21+
final String expected = fileComparisonFailure.getExpected();
22+
final String actual = fileComparisonFailure.getActual();
23+
throw new AssertionFailedError("Expected file do not match actual file", expected, actual);
2624
}
25+
2726
throw error;
2827
}
2928

0 commit comments

Comments
 (0)