#26277 sonarq feedback

dotCMS · Jan 9, 2024 · 15a43ed · 15a43ed
1 parent bebeed1
commit 15a43ed
Showing 1 changed file with 7 additions and 7 deletions.
diff --git a/dotCMS/src/main/java/com/dotcms/rendering/js/viewtools/WorkflowJsViewTool.java b/dotCMS/src/main/java/com/dotcms/rendering/js/viewtools/WorkflowJsViewTool.java
@@ -122,7 +122,7 @@ public JsContentMap fireEdit(final Map contentletMap,
             throw new IllegalArgumentException("Contentlet map is required or identifier");
         }
 
-        if (null == contentletMap || contentletMap.isEmpty() || !contentletMap.containsKey(CONTENT_TYPE)) {
+        if (!contentletMap.containsKey(CONTENT_TYPE)) {
 
             throw new IllegalArgumentException("ContentType attribute is required on the contentMap");
         }
@@ -161,7 +161,7 @@ public JsContentMap firePublish(final Map contentletMap,
             throw new IllegalArgumentException("Contentlet map is required or identifier");
         }
 
-        if (null == contentletMap || contentletMap.isEmpty() || !contentletMap.containsKey(CONTENT_TYPE)) {
+        if (!contentletMap.containsKey(CONTENT_TYPE)) {
 
             throw new IllegalArgumentException("ContentType attribute is required on the contentMap");
         }
@@ -200,7 +200,7 @@ public JsContentMap fireUnpublish(final Map contentletMap,
             throw new IllegalArgumentException("Contentlet map is required or identifier");
         }
 
-        if (null == contentletMap || contentletMap.isEmpty() || !contentletMap.containsKey(CONTENT_TYPE)) {
+        if (!contentletMap.containsKey(CONTENT_TYPE)) {
 
             throw new IllegalArgumentException("ContentType attribute is required on the contentMap");
         }
@@ -239,7 +239,7 @@ public JsContentMap fireArchive(final Map contentletMap,
             throw new IllegalArgumentException("Contentlet map is required or identifier");
         }
 
-        if (null == contentletMap || contentletMap.isEmpty() || !contentletMap.containsKey(CONTENT_TYPE)) {
+        if (!contentletMap.containsKey(CONTENT_TYPE)) {
 
             throw new IllegalArgumentException("contentType attribute is required on the contentMap");
         }
@@ -279,7 +279,7 @@ public JsContentMap fireUnarchive(final Map contentletMap,
             throw new IllegalArgumentException("Contentlet map is required or identifier");
         }
 
-        if (null == contentletMap || contentletMap.isEmpty() || !contentletMap.containsKey(CONTENT_TYPE)) {
+        if (!contentletMap.containsKey(CONTENT_TYPE)) {
 
             throw new IllegalArgumentException("ContentType attribute is required on the contentMap");
         }
@@ -319,7 +319,7 @@ public JsContentMap fireDelete(final Map contentletMap,
             throw new IllegalArgumentException("Contentlet map is required or identifier");
         }
 
-        if (null == contentletMap || contentletMap.isEmpty() || !contentletMap.containsKey(CONTENT_TYPE)) {
+        if (!contentletMap.containsKey(CONTENT_TYPE)) {
 
             throw new IllegalArgumentException("contentType attribute is required on the contentMap");
         }
@@ -361,7 +361,7 @@ public JsContentMap fireDestroy(final Map contentletMap,
             throw new IllegalArgumentException("Contentlet map is required or identifier");
         }
 
-        if (null == contentletMap || contentletMap.isEmpty() || !contentletMap.containsKey(CONTENT_TYPE)) {
+        if (!contentletMap.containsKey(CONTENT_TYPE)) {
 
             throw new IllegalArgumentException("ContentType attribute is required on the contentMap");
         }