ofbiz-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Kyra Pritzel-Hentley (JIRA)" <j...@apache.org>
Subject [jira] [Updated] (OFBIZ-9530) Missing action in empty if-statement in accounting.PaymentGatewayServices
Date Mon, 31 Jul 2017 11:06:02 GMT

     [ https://issues.apache.org/jira/browse/OFBIZ-9530?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]

Kyra Pritzel-Hentley updated OFBIZ-9530:
----------------------------------------
    Attachment: OFBIZ-9530_PaymentGatewayServices_bugfix.patch

Hello Jacques,

thank you for your feedback. I added an UiLabel which is used as the serviceError message.


> Missing action in empty if-statement in accounting.PaymentGatewayServices
> -------------------------------------------------------------------------
>
>                 Key: OFBIZ-9530
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-9530
>             Project: OFBiz
>          Issue Type: Bug
>          Components: accounting
>    Affects Versions: Trunk
>            Reporter: Kyra Pritzel-Hentley
>            Assignee: Jacques Le Roux
>            Priority: Minor
>         Attachments: OFBIZ-9530_PaymentGatewayServices_bugfix.patch, OFBIZ-9530_PaymentGatewayServices_bugfix.patch
>
>
> 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.
> 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.
> {code:java}
> boolean authResultOk = authResult.booleanValue();
>             
>             if (authResultOk) {
>                 orderPaymentPreference.set("statusId", "PAYMENT_AUTHORIZED");
>             } else {
>                 orderPaymentPreference.set("statusId", "PAYMENT_DECLINED");
>             }
> {code}



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Mime
View raw message