From c5907dbc102287fc8ea021ff3cb001060419ed9f Mon Sep 17 00:00:00 2001 From: cagache Date: Mon, 1 Jul 2019 16:28:48 +0300 Subject: [PATCH] Removed unnecessary null check before Boolean.parseBoolean --- .../module/org_alfresco_module_rm/script/AuditLogGet.java | 2 +- .../alfresco/repo/web/scripts/roles/DynamicAuthoritiesGet.java | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/rm-community/rm-community-repo/source/java/org/alfresco/module/org_alfresco_module_rm/script/AuditLogGet.java b/rm-community/rm-community-repo/source/java/org/alfresco/module/org_alfresco_module_rm/script/AuditLogGet.java index 9e880b37bd..a07ff9790d 100644 --- a/rm-community/rm-community-repo/source/java/org/alfresco/module/org_alfresco_module_rm/script/AuditLogGet.java +++ b/rm-community/rm-community-repo/source/java/org/alfresco/module/org_alfresco_module_rm/script/AuditLogGet.java @@ -138,7 +138,7 @@ public class AuditLogGet extends BaseAuditRetrievalWebScript boolean attach = false; String attachFileName = null; String export = req.getParameter(PARAM_EXPORT); - if (export != null && Boolean.parseBoolean(export)) + if (Boolean.parseBoolean(export)) { attach = true; attachFileName = auditTrail.getName(); diff --git a/rm-community/rm-community-repo/source/java/org/alfresco/repo/web/scripts/roles/DynamicAuthoritiesGet.java b/rm-community/rm-community-repo/source/java/org/alfresco/repo/web/scripts/roles/DynamicAuthoritiesGet.java index 6829ec2c9f..f74cacae1e 100644 --- a/rm-community/rm-community-repo/source/java/org/alfresco/repo/web/scripts/roles/DynamicAuthoritiesGet.java +++ b/rm-community/rm-community-repo/source/java/org/alfresco/repo/web/scripts/roles/DynamicAuthoritiesGet.java @@ -268,7 +268,7 @@ public class DynamicAuthoritiesGet extends AbstractWebScript implements RecordsM { boolean attach = false; String export = req.getParameter(PARAM_EXPORT); - if (export != null && Boolean.parseBoolean(export)) + if (Boolean.parseBoolean(export)) { attach = true; }