[jira] Created: (OFBIZ-2792) setPaymentStatus service returns incorrect oldStatusId

[Nicolas Malin (Jira)]

setPaymentStatus service returns incorrect oldStatusId
------------------------------------------------------

                 Key: OFBIZ-2792
                 URL: https://issues.apache.org/jira/browse/OFBIZ-2792
             Project: OFBiz
          Issue Type: Bug
          Components: accounting
    Affects Versions: Release Branch 9.04, SVN trunk
            Reporter: Anne Jessel
            Priority: Minor


As reported on mailing list, setPaymentStatus service sets the result field oldStatusId to payment.paymentId instead of payment.statusId, which is presumably incorrect. I will attach a patch to fix this.



--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

[Nicolas Malin (Jira)]

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

Anne Jessel updated OFBIZ-2792:
-------------------------------

    Attachment: paymentstatus.patch
                paymentstatus09.04.patch

Attached are patches for 9.04 (paymentstatus09.04.patch) and against trunk (paymentstatus.patch).

These have not been tested - I don't know how to test this change!

> setPaymentStatus service returns incorrect oldStatusId
> ------------------------------------------------------
>
>                 Key: OFBIZ-2792
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-2792
>             Project: OFBiz
>          Issue Type: Bug
>          Components: accounting
>    Affects Versions: Release Branch 9.04, SVN trunk
>            Reporter: Anne Jessel
>            Priority: Minor
>         Attachments: paymentstatus.patch, paymentstatus09.04.patch
>
>
> As reported on mailing list, setPaymentStatus service sets the result field oldStatusId to payment.paymentId instead of payment.statusId, which is presumably incorrect. I will attach a patch to fix this.

--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

[Nicolas Malin (Jira)]

    [ https://issues.apache.org/jira/browse/OFBIZ-2792?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12738805#action_12738805 ]

Sumit Pandit commented on OFBIZ-2792:
-------------------------------------

Hello Anne and Scott,
I have verified that it will not affect any part of system. Thanks for the patch.

> setPaymentStatus service returns incorrect oldStatusId
> ------------------------------------------------------
>
>                 Key: OFBIZ-2792
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-2792
>             Project: OFBiz
>          Issue Type: Bug
>          Components: accounting
>    Affects Versions: Release Branch 9.04, SVN trunk
>            Reporter: Anne Jessel
>            Priority: Minor
>         Attachments: paymentstatus.patch, paymentstatus09.04.patch
>
>
> As reported on mailing list, setPaymentStatus service sets the result field oldStatusId to payment.paymentId instead of payment.statusId, which is presumably incorrect. I will attach a patch to fix this.

--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

[Nicolas Malin (Jira)]

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

Adrian Crum reassigned OFBIZ-2792:
----------------------------------

    Assignee: Adrian Crum

> setPaymentStatus service returns incorrect oldStatusId
> ------------------------------------------------------
>
>                 Key: OFBIZ-2792
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-2792
>             Project: OFBiz
>          Issue Type: Bug
>          Components: accounting
>    Affects Versions: Release Branch 9.04, SVN trunk
>            Reporter: Anne Jessel
>            Assignee: Adrian Crum
>            Priority: Minor
>         Attachments: paymentstatus.patch, paymentstatus09.04.patch
>
>
> As reported on mailing list, setPaymentStatus service sets the result field oldStatusId to payment.paymentId instead of payment.statusId, which is presumably incorrect. I will attach a patch to fix this.

--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

[Nicolas Malin (Jira)]

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

Adrian Crum closed OFBIZ-2792.
------------------------------

    Resolution: Fixed

Thanks Anne!

Fixed, trunk rev 800846, R9.04 rev 800848.

> setPaymentStatus service returns incorrect oldStatusId
> ------------------------------------------------------
>
>                 Key: OFBIZ-2792
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-2792
>             Project: OFBiz
>          Issue Type: Bug
>          Components: accounting
>    Affects Versions: Release Branch 9.04, SVN trunk
>            Reporter: Anne Jessel
>            Assignee: Adrian Crum
>            Priority: Minor
>         Attachments: paymentstatus.patch, paymentstatus09.04.patch
>
>
> As reported on mailing list, setPaymentStatus service sets the result field oldStatusId to payment.paymentId instead of payment.statusId, which is presumably incorrect. I will attach a patch to fix this.

--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.