-
Notifications
You must be signed in to change notification settings - Fork 278
Labels
ApprovedThe issue is approvedThe issue is approvedFinanceGitHub request for Finance areaGitHub request for Finance area
Description
Describe the issue
The paramter SalesLine in the event OnAfterCreateAdditionalInvoiceLine is currently not useable, since the SalesLine is not the new created additional sales line.
Expected behavior
The SalesLine in the event should be the the newly created Attached Text Sales Line.
Steps to reproduce
- Prepare the subscription contract setup
- Create a customer contract
- Create a sales invoice
Additional context
local procedure CreateAdditionalInvoiceLine(ServiceContractSetupFieldNo: Integer; SalesHeader2: Record "Sales Header"; ParentSalesLine: Record "Sales Line"; ServiceObject: Record "Subscription Header"; ServiceCommitment: Record "Subscription Line")
var
SalesLine: Record "Sales Line";
DescriptionText: Text;
begin
DescriptionText := GetAdditionalLineText(ServiceContractSetupFieldNo, ParentSalesLine, ServiceObject, ServiceCommitment);
if DescriptionText = '' then
exit;
SalesLine.InsertDescriptionSalesLine(SalesHeader2, DescriptionText, ParentSalesLine."Line No.");
OnAfterCreateAdditionalInvoiceLine(SalesLine, ParentSalesLine);
end;BCApps/src/Apps/W1/Subscription Billing/App/Billing/Codeunits/CreateBillingDocuments.Codeunit.al
Line 952 in 73751fc
| OnAfterCreateAdditionalInvoiceLine(SalesLine, ParentSalesLine); |
I'd like to InsertDescriptionSalesLine to a new procedure that uses the Rec, instead of a local sales line parameter.
// Replace
procedure InsertDescriptionSalesLine(SourceSalesHeader: Record "Sales Header"; NewDescription: Text; AttachedToLineNo: Integer)
var
SalesLine: Record "Sales Line";
begin
SalesLine.InitFromSalesHeader(SourceSalesHeader);
SalesLine."Attached to Line No." := AttachedToLineNo;
SalesLine.Description := CopyStr(NewDescription, 1, MaxStrLen(SalesLine.Description));
SalesLine.Insert(false);
end;
// with:
procedure CreateAttachedSalesLine(SourceSalesHeader: Record "Sales Header"; NewDescription: Text; AttachedToLineNo: Integer)
begin
Rec.InitFromSalesHeader(SourceSalesHeader);
Rec."Attached to Line No." := AttachedToLineNo;
Rec.Description := CopyStr(NewDescription, 1, MaxStrLen(SalesLine.Description));
Rec.Insert(false);
end;I will provide a fix for a bug
- I will provide a fix for a bug
Metadata
Metadata
Assignees
Labels
ApprovedThe issue is approvedThe issue is approvedFinanceGitHub request for Finance areaGitHub request for Finance area