From 64721a895973f935c1adb975247770f402a88fdf Mon Sep 17 00:00:00 2001
From: Jean-Noël Rouvignac <jean-noel.rouvignac@forgerock.com>
Date: Mon, 25 Apr 2016 15:10:24 +0000
Subject: [PATCH] ACI UCDetector and AutoRefactor code cleanup
---
opendj-server-legacy/src/main/java/org/opends/server/authorization/dseecompat/ExtOp.java | 46 +++++++++++++++++++++-------------------------
1 files changed, 21 insertions(+), 25 deletions(-)
diff --git a/opendj-server-legacy/src/main/java/org/opends/server/authorization/dseecompat/ExtOp.java b/opendj-server-legacy/src/main/java/org/opends/server/authorization/dseecompat/ExtOp.java
index f137b79..d03cfd0 100644
--- a/opendj-server-legacy/src/main/java/org/opends/server/authorization/dseecompat/ExtOp.java
+++ b/opendj-server-legacy/src/main/java/org/opends/server/authorization/dseecompat/ExtOp.java
@@ -12,29 +12,20 @@
* information: "Portions Copyright [year] [name of copyright owner]".
*
* Copyright 2008 Sun Microsystems, Inc.
- * Portions Copyright 2013-2015 ForgeRock AS.
+ * Portions Copyright 2013-2016 ForgeRock AS.
*/
package org.opends.server.authorization.dseecompat;
import static org.opends.messages.AccessControlMessages.*;
-import java.util.HashSet;
import java.util.Set;
-/**
- * This class represents an ACI's extop keyword rule.
- */
+/** This class represents an ACI's extop keyword rule. */
public class ExtOp {
-
- /**
- * HashSet of OID strings parsed from the decode.
- */
- private Set<String> extOpOIDs = new HashSet<>();
-
- /**
- * Enumeration representing the extop operator.
- */
- private EnumTargetOperator op = EnumTargetOperator.EQUALITY;
+ /** Set of OID strings parsed from the decode. */
+ private final Set<String> extOpOIDs;
+ /** Enumeration representing the extop operator. */
+ private final EnumTargetOperator op;
/**
* Creates a class that can be used to evaluate a extop rule.
@@ -78,18 +69,23 @@
{
return false;
}
- boolean ret = false;
- for(String oid : extOpOIDs)
+ boolean ret = isApplicable(matchCtx.getExtOpOID());
+ if (EnumTargetOperator.NOT_EQUALITY.equals(op))
{
- if(oid.equals("*") || matchCtx.getExtOpOID().equals(oid)) {
- ret=true;
- break;
- }
- }
- if(op.equals(EnumTargetOperator.NOT_EQUALITY))
- {
- ret = !ret;
+ return !ret;
}
return ret;
}
+
+ private boolean isApplicable(String matchOID)
+ {
+ for(String oid : extOpOIDs)
+ {
+ if ("*".equals(oid) || matchOID.equals(oid))
+ {
+ return true;
+ }
+ }
+ return false;
+ }
}
--
Gitblit v1.10.0