#29165 use final and Objects.isNull()

dotCMS · Aug 23, 2024 · c0a1417 · c0a1417
1 parent 01ede6c
commit c0a1417
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/dotCMS/src/main/java/com/dotmarketing/servlets/ShortyServlet.java b/dotCMS/src/main/java/com/dotmarketing/servlets/ShortyServlet.java
@@ -2,6 +2,7 @@
 
 import com.dotcms.variant.business.web.VariantWebAPI.RenderContext;
 import java.io.IOException;
+import java.util.Objects;
 import java.util.Optional;
 import java.util.StringTokenizer;
 import java.util.regex.Matcher;
@@ -538,9 +539,9 @@ protected final String inodePath(final Contentlet contentlet,
                 .append(StringPool.FORWARD_SLASH).append(field.variable()).toString();
     }
 
-    private void validateContentlet(Contentlet contentlet, boolean live, String inode) throws DotDataException {
-        if (contentlet == null) {
-            String versionType = live ? PageMode.LIVE.name() : PageMode.WORKING.name();
+    private void validateContentlet(final Contentlet contentlet, final boolean live, final String inode) throws DotDataException {
+        if (Objects.isNull(contentlet)) {
+            final String versionType = live ? PageMode.LIVE.name() : PageMode.WORKING.name();
             throw new DotDataException(String.format("No contentlet found for %s inode %s", versionType, inode));
         }
     }