From 408e18619d6ea0cb3cd55c19ba224d80e73682c1 Mon Sep 17 00:00:00 2001
From: Gary <982483+gmalouf@users.noreply.github.com>
Date: Wed, 25 Oct 2023 10:54:37 -0400
Subject: [PATCH 1/2] Remove de-duping via HashSet from accounts,foriegn apps,
 and foreign assets setters in MethodCallTransactionBuilder. Now set solely
 via lists to ensure stable ordering maintained. (#652)

---
 .../builder/transaction/MethodCallTransactionBuilder.java  | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/src/main/java/com/algorand/algosdk/builder/transaction/MethodCallTransactionBuilder.java b/src/main/java/com/algorand/algosdk/builder/transaction/MethodCallTransactionBuilder.java
index d42b2dd7f..7fb8ac54c 100644
--- a/src/main/java/com/algorand/algosdk/builder/transaction/MethodCallTransactionBuilder.java
+++ b/src/main/java/com/algorand/algosdk/builder/transaction/MethodCallTransactionBuilder.java
@@ -12,7 +12,6 @@
 
 import java.math.BigInteger;
 import java.util.ArrayList;
-import java.util.HashSet;
 import java.util.List;
 
 @SuppressWarnings("unchecked")
@@ -105,7 +104,7 @@ public T signer(TxnSigner signer) {
     @Override
     public T accounts(List<Address> accounts) {
         if (accounts != null)
-            this.foreignAccounts = new ArrayList<>(new HashSet<>(accounts));
+            this.foreignAccounts = new ArrayList<>(accounts);
         else
             this.foreignAccounts.clear();
         return (T) this;
@@ -114,7 +113,7 @@ public T accounts(List<Address> accounts) {
     @Override
     public T foreignApps(List<Long> foreignApps) {
         if (foreignApps != null)
-            this.foreignApps = new ArrayList<>(new HashSet<>(foreignApps));
+            this.foreignApps = new ArrayList<>(foreignApps);
         else
             this.foreignApps.clear();
         return (T) this;
@@ -123,7 +122,7 @@ public T foreignApps(List<Long> foreignApps) {
     @Override
     public T foreignAssets(List<Long> foreignAssets) {
         if (foreignAssets != null)
-            this.foreignAssets = new ArrayList<>(new HashSet<>(foreignAssets));
+            this.foreignAssets = new ArrayList<>(foreignAssets);
         else
             this.foreignAssets.clear();
         return (T) this;

From af7c719ef8d91209f63957f4beb6edf5d6a45e7a Mon Sep 17 00:00:00 2001
From: gmalouf <gmalouf@users.noreply.github.com>
Date: Wed, 25 Oct 2023 16:14:16 +0000
Subject: [PATCH 2/2] bump up version to 2.4.0

---
 CHANGELOG.md | 13 +++++++++++++
 README.md    |  2 +-
 pom.xml      |  2 +-
 3 files changed, 15 insertions(+), 2 deletions(-)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 0c7c222cc..8d1343a17 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -1,3 +1,16 @@
+# 2.4.0
+
+<!-- Release notes generated using configuration in .github/release.yml at release/2.4.0 -->
+
+## What's Changed
+### Enhancements
+* Patch: Remove de-duping via HashSet from accounts,foriegn apps, and foreign … by @gmalouf in https://github.com/algorand/java-algorand-sdk/pull/652
+
+## New Contributors
+* @gmalouf made their first contribution in https://github.com/algorand/java-algorand-sdk/pull/652
+
+**Full Changelog**: https://github.com/algorand/java-algorand-sdk/compare/2.3.0...2.4.0
+
 # 2.3.0
 
 <!-- Release notes generated using configuration in .github/release.yml at release/2.3.0 -->
diff --git a/README.md b/README.md
index 76a53027c..2964d18b8 100644
--- a/README.md
+++ b/README.md
@@ -19,7 +19,7 @@ Maven:
 <dependency>
     <groupId>com.algorand</groupId>
     <artifactId>algosdk</artifactId>
-    <version>2.3.0</version>
+    <version>2.4.0</version>
 </dependency>
 ```
 
diff --git a/pom.xml b/pom.xml
index e94d3abe6..e18984ea8 100755
--- a/pom.xml
+++ b/pom.xml
@@ -4,7 +4,7 @@
 
     <groupId>com.algorand</groupId>
     <artifactId>algosdk</artifactId>
-    <version>2.3.0</version>
+    <version>2.4.0</version>
     <packaging>jar</packaging>
 
     <name>${project.groupId}:${project.artifactId}</name>