Skip to content

Commit f6d3f32

Browse files
committed
- fix merge conflicts
- correct onInit_inValidAmount_correctParamPassed
2 parents bf3be5c + 05ac197 commit f6d3f32

File tree

7 files changed

+848
-240
lines changed

7 files changed

+848
-240
lines changed

raveandroid/src/main/java/com/flutterwave/raveandroid/account/AccountPresenter.java

Lines changed: 8 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -8,9 +8,10 @@
88
import com.flutterwave.raveandroid.FeeCheckRequestBody;
99
import com.flutterwave.raveandroid.Payload;
1010
import com.flutterwave.raveandroid.PayloadBuilder;
11+
import com.flutterwave.raveandroid.PayloadEncryptor;
12+
import com.flutterwave.raveandroid.PayloadToJsonConverter;
1113
import com.flutterwave.raveandroid.RavePayInitializer;
1214
import com.flutterwave.raveandroid.TransactionStatusChecker;
13-
import com.flutterwave.raveandroid.Utils;
1415
import com.flutterwave.raveandroid.ViewObject;
1516
import com.flutterwave.raveandroid.card.ChargeRequestBody;
1617
import com.flutterwave.raveandroid.data.Bank;
@@ -89,6 +90,10 @@ public class AccountPresenter implements AccountContract.UserActionsListener {
8990
TransactionStatusChecker transactionStatusChecker;
9091
@Inject
9192
NetworkRequestImpl networkRequest;
93+
@Inject
94+
PayloadToJsonConverter payloadToJsonConverter;
95+
@Inject
96+
PayloadEncryptor payloadEncryptor;
9297

9398
@Inject
9499
AccountPresenter(Context context, AccountContract.View mView) {
@@ -120,8 +125,8 @@ public void onError(String message) {
120125
@Override
121126
public void chargeAccount(final Payload payload, String encryptionKey, final boolean internetBanking) {
122127

123-
String cardRequestBodyAsString = Utils.convertChargeRequestPayloadToJson(payload);
124-
String encryptedCardRequestBody = Utils.getEncryptedData(cardRequestBodyAsString, encryptionKey);
128+
String cardRequestBodyAsString = payloadToJsonConverter.convertChargeRequestPayloadToJson(payload);
129+
String encryptedCardRequestBody = payloadEncryptor.getEncryptedData(cardRequestBodyAsString, encryptionKey);
125130

126131
ChargeRequestBody body = new ChargeRequestBody();
127132
body.setAlg("3DES-24");

raveandroid/src/main/java/com/flutterwave/raveandroid/ach/AchContract.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@ interface View {
3232
interface UserActionsListener {
3333
void init(RavePayInitializer ravePayInitializer);
3434

35-
void onPayButtonClicked(RavePayInitializer ravePayInitializer, String amount);
35+
void onDataCollected(RavePayInitializer ravePayInitializer, String amount);
3636

3737
void verifyRequeryResponse(RequeryResponse response, String responseAsJSONString, RavePayInitializer ravePayInitializer, String flwRef);
3838

raveandroid/src/main/java/com/flutterwave/raveandroid/ach/AchFragment.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -96,7 +96,7 @@ public void onClick(View view) {
9696
int i = view.getId();
9797
if (i == R.id.rave_payButton) {
9898
String amount = amountEt.getText().toString();
99-
presenter.onPayButtonClicked(ravePayInitializer, amount);
99+
presenter.onDataCollected(ravePayInitializer, amount);
100100
}
101101
}
102102

raveandroid/src/main/java/com/flutterwave/raveandroid/ach/AchPresenter.java

Lines changed: 25 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -69,11 +69,11 @@ public void init(RavePayInitializer ravePayInitializer) {
6969
}
7070

7171
@Override
72-
public void onPayButtonClicked(RavePayInitializer ravePayInitializer, String amount) {
72+
public void onDataCollected(RavePayInitializer ravePayInitializer, String amount) {
7373

7474
mView.showAmountError(null);
7575

76-
boolean isAmountValid = amountValidator.isAmountValid(ravePayInitializer.getAmount());
76+
boolean isAmountValid = amountValidator.isAmountValid(amount);
7777

7878
if (isAmountValid) {
7979
mView.onValidationSuccessful(amount);
@@ -86,28 +86,31 @@ public void onPayButtonClicked(RavePayInitializer ravePayInitializer, String amo
8686
@Override
8787
public void processTransaction(String amount, final RavePayInitializer ravePayInitializer, final boolean isDisplayFee) {
8888

89-
ravePayInitializer.setAmount(Double.parseDouble(amount));
90-
PayloadBuilder builder = new PayloadBuilder();
91-
builder.setAmount(ravePayInitializer.getAmount() + "")
92-
.setCountry(ravePayInitializer.getCountry())
93-
.setCurrency(ravePayInitializer.getCurrency())
94-
.setEmail(ravePayInitializer.getEmail())
95-
.setFirstname(ravePayInitializer.getfName())
96-
.setLastname(ravePayInitializer.getlName())
97-
.setIP(deviceIdGetter.getDeviceId())
98-
.setTxRef(ravePayInitializer.getTxRef())
99-
.setMeta(ravePayInitializer.getMeta())
100-
.setPBFPubKey(ravePayInitializer.getPublicKey())
101-
.setIsUsBankCharge(ravePayInitializer.isWithAch())
102-
.setDevice_fingerprint(deviceIdGetter.getDeviceId());
103-
104-
if (ravePayInitializer.getPayment_plan() != null) {
105-
builder.setPaymentPlan(ravePayInitializer.getPayment_plan());
106-
}
89+
if (ravePayInitializer != null) {
10790

108-
Payload body = builder.createBankPayload();
91+
ravePayInitializer.setAmount(Double.parseDouble(amount));
92+
PayloadBuilder builder = new PayloadBuilder();
93+
builder.setAmount(ravePayInitializer.getAmount() + "")
94+
.setCountry(ravePayInitializer.getCountry())
95+
.setCurrency(ravePayInitializer.getCurrency())
96+
.setEmail(ravePayInitializer.getEmail())
97+
.setFirstname(ravePayInitializer.getfName())
98+
.setLastname(ravePayInitializer.getlName())
99+
.setIP(deviceIdGetter.getDeviceId())
100+
.setTxRef(ravePayInitializer.getTxRef())
101+
.setMeta(ravePayInitializer.getMeta())
102+
.setPBFPubKey(ravePayInitializer.getPublicKey())
103+
.setIsUsBankCharge(ravePayInitializer.isWithAch())
104+
.setDevice_fingerprint(deviceIdGetter.getDeviceId());
105+
106+
if (ravePayInitializer.getPayment_plan() != null) {
107+
builder.setPaymentPlan(ravePayInitializer.getPayment_plan());
108+
}
109109

110-
chargeAccount(body, ravePayInitializer.getEncryptionKey(), ravePayInitializer.getIsDisplayFee());
110+
Payload body = builder.createBankPayload();
111+
112+
chargeAccount(body, ravePayInitializer.getEncryptionKey(), ravePayInitializer.getIsDisplayFee());
113+
}
111114
}
112115

113116

raveandroid/src/main/java/com/flutterwave/raveandroid/responses/ChargeResponse.java

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,7 @@ public class ChargeResponse {
99
String message;
1010
Data data;
1111

12+
1213
public String getStatus() {
1314
return status;
1415
}
@@ -33,6 +34,7 @@ public void setData(Data data) {
3334
this.data = data;
3435
}
3536

37+
3638
public static class AccountValidateInstructions {
3739
public String getInstruction() {
3840
return instruction;
@@ -60,6 +62,11 @@ public static class Data {
6062
String bankname;
6163
String note;
6264
String amount;
65+
String validateInstruction;
66+
67+
public void setValidateInstruction(String validateInstruction) {
68+
this.validateInstruction = validateInstruction;
69+
}
6370

6471
public String getFlw_reference() {
6572
return flw_reference;
@@ -99,8 +106,6 @@ public String getValidateInstruction() {
99106
return validateInstruction;
100107
}
101108

102-
String validateInstruction;
103-
104109
public String getTx_ref() {
105110
return txRef;
106111
}

0 commit comments

Comments
 (0)