minor changes to DeferredDocument and UnsignedDocument

eu-digital-identity-wallet · Sep 26, 2024 · b7cc53e · b7cc53e
1 parent 1e258cf
commit b7cc53e
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 36 deletions.
diff --git a/document-manager/src/main/java/eu/europa/ec/eudi/wallet/document/DeferredDocument.kt b/document-manager/src/main/java/eu/europa/ec/eudi/wallet/document/DeferredDocument.kt
@@ -59,36 +59,6 @@ class DeferredDocument(
     override val state: State
         get() = base?.state ?: State.DEFERRED
 
-    override fun equals(other: Any?): Boolean {
-        if (this === other) return true
-        if (javaClass != other?.javaClass) return false
-
-        other as DeferredDocument
-
-        if (id != other.id) return false
-        if (docType != other.docType) return false
-        if (name != other.name) return false
-        if (usesStrongBox != other.usesStrongBox) return false
-        if (requiresUserAuth != other.requiresUserAuth) return false
-        if (createdAt != other.createdAt) return false
-        if (state != other.state) return false
-        if (!relatedData.contentEquals(other.relatedData)) return false
-
-        return true
-    }
-
-    override fun hashCode(): Int {
-        var result = id.hashCode()
-        result = 31 * result + docType.hashCode()
-        result = 31 * result + name.hashCode()
-        result = 31 * result + usesStrongBox.hashCode()
-        result = 31 * result + requiresUserAuth.hashCode()
-        result = 31 * result + createdAt.hashCode()
-        result = 31 * result + state.hashCode()
-        result = 31 * result + relatedData.contentHashCode()
-        return result
-    }
-
     override fun toString(): String {
         return "DeferredDocument(id='$id', docType='$docType', name='$name', usesStrongBox=$usesStrongBox, requiresUserAuth=$requiresUserAuth, createdAt=$createdAt, state=$state, relatedData=${relatedData.contentToString()})"
     }
@@ -103,8 +73,8 @@ class DeferredDocument(
             requiresUserAuth = baseDocument.requiresUserAuth,
             createdAt = baseDocument.createdAt,
             certificatesNeedAuth = baseDocument.pendingCredentials
-                .firstOrNull { it is SecureAreaBoundCredential }
-                ?.let { it as SecureAreaBoundCredential }
+                .filterIsInstance<SecureAreaBoundCredential>()
+                .firstOrNull()
                 ?.attestation
                 ?.certChain
                 ?.javaX509Certificates

diff --git a/document-manager/src/main/java/eu/europa/ec/eudi/wallet/document/UnsignedDocument.kt b/document-manager/src/main/java/eu/europa/ec/eudi/wallet/document/UnsignedDocument.kt
@@ -141,14 +141,13 @@ open class UnsignedDocument(
             usesStrongBox = baseDocument.usesStrongBox,
             requiresUserAuth = baseDocument.requiresUserAuth,
             createdAt = baseDocument.createdAt,
-            certificatesNeedAuth = baseDocument.pendingCredentials.firstOrNull() { it is SecureAreaBoundCredential }
-                ?.let { it as SecureAreaBoundCredential }
+            certificatesNeedAuth = baseDocument.pendingCredentials
+                .filterIsInstance<SecureAreaBoundCredential>()
+                .firstOrNull()
                 ?.attestation
                 ?.certChain
                 ?.javaX509Certificates
                 ?: emptyList(),
         ).also { it.base = baseDocument }
     }
-
-
 }