(maven-enforcer) branch master updated: null analysis (#907)

[elharo]

This is an automated email from the ASF dual-hosted git repository.

elharo pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/maven-enforcer.git


The following commit(s) were added to refs/heads/master by this push:
     new 1f7ac3c3 null analysis (#907)
1f7ac3c3 is described below

commit 1f7ac3c384760fc8d5559b2786ba443f9cce8ca5
Author: Elliotte Rusty Harold <elh...@users.noreply.github.com>
AuthorDate: Tue Jul 8 18:56:32 2025 +0000

    null analysis (#907)
---
 .../main/java/org/apache/maven/enforcer/rules/BannedRepositories.java   | 2 +-
 .../java/org/apache/maven/enforcer/rules/RequirePluginVersions.java     | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git 
a/enforcer-rules/src/main/java/org/apache/maven/enforcer/rules/BannedRepositories.java
 
b/enforcer-rules/src/main/java/org/apache/maven/enforcer/rules/BannedRepositories.java
index ca375cad..368a7f84 100644
--- 
a/enforcer-rules/src/main/java/org/apache/maven/enforcer/rules/BannedRepositories.java
+++ 
b/enforcer-rules/src/main/java/org/apache/maven/enforcer/rules/BannedRepositories.java
@@ -90,7 +90,7 @@ public final class BannedRepositories extends 
AbstractStandardEnforcerRule {
 
         String errMsg = repoErrMsg + pluginRepoErrMsg;
 
-        if (errMsg != null && !errMsg.isEmpty()) {
+        if (!errMsg.isEmpty()) {
             throw new EnforcerRuleException(errMsg);
         }
     }
diff --git 
a/enforcer-rules/src/main/java/org/apache/maven/enforcer/rules/RequirePluginVersions.java
 
b/enforcer-rules/src/main/java/org/apache/maven/enforcer/rules/RequirePluginVersions.java
index f38170bc..5ba97591 100644
--- 
a/enforcer-rules/src/main/java/org/apache/maven/enforcer/rules/RequirePluginVersions.java
+++ 
b/enforcer-rules/src/main/java/org/apache/maven/enforcer/rules/RequirePluginVersions.java
@@ -401,7 +401,7 @@ public final class RequirePluginVersions extends 
AbstractStandardEnforcerRule {
                 throw new EnforcerRuleError("Invalid " + field + " string: " + 
pluginString);
             }
         } else {
-            throw new EnforcerRuleError("Invalid " + field + " string: " + 
pluginString);
+            throw new EnforcerRuleError("Invalid " + field + " null plugin 
string.");
         }
     }