From f983fc4bc7a4dc0e9d175e77cfaf8a2127aaeb2d Mon Sep 17 00:00:00 2001
From: Jean-Noel Rouvignac <jean-noel.rouvignac@forgerock.com>
Date: Wed, 25 Feb 2015 14:16:50 +0000
Subject: [PATCH] AutoRefactor: fix boolean usage
---
opendj-server-legacy/src/main/java/org/opends/server/loggers/AbstractTextAccessLogPublisher.java | 20 ++++----------------
1 files changed, 4 insertions(+), 16 deletions(-)
diff --git a/opendj-server-legacy/src/main/java/org/opends/server/loggers/AbstractTextAccessLogPublisher.java b/opendj-server-legacy/src/main/java/org/opends/server/loggers/AbstractTextAccessLogPublisher.java
index 8309997..cfc1411 100644
--- a/opendj-server-legacy/src/main/java/org/opends/server/loggers/AbstractTextAccessLogPublisher.java
+++ b/opendj-server-legacy/src/main/java/org/opends/server/loggers/AbstractTextAccessLogPublisher.java
@@ -343,16 +343,9 @@
private boolean filterRequest(final Operation operation)
{
- // Check target DN.
- if ((targetDNNotEqualTo.length > 0 || targetDNEqualTo.length > 0)
- && !filterRequestTargetDN(operation))
- {
- return false;
- }
-
+ return (targetDNNotEqualTo.length == 0 && targetDNEqualTo.length == 0)
+ || filterRequestTargetDN(operation);
// TODO: check required controls.
-
- return true;
}
@@ -557,13 +550,8 @@
}
// Check group membership.
- if ((userIsNotMemberOf.length > 0 || userIsMemberOf.length > 0)
- && !filterUserIsMemberOf(connection))
- {
- return false;
- }
-
- return true;
+ return (userIsNotMemberOf.length == 0 && userIsMemberOf.length == 0)
+ || filterUserIsMemberOf(connection);
}
--
Gitblit v1.10.0