Skip to content
This repository was archived by the owner on Oct 19, 2026. It is now read-only.
This repository was archived by the owner on Oct 19, 2026. It is now read-only.

[Events Request] Report 952 "Suggest Job Jnl. Lines" ref. #30419 #30431

Closed

Description

@AlbBard

Why do you need this change?

The #30419 was partially carried out.

Describe the request

In my original request (ref. #30419) I asked for 2 new event in this mode:
[IntegrationEvent(true, true)]

but you created:
[IntegrationEvent(false, false)]

I need to extend standard report to add a new protected global variable and I need to see it in the events. If you do not pass at the event the sender and the global variables I can not carry out the customization.

Please take note and correct it.

Thank you
Internal work item: AB#647662

Activity

  1. added theissue type on Aug 24, 2026
  2. github-actions commented on Aug 24, 2026

    @github-actions
    Contributor

    ✅ Analysis complete - ready for further processing


    Implementation:

    Change the two existing integration events in report 952 "Suggest Job Jnl. Lines" to use [IntegrationEvent(true, true)] so the sender and protected globals are available in event subscribers.

    Location in report 952 "Suggest Job Jnl. Lines" (event declarations):

            if ResourcesSetup."Incl. Time Sheet Date in Jnl." then
                JobJnlLine.Description := CopyStr(StrSubstNo(TimeSheetDescriptionWithDateTok, TimeSheetDetail.Date, TimeSheetDescription), 1, MaxStrLen(JobJnlLine.Description))
            else
                JobJnlLine.Description := CopyStr(TimeSheetDescription, 1, MaxStrLen(JobJnlLine.Description));
        end;
    
        [IntegrationEvent(true, true)]
        local procedure OnBeforeInsertTempTimeSheetLine(JobJournalLine: Record "Job Journal Line"; TimeSheetHeader: Record "Time Sheet Header"; var TempTimeSheetLine: Record "Time Sheet Line" temporary; var SkipLine: Boolean)
        begin
        end;
    
        [IntegrationEvent(true, true)]
        local procedure OnAfterTransferTimeSheetDetailToJobJnlLine(var JobJournalLine: Record "Job Journal Line"; JobJournalTemplate: Record "Job Journal Template"; var TempTimeSheetLine: Record "Time Sheet Line" temporary; TimeSheetDetail: Record "Time Sheet Detail"; JobJournalBatch: Record "Job Journal Batch"; var LineNo: Integer)
        begin
        end;
    
        [IntegrationEvent(false, false)]
        local procedure OnOnPostReportOnTempTimeSheetLineEndLoop(var JobJournalLine: Record "Job Journal Line"; var NextDocNo: Code[20]; var LineNo: Integer)
        begin

    Event Publisher (add at the end of the codeunit with other events):

    [IntegrationEvent(true, true)]
    local procedure OnBeforeInsertTempTimeSheetLine(JobJournalLine: Record "Job Journal Line"; TimeSheetHeader: Record "Time Sheet Header"; var TempTimeSheetLine: Record "Time Sheet Line" temporary; var SkipLine: Boolean)
    begin
    end;
    
    [IntegrationEvent(true, true)]
    local procedure OnAfterTransferTimeSheetDetailToJobJnlLine(var JobJournalLine: Record "Job Journal Line"; JobJournalTemplate: Record "Job Journal Template"; var TempTimeSheetLine: Record "Time Sheet Line" temporary; TimeSheetDetail: Record "Time Sheet Detail"; JobJournalBatch: Record "Job Journal Batch"; var LineNo: Integer)
    begin
    end;

    File: SuggestJobJnlLines.Report.al

    Note

    Automatically generated by the Triage Agent — advisory only. An engineer will review before any action is taken.

    Tip

    Not accurate? Reply with /not-accurate (optional: add a short explanation). Your feedback improves future responses.

  3. AleksandricMarko commented on Sep 18, 2026

    @AleksandricMarko
    Contributor

    Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
    PR: microsoft/BCApps#10900

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

Metadata

Metadata

Assignees

No one assigned

    Labels

    SCMGitHub request for SCM areaevent-requestRequest for adding an eventext-ready-to-implementReviewed and ready to implement and create PR

    Type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions