remove account id from revolut

This commit is contained in:
woodser 2023-08-29 07:14:54 -04:00
parent c46d210822
commit e6853d2e76
4 changed files with 5 additions and 63 deletions

View File

@ -90,18 +90,10 @@ public final class RevolutAccount extends PaymentAccount {
return (revolutAccountPayload()).getUserName();
}
public String getAccountId() {
return (revolutAccountPayload()).getAccountId();
}
public boolean userNameNotSet() {
return (revolutAccountPayload()).userNameNotSet();
}
public boolean hasOldAccountId() {
return (revolutAccountPayload()).hasOldAccountId();
}
private RevolutAccountPayload revolutAccountPayload() {
return (RevolutAccountPayload) paymentAccountPayload;
}
@ -109,9 +101,6 @@ public final class RevolutAccount extends PaymentAccount {
@Override
public void onAddToUser() {
super.onAddToUser();
// At save we apply the userName to accountId in case it is empty for backward compatibility
revolutAccountPayload().maybeApplyUserNameToAccountId();
}
@Override

View File

@ -37,10 +37,6 @@ import static com.google.common.base.Preconditions.checkArgument;
@ToString
@Slf4j
public final class RevolutAccountPayload extends PaymentAccountPayload {
// Only used as internal Id to not break existing account witness objects
// We still show it in case it is different to the userName for additional security
@Getter
private String accountId = "";
// Was added in 1.3.8
// To not break signed accounts we keep accountId as internal id used for signing.
@ -66,7 +62,6 @@ public final class RevolutAccountPayload extends PaymentAccountPayload {
private RevolutAccountPayload(String paymentMethod,
String id,
String accountId,
@Nullable String userName,
long maxTradePeriod,
Map<String, String> excludeFromJsonDataMap) {
@ -75,14 +70,12 @@ public final class RevolutAccountPayload extends PaymentAccountPayload {
maxTradePeriod,
excludeFromJsonDataMap);
this.accountId = accountId;
this.userName = userName;
}
@Override
public Message toProtoMessage() {
protobuf.RevolutAccountPayload.Builder revolutBuilder = protobuf.RevolutAccountPayload.newBuilder()
.setAccountId(accountId)
.setUserName(userName);
return getPaymentAccountPayloadBuilder().setRevolutAccountPayload(revolutBuilder).build();
}
@ -92,7 +85,6 @@ public final class RevolutAccountPayload extends PaymentAccountPayload {
protobuf.RevolutAccountPayload revolutAccountPayload = proto.getRevolutAccountPayload();
return new RevolutAccountPayload(proto.getPaymentMethodId(),
proto.getId(),
revolutAccountPayload.getAccountId(),
revolutAccountPayload.getUserName(),
proto.getMaxTradePeriod(),
new HashMap<>(proto.getExcludeFromJsonDataMap()));
@ -112,20 +104,9 @@ public final class RevolutAccountPayload extends PaymentAccountPayload {
private Tuple2<String, String> getLabelValueTuple() {
String label;
String value;
checkArgument(!userName.isEmpty() || hasOldAccountId(),
"Either username must be set or we have an old account with accountId");
if (!userName.isEmpty()) {
label = Res.get("payment.account.userName");
value = userName;
if (hasOldAccountId()) {
label += "/" + Res.get("payment.account.phoneNr");
value += "/" + accountId;
}
} else {
label = Res.get("payment.account.phoneNr");
value = accountId;
}
checkArgument(!userName.isEmpty(), "Username must be set");
label = Res.get("payment.account.userName");
value = userName;
return new Tuple2<>(label, value);
}
@ -142,35 +123,14 @@ public final class RevolutAccountPayload extends PaymentAccountPayload {
@Override
public byte[] getAgeWitnessInputData() {
// getAgeWitnessInputData is called at new account creation when accountId is empty string.
if (hasOldAccountId()) {
// If the accountId was already in place (updated user who had used accountId for account age) we keep the
// old accountId to not invalidate the existing account age witness.
return super.getAgeWitnessInputData(accountId.getBytes(StandardCharsets.UTF_8));
} else {
// If a new account was registered from version 1.3.8 or later we use the userName.
return super.getAgeWitnessInputData(userName.getBytes(StandardCharsets.UTF_8));
}
return super.getAgeWitnessInputData(userName.getBytes(StandardCharsets.UTF_8));
}
public boolean userNameNotSet() {
return userName.isEmpty();
}
public boolean hasOldAccountId() {
return !accountId.equals(userName);
}
public void setUserName(String userName) {
this.userName = userName;
}
// In case it is a new account we need to fill the accountId field to support not-updated traders who are not
// aware of the new userName field
public void maybeApplyUserNameToAccountId() {
if (accountId.isEmpty()) {
accountId = userName;
}
}
}

View File

@ -105,12 +105,6 @@ public class RevolutForm extends PaymentMethodForm {
TextField userNameTf = addCompactTopLabelTextField(gridPane, ++gridRow, Res.get("payment.account.userName"), userName).second;
userNameTf.setMouseTransparent(false);
if (account.hasOldAccountId()) {
String accountId = account.getAccountId();
TextField accountIdTf = addCompactTopLabelTextField(gridPane, ++gridRow, Res.get("payment.account.phoneNr"), accountId).second;
accountIdTf.setMouseTransparent(false);
}
addLimitations(true);
addCurrenciesGrid(false);
}

View File

@ -1052,8 +1052,7 @@ message PopmoneyAccountPayload {
}
message RevolutAccountPayload {
string account_id = 1;
string user_name = 2;
string user_name = 1;
}
message PerfectMoneyAccountPayload {