[camel] 02/04: (chores) camel-hyperledger-aries: cleaned unnecessary unboxing/boxing

[orpiske]

This is an automated email from the ASF dual-hosted git repository.

orpiske pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/camel.git

commit 83734ce223381b4c2d881734ef396bc6e97588ed
Author: Otavio Rodolfo Piske <angusyo...@gmail.com>
AuthorDate: Sat Jan 7 09:17:31 2023 +0100

    (chores) camel-hyperledger-aries: cleaned unnecessary unboxing/boxing
---
 .../CredentialDefinitionsServiceHandler.java       | 25 ++++++++--------------
 1 file changed, 9 insertions(+), 16 deletions(-)

diff --git 
a/components/camel-hyperledger-aries/src/main/java/org/apache/camel/component/aries/handler/CredentialDefinitionsServiceHandler.java
 
b/components/camel-hyperledger-aries/src/main/java/org/apache/camel/component/aries/handler/CredentialDefinitionsServiceHandler.java
index f1f69bc0571..324fc42a0f0 100644
--- 
a/components/camel-hyperledger-aries/src/main/java/org/apache/camel/component/aries/handler/CredentialDefinitionsServiceHandler.java
+++ 
b/components/camel-hyperledger-aries/src/main/java/org/apache/camel/component/aries/handler/CredentialDefinitionsServiceHandler.java
@@ -19,6 +19,7 @@ package org.apache.camel.component.aries.handler;
 import java.util.Arrays;
 import java.util.List;
 import java.util.Map;
+import java.util.function.Supplier;
 
 import io.nessus.aries.util.AssertState;
 import org.apache.camel.Exchange;
@@ -83,8 +84,7 @@ public class CredentialDefinitionsServiceHandler extends 
AbstractServiceHandler
                 }
                 AssertState.notNull(schemaVersion, "Cannot obtain 
schemaVersion");
 
-                Object auxValue = spec.get("autoSchema");
-                boolean autoSchema = isAutoSchema(auxValue);
+                boolean autoSchema = toBoolean(spec.get("autoSchema"), () -> 
endpoint.getConfiguration().isAutoSchema());
 
                 // Search existing schemas
                 DID publicDid = createClient().walletDidPublic().get();
@@ -110,14 +110,7 @@ public class CredentialDefinitionsServiceHandler extends 
AbstractServiceHandler
                 AssertState.isFalse(schemaIds.isEmpty(), "Cannot obtain schema 
ids for: " + filter);
                 AssertState.isEqual(1, schemaIds.size(), "Unexpected number of 
schema ids for: " + filter);
 
-                boolean supportRevocation = false;
-                auxValue = spec.get("supportRevocation");
-                if (auxValue instanceof Boolean) {
-                    supportRevocation = Boolean.valueOf((Boolean) auxValue);
-                }
-                if (auxValue instanceof String) {
-                    supportRevocation = Boolean.valueOf((String) auxValue);
-                }
+                boolean supportRevocation = 
toBoolean(spec.get("supportRevocation"), () -> false);
 
                 credDefReq = CredentialDefinitionRequest.builder()
                         .supportRevocation(supportRevocation)
@@ -133,14 +126,14 @@ public class CredentialDefinitionsServiceHandler extends 
AbstractServiceHandler
         }
     }
 
-    private boolean isAutoSchema(Object auxValue) {
-        if (auxValue instanceof Boolean) {
-            return Boolean.valueOf((Boolean) auxValue);
+    private static boolean toBoolean(Object object, Supplier<Boolean> 
provider) {
+        if (object instanceof Boolean) {
+            return (Boolean) object;
         }
-        if (auxValue instanceof String) {
-            return Boolean.valueOf((String) auxValue);
+        if (object instanceof String) {
+            return Boolean.valueOf((String) object);
         }
 
-        return endpoint.getConfiguration().isAutoSchema();
+        return provider.get();
     }
 }