Skip to content

Commit

Permalink
Page summary telemetry - fix wrong condition (#2055)
Browse files Browse the repository at this point in the history
<!-- Thank you for submitting a Pull Request. If you're new to
contributing to BCApps please read our pull request guideline below
* https://github.com/microsoft/BCApps/Contributing.md
-->
#### Summary Fix wrong telemetry on the Page Summary Provider

#### Work Item(s) <!-- Add the issue number here after the #. The issue
needs to be open and approved. Submitting PRs with no linked issues or
unapproved issues is highly discouraged. -->
Fixes
[AB#549213](https://dynamicssmb2.visualstudio.com/1fcb79e7-ab07-432a-a3c6-6cf5a88ba4a5/_workitems/edit/549213)
  • Loading branch information
nikolakukrika authored Sep 17, 2024
1 parent dae90ad commit 32b5663
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -195,7 +195,7 @@ codeunit 2717 "Page Summary Provider Impl."
local procedure GetRecordFields(PageId: Integer; Bookmark: Text; var ResultJsonObject: JsonObject)
begin
// Get all visible and available table fields that back the controls that are visible on the page
if TryGetAvailableRecordFieldsData(PageId, Bookmark, ResultJsonObject) then
if not TryGetAvailableRecordFieldsData(PageId, Bookmark, ResultJsonObject) then
Session.LogMessage('0000NFZ', StrSubstNo(GetRecordFieldsFailureTelemetryTxt, PageId), Verbosity::Error, DataClassification::SystemMetadata, TelemetryScope::ExtensionPublisher, 'Category', PageSummaryCategoryLbl);
exit;
end;
Expand Down

0 comments on commit 32b5663

Please sign in to comment.