Skip to content

Validation + UI update when trying to create Quality Inspection Results records with empty fields. #6805

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
6CRIPT wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Validation + UI update when trying to create Quality Inspection Results records with empty fields. #6805

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
9455bb8
Now the fields "Code" and "Description" have a red asterix showing th…
6CRIPT Feb 23, 2026
1ae7af4
TestField removed from the table
6CRIPT Feb 24, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 3 additions & 0 deletions src/Apps/W1/Quality Management/app/src/Configuration/Result/QltyInspectionResultList.Page.al
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,7 @@ page 20416 "Qlty. Inspection Result List"
SourceTable = "Qlty. Inspection Result";
SourceTableView = sorting("Evaluation Sequence");
PageType = List;
DelayedInsert = true;
Copy link
Contributor

@AndreiPanko AndreiPanko Feb 25, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

is it needed? I would not change this setting.

Copy link
Author

@6CRIPT 6CRIPT Feb 25, 2026 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

DelayedInsert = true; is used because if don't, the error coming from the testField("Description") will be thrown as soon as you click into the description field.

But as said, if you remove testField("Description"), which would make DelayedInsert = true; also not needed, then the application allows the user to create a rec without description.

image

Therefore DelayedInsert = true; is used here to avoid this error until the user actually try to save the changes; it will trigger then if the description field is empty.

ApplicationArea = QualityManagement;
UsageCategory = Lists;
AboutTitle = 'About Inspection Results';
Expand All @@ -26,9 +27,11 @@ page 20416 "Qlty. Inspection Result List"
{
field(Code; Rec.Code)
{
ShowMandatory = true;
}
field(Description; Rec.Description)
{
ShowMandatory = true;
}
field("Evaluation Sequence"; Rec."Evaluation Sequence")
{
Expand Down
Loading