Skip to content

[Extensibility Request] issue 30334: fix OnAfterCarryOutToReqWksh record order#9449

Draft
AleksandricMarko wants to merge 1 commit into
mainfrom
ai-ext-fix/ext_issue-30334
Draft

[Extensibility Request] issue 30334: fix OnAfterCarryOutToReqWksh record order#9449
AleksandricMarko wants to merge 1 commit into
mainfrom
ai-ext-fix/ext_issue-30334

Conversation

@AleksandricMarko

@AleksandricMarko AleksandricMarko commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

Summary

A subscriber to the OnAfterCarryOutToReqWksh integration event in codeunit 99000813 "Carry Out Action" needs to update the newly created target requisition line based on the source line it was copied from. The event's first parameter is passed by var precisely so subscribers can modify that record, but in version 28.1 the call was changed to pass the source line as the first argument and the target line by value, removing any way to persist changes to the target line. This regression blocks the reporter's process. This PR restores the previous argument order so the modifiable var parameter is once again the target requisition line.

Source issue repository: microsoft/ALAppExtensions; issue number: 30334

Changes Made

  • Carry Out Action.CarryOutToReqWksh - restored the OnAfterCarryOutToReqWksh call to pass the target requisition line as the first (var) argument and the source line second, so subscribers can again modify and persist the target line; applied in both the W1 and NA layers.
  • Mfg. Carry Out Action.OnAfterCarryOutToReqWksh - updated the subscriber to read the source worksheet identifiers from the second parameter to match the restored argument order, keeping planning routing line and capacity need propagation correct.

Fixes AB#641682

… subscribers can modify the target requisition line

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@AleksandricMarko AleksandricMarko added event-request SCM GitHub request for SCM area labels Jul 14, 2026
@github-actions github-actions Bot added this to the Version 29.0 milestone Jul 14, 2026
@github-actions

Copy link
Copy Markdown
Contributor

⚠️ Stale Status Check Deleted

The Pull Request Build workflow run for this PR was older than 72 hours and has been deleted.

📋 Why was it deleted?

Status checks that are too old may no longer reflect the current state of the target branch. To ensure this PR is validated against the latest code and passes up-to-date checks, a fresh build is required.


🔄 How to trigger a new status check:

  1. 📤 Push a new commit to the PR branch, or
  2. 🔁 Close and reopen the PR

This will automatically trigger a new Pull Request Build workflow run.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

event-request SCM GitHub request for SCM area

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant