Hacktoberfest 2026: the issues maintainers tagged for October, open and beginner-friendly. Browse Hacktoberfest issues

[EventRequest][W1][codeunit][426][Payment Tolerance Management] New event OnBeforePmtTolCBGJnl in procedure PmtTolCBGJnl

Open Beginner friendly
  • Is something wrong?
#12,900 1 comment 0 reactions 0 assignees View on GitHub

Maintainers usually reply within 1 day

Nobody has claimed this yet.

Assessment

Difficulty
2/5
Estimated time
1-3 hours
Newbie friendliness
62/100
Issue type
Feature
Clarity
Clearly specified
Activity status
Active
Domain
backend

Research direction

The change belongs in codeunit 426 "Payment Tolerance Management", procedure PmtTolCBGJnl, which the issue quotes in full. Study how other integration events in that codeunit are declared and called with the Result and IsHandled pattern, then add the OnBeforePmtTolCBGJnl local integration event and call it at the top of the procedure. Done when a subscriber can set IsHandled and skip the remaining logic, and the existing tests still pass.

Written by the indexing model from the issue text.

Description

missing-info Team: Finance
Why do you need this change?

In 4PS Construct we need to skip execution of procedure PmtTolCBGJnl if specific conditions are met. In this case we implement our custom logic.

Describe the request

Dear ALAppExtensions team,
On behalf of 4PS I would like to request integration event 'OnBeforePmtTolCBGJnl' to be added to procedure PmtTolCBGJnl of codeunit 426 "Payment Tolerance Management".

  procedure PmtTolCBGJnl(var CBGStatementLine: Record "CBG Statement Line"): Boolean
    var
        GLSetup: Record "General Ledger Setup";
        Customer: Record Customer;
        Vendor: Record Vendor;
        NewCustLedgEntry: Record "Cust. Ledger Entry";
        NewVendLedgEntry: Record "Vendor Ledger Entry";
        AppliedAmount: Decimal;
        OriginalAppliedAmount: Decimal;
        ApplyingAmount: Decimal;
        AmounttoApply: Decimal;
        PmtDiscAmount: Decimal;
        MaxPmtTolAmount: Decimal;
        CBGStatementLineApplID: Code[20];
        ApplnRoundingPrecision: Decimal;
        CBGStatement: Record "CBG Statement";
        UseDocumentNo: Code[20];
        IsHandled: Boolean;
        Result: Boolean;
    begin
        OnBeforePmtTolCBGJnl(CBGStatementLine, Result, IsHandled);
        if IsHandled then
            exit(Result);
        MaxPmtTolAmount := 0;
        PmtDiscAmount := 0;
        AppliedAmount := 0;
        ...
    end;
 
    [IntegrationEvent(false, false)]
    local procedure OnBeforePmtTolCBGJnl(var CBGStatementLine: Record "CBG Statement Line"; var Result: Boolean; var IsHandled: Boolean)
    begin
    end;
Provide an implementation (optional)
  • I will provide the implementation for this extensibility request
Dominant language
AL
Stars
697
Forks
473
Avg merge
2d 19h
Merged PRs (30d)
493

Getting set up

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from microsoft/BCApps

All issues in microsoft/BCApps

Similar issues

More Backend & API Design issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.