Merge pull request #8 from Foundation-Devices/jeandudey/coin-info

handle null values in coininfo constructor
This commit is contained in:
craigraw 2024-10-22 19:18:38 +02:00 committed by GitHub
commit e4071ab1cc
No known key found for this signature in database
GPG key ID: B5690EEEBB952194

View file

@ -4,6 +4,8 @@ import co.nstant.in.cbor.model.DataItem;
import co.nstant.in.cbor.model.Map; import co.nstant.in.cbor.model.Map;
import co.nstant.in.cbor.model.UnsignedInteger; import co.nstant.in.cbor.model.UnsignedInteger;
import java.util.Objects;
public class CryptoCoinInfo extends RegistryItem { public class CryptoCoinInfo extends RegistryItem {
public static final int TYPE_KEY = 1; public static final int TYPE_KEY = 1;
public static final int NETWORK_KEY = 2; public static final int NETWORK_KEY = 2;
@ -12,7 +14,7 @@ public class CryptoCoinInfo extends RegistryItem {
private final Integer network; private final Integer network;
public CryptoCoinInfo(Integer type, Integer network) { public CryptoCoinInfo(Integer type, Integer network) {
if(network.equals(Network.GOERLI.networkValue) && !type.equals(Type.ETHEREUM.typeValue)) { if(Objects.equals(network, Network.GOERLI.networkValue) && !Objects.equals(type, Type.ETHEREUM.typeValue)) {
throw new IllegalArgumentException("Goerli network can only be selected for Ethereum"); throw new IllegalArgumentException("Goerli network can only be selected for Ethereum");
} }
this.type = type; this.type = type;