Skip to content
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

[Event Change Request] Codeunit 80 Sales Post - procedure PostUpdateReturnReceiptLine - Event OnAfterPostUpdateReturnReceiptLine #26197

Open
MH1612 opened this issue Mar 27, 2024 · 4 comments
Labels
event-request Request for adding an event SCM GitHub request for SCM area

Comments

@MH1612
Copy link

MH1612 commented Mar 27, 2024

Describe the request

We would like var SalesHeader added to the following event.

Original:

Codeunit 80 Sales Post - procedure PostUpdateReturnReceiptLine - Event OnAfterPostUpdateReturnReceiptLine

    local procedure PostUpdateReturnReceiptLine()
    var
        SalesOrderLine: Record "Sales Line";
        ReturnRcptLine: Record "Return Receipt Line";
        TempSalesLine: Record "Sales Line" temporary;
        IsHandled: Boolean;
    begin

    ...

    OnAfterPostUpdateReturnReceiptLine(TempSalesLine);

    [IntegrationEvent(true, false)]
    local procedure OnAfterPostUpdateReturnReceiptLine(var TempSalesLine: Record "Sales Line" temporary)
    begin
    end;

Change:

Codeunit 80 - Sales Post - procedure PostUpdateReturnReceiptLine - Event OnAfterPostUpdateReturnReceiptLine

    local procedure PostUpdateReturnReceiptLine(var SalesHeader: Record "Sales Header")
    // similar to PostUpdateInvoiceLine
    var
        SalesOrderLine: Record "Sales Line";
        ReturnRcptLine: Record "Return Receipt Line";
        TempSalesLine: Record "Sales Line" temporary;
        IsHandled: Boolean;
    begin

    ...

    OnAfterPostUpdateReturnReceiptLine(TempSalesLine, SalesHeader);

    [IntegrationEvent(true, false)]
    procedure OnAfterPostUpdateReturnReceiptLine(var TempSalesLine: Record "Sales Line" temporary; var SalesHeader: Record "Sales Header")
    begin
    end;

Additional context

Post Invoice with Return Receipts
Internal work item: AB#523918

@JesperSchulz JesperSchulz added event-request Request for adding an event SCM GitHub request for SCM area labels Apr 3, 2024
@tomasevicst
Copy link

In what localization we have SalesHeader parameter in procedure PostUpdateReturnReceiptLine?

@MH1612
Copy link
Author

MH1612 commented Apr 10, 2024

@tomasevicst
It relates to this request: #26195

image

@AleksandricMarko
Copy link

Hello @MH1612
Related to a request #26195 - it is completed with PostUpdateReturnReceiptLine become public.
So, does the current request still have a purpose?

@SKeiler
Copy link

SKeiler commented Jul 17, 2024

Hi @AleksandricMarko,

yes, the current request is still needed. We still need the Sales Header parameter in the OnAfterPostUpdateReturnReceiptLine-Event.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
event-request Request for adding an event SCM GitHub request for SCM area
Projects
None yet
Development

No branches or pull requests

5 participants