svn propchange: r1804843 - svn:log

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

svn propchange: r1804843 - svn:log

jleroux-2
Author: jleroux
Revision: 1804843
Modified property: svn:log

Modified: svn:log at Fri Aug 11 19:33:38 2017
------------------------------------------------------------------------------
--- svn:log (original)
+++ svn:log Fri Aug 11 19:33:38 2017
@@ -1,13 +1,10 @@
-Fixed: [FB] Package org.apache.ofbiz.accounting.payment (Additional Bugs)
-(OFBIZ-9529)
+(OFBIZ-9530)
 
-This patch resolves bugs from GiftCertificateServices
+In the method processAuthResult() line 1889 there is an empty if-statement for
+the case that authResult is null. It seems like somebody was planning to cover
+the case but forgot to add code.
 
-PaymentGatewayServices.java:1889, UCF_USELESS_CONTROL_FLOW
-
-UCF: Useless control flow in
-PaymentGatewayServices.processAuthResult(DispatchContext, Map)
-This method contains a useless control flow statement, where control flow continues onto
-the same place regardless of whether or not the branch is taken.
+I would suggest to return a service error when the authResult is null since
+otherwise it cannot be determined if the payment is authorized or not.
 
 Thanks: Kyra Pritzel-Hentley