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 Request} Codeunit 205 - Procedure UpdateAltCustVATRegInSalesHeader #28008

Open
AndreBreitenbach opened this issue Jan 28, 2025 · 4 comments
Labels
event-request Request for adding an event SCM GitHub request for SCM area

Comments

@AndreBreitenbach
Copy link

AndreBreitenbach commented Jan 28, 2025

Describe the request

Please add a new Publisher OnAfterUpdateAltCustVATRegInSalesHeader here:

Image

Additional context

We plan additional fields in this process and need to handle and transfer them if filled.
Internal work item: AB#563818

@nikolakukrika
Copy link
Contributor

@AndreBreitenbach The event description is missing, which parameters do you need?

@nikolakukrika nikolakukrika added SCM GitHub request for SCM area event-request Request for adding an event labels Jan 28, 2025
@AndreBreitenbach
Copy link
Author

@nikolakukrika we need SalesHeader (as var) and AltCustVATReg.
OnAfterUpdateAltCustVATRegInSalesHeader(var SalesHeader: Record "Sales Header"; AltCustVATReg: Record "Alt. Cust. VAT Reg.");

@nikolakukrika
Copy link
Contributor

@AndreBreitenbach adding a record in the signature without var can be a performance hit if this code is invoked from the loop. Should we add a var to the second rec too? It would be a better event.

@AndreBreitenbach
Copy link
Author

@nikolakukrika its ok for me to add var to both parameters. In our case we only read the altcustvatreg-record.

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

2 participants