Skip to content

Fixes #8156#8130

Open
FrankNeeckx wants to merge 3 commits into
microsoft:mainfrom
FrankNeeckx:patch-2
Open

Fixes #8156#8130
FrankNeeckx wants to merge 3 commits into
microsoft:mainfrom
FrankNeeckx:patch-2

Conversation

@FrankNeeckx
Copy link
Copy Markdown

@FrankNeeckx FrankNeeckx commented May 13, 2026

Hi,
If at least one security filter is applied to the “Purchase Header” table, the user will have a problem when push ‘Add file from source document’ action on the “Email Attachments” subpage. Message: "Did not find any attachments related to this email"

With this event, we will be able to add the following condition: SourceRecordRef.SecurityFiltering(SecurityFilter::Ignored); before reading permissions.

Thx in advance,
Frank Neeckx

What & why

Linked work

Fixes #8156

How I validated this

  • I read the full diff and it contains only changes I intended.
  • I built the affected app(s) locally with no new analyzer warnings.
  • I ran the change in Business Central and confirmed it behaves as expected.
  • I added or updated tests for the new behavior, or explained below why none are needed.

What I tested and the outcome (required — be specific: scenarios, commands, screenshots for UI changes)

Risk & compatibility

Hi,
If at least one security filter is applied to the “Purchase Header” table, the user will have a problem when push ‘Add file from source document’ action on the “Email Attachments” subpage. Message: "Did not find any attachments related to this email"

With this event, we will be able to add the following condition:
SourceRecordRef.SecurityFiltering(SecurityFilter::Ignored);
before reading permissions.

Thx in advance,
Frank Neeckx
@FrankNeeckx FrankNeeckx requested a review from a team as a code owner May 13, 2026 12:58
@FrankNeeckx FrankNeeckx requested a review from msft-sam May 13, 2026 12:58
@github-actions github-actions Bot added AL: System Application From Fork Pull request is coming from a fork labels May 13, 2026
@FrankNeeckx
Copy link
Copy Markdown
Author

Hi,
Do we need to create a separate issue, for the BE localization, even though its a system app modification?

Thx
Frank Neeckx

@github-actions
Copy link
Copy Markdown
Contributor

Issue #30057 is not valid. Please make sure you link an issue that exists, is open and is approved.

Comment thread src/System Application/App/Email/src/Email/EmailImpl.Codeunit.al Outdated
Comment thread src/System Application/App/Email/src/Email/EmailImpl.Codeunit.al Outdated
FrankNeeckx and others added 2 commits May 13, 2026 16:47
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@JesperSchulz JesperSchulz added the Integration GitHub request for Integration area label May 18, 2026
@FrankNeeckx FrankNeeckx changed the title Fixes #30057 Fixes #8156 May 19, 2026
@github-actions
Copy link
Copy Markdown
Contributor

Issue #8156 is not valid. Please make sure you link an issue that exists, is open and is approved.

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

Labels

AL: System Application From Fork Pull request is coming from a fork Integration GitHub request for Integration area

Projects

None yet

2 participants