Skip to content

Reintroduce direct historical line matching for E-Document drafts#7602

Open
mynjj wants to merge 1 commit intomainfrom
bugs/edocs-reintroducehistoricalmapping
Open

Reintroduce direct historical line matching for E-Document drafts#7602
mynjj wants to merge 1 commit intomainfrom
bugs/edocs-reintroducehistoricalmapping

Conversation

@mynjj
Copy link
Copy Markdown
Contributor

@mynjj mynjj commented Apr 8, 2026

Summary

  • Wire up FindRelatedPurchaseLineInHistory (previously dead code) as a direct matching step before the AI pipeline in E-Doc. Historical Matching
  • Sets E-Doc. Purch. Line History Id on matched lines, enabling the source invoice link and additional fields features
  • If all lines resolve via direct history, the AI pipeline is skipped entirely

Fixes AB#630431

@mynjj mynjj requested a review from a team as a code owner April 8, 2026 13:23
@github-actions github-actions bot added the AL: Apps (W1) Add-on apps for W1 label Apr 8, 2026
@mynjj mynjj requested a review from a team April 8, 2026 14:33
@mynjj mynjj enabled auto-merge (squash) April 8, 2026 14:33
@github-actions github-actions bot added this to the Version 29.0 milestone Apr 8, 2026
if VendorNo = '' then
exit(false);

if EDocumentPurchaseLine.FindSet() then
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It looks like you did the FindFirst above, in the very beginning, already. It is possible to have a single SELECT request to the e-document purchase line table instead of two?

@mynjj mynjj requested a review from Groenbech96 April 9, 2026 10:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

AL: Apps (W1) Add-on apps for W1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants