New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Continuous payment #35
Conversation
Black Duck Security ReportMerging #35 into master will not change security risk. |
Code Climate has analyzed commit be8b252 and detected 0 issues on this pull request. View more on Code Climate. |
@@ -106,6 +107,32 @@ private static void createAccountLinkQrCode(final PaymentApi apiInstance){ | |||
System.out.println(e.getResponseBody()); | |||
} | |||
|
|||
} | |||
private static void continuousPaymentFlow(WalletApi walletApiInstance, PaymentApi paymentApi, String userAuthorizationId){ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Similar blocks of code found in 2 locations. Consider refactoring.
@@ -270,6 +297,36 @@ private static PaymentDetails createPayment(final PaymentApi apiInstance, String | |||
return result; | |||
} | |||
|
|||
private static PaymentDetails createContinuousPayment(final PaymentApi apiInstance, String merchantPaymentId, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Method createContinuousPayment
has 26 lines of code (exceeds 25 allowed). Consider refactoring.
@@ -106,6 +107,32 @@ private static void createAccountLinkQrCode(final PaymentApi apiInstance){ | |||
System.out.println(e.getResponseBody()); | |||
} | |||
|
|||
} | |||
private static void continuousPaymentFlow(WalletApi walletApiInstance, PaymentApi paymentApi, String userAuthorizationId){ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Method continuousPaymentFlow
has a Cognitive Complexity of 7 (exceeds 5 allowed). Consider refactoring.
@@ -142,15 +146,20 @@ private static void appInvokeFlow(final PaymentApi paymentApi, final WalletApi w | |||
} | |||
} | |||
|
|||
private static void directDebitFlow(WalletApi walletApiInstance, PaymentApi paymentApi, String userAuthorizationId){ | |||
private static void directDebitFlow(WalletApi walletApiInstance, PaymentApi paymentApi, String userAuthorizationId, boolean continuousPayment){ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Method directDebitFlow
has 27 lines of code (exceeds 25 allowed). Consider refactoring.
All Submissions:
New Feature Submissions:
Changes to Core Features: