[Bug]: HideDialog is not transferred to codeunit 452 "Report Distribution Management"
Nobody has claimed this yet.
Assessment
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Newbie friendliness
- 72/100
- Issue type
- Bug
- Clarity
- Clearly specified
- Activity status
- Quiet
- Domain
- backend
Research direction
Start in src/Layers/W1/BaseApp/Foundation/Reporting/DocumentSendingProfile.Table.al and follow the Electronic Document branch into ReportDistributionManagement.Codeunit.al. Inspect how ShowDialog and SendXmlEmailAttachment are handled, then reproduce the documented Document Sending Profile scenario. Done means sending the sales invoice without dialog no longer shows the e-mail message dialog.
Written by the indexing model from the issue text.
Description
Describe the issue
If a Document Sending Profile is configured for E-Mail attachment Electronic Document and the sending is executed without dialog then the e-mail dialog is still shown.
Expected behavior
No E-mail message dialog should be shown.
Steps to reproduce
Configure an Electronic Document Format.
Configure a Document Sending Profile with "E-Mail Attachment":: "Electronic Document" and "E-Mail"::"Yes (Use Default Settings)".
Set this document sending profiel for a customer
Then try to send e.g. a Sales Invoice with this format.
Now the e-mail dialog is shown although it shouldn't be shown.
Additional context
Suggest change:
"E-Mail Attachment"::"Electronic Document":
begin
ReportSelections.GetEmailBodyForCust(EmailBodyTempBlob, ReportUsage, RecordVariant, ToCust, SendToEmailAddress);
ReportDistributionManagement.InitializeFrom(not ShowDialog) // Line to be added
ReportDistributionManagement.SendXmlEmailAttachment(
RecordVariant, "E-Mail Format", EmailBodyTempBlob, SendToEmailAddress, ReportUsage);
end;
https://github.com/microsoft/BCApps/blob/main/src/Layers/W1/BaseApp/Foundation/Reporting/DocumentSendingProfile.Table.al
https://github.com/microsoft/BCApps/blob/main/src/Layers/W1/BaseApp/Foundation/Reporting/ReportDistributionManagement.Codeunit.al
I will provide a fix for a bug
- I will provide a fix for a bug
- Dominant language
- AL
- Stars
- 683
- Forks
- 459
- Avg merge
- 2d 20h
- Merged PRs (30d)
- 621
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from microsoft/BCApps
-
Team: Finance
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
-
Approved event-request ext-ready-to-implement Team: Finance
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
-
Approved event-request ext-ready-to-implement Team: SCM
Difficulty 1/5 Under an hour Newbie friendliness 90/100
-
event-request Team: SCM
Difficulty 2/5 1-3 hours Newbie friendliness 82/100
-
Team: SCM
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
All issues in microsoft/BCApps
Similar issues
-
bug clawsweeper:linked-pr-open clawsweeper:needs-live-repro clawsweeper:no-new-fix-pr impact:message-loss issue-rating: 🐚 platinum hermit P2 regression
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
-
Difficulty 1/5 Under an hour Newbie friendliness 90/100
AXERA-TECH/ax-llm#77 ·
-
Difficulty 1/5 Under an hour Newbie friendliness 90/100
games-on-whales/wolf#509 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
-
enhancement
Difficulty 2/5 1-3 hours Newbie friendliness 72/100