Skip to content

[28.0] [Quality Mgmt.] aligning the Deletion UX of Qlty Test with standard BC: delete from c… #7014

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
attilatoury wants to merge 1 commit into
base: releases/28.0
Choose a base branch
from
+2 −57
Open

[28.0] [Quality Mgmt.] aligning the Deletion UX of Qlty Test with standard BC: delete from c… #7014

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
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
2 changes: 1 addition & 1 deletion src/Apps/W1/Quality Management/app/src/Configuration/Template/Test/QltyTest.Table.al
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -366,7 +366,7 @@ table 20401 "Qlty. Test"

trigger OnDelete()
begin
CheckDeleteConstraints(false);
CheckDeleteConstraints(true);
end;

procedure CheckDeleteConstraints(AskQuestion: Boolean)
Expand Down
26 changes: 0 additions & 26 deletions src/Apps/W1/Quality Management/app/src/Configuration/Template/Test/QltyTestCard.Page.al
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,6 @@ page 20479 "Qlty. Test Card"
Caption = 'Quality Test';
AboutTitle = 'About Quality Test details';
AboutText = 'Use this page to define questions, measurements, allowed values, and default passing conditions. Add tests to templates to use them in quality inspections.';
DeleteAllowed = false;
PageType = Card;
SourceTable = "Qlty. Test";
SourceTableView = sorting(Code);
Expand Down Expand Up @@ -545,32 +544,7 @@ page 20479 "Qlty. Test Card"
}
}

actions
{
area(Processing)
{
action(DeleteRecordSafe)
{
Caption = 'Delete';
Image = Delete;
ToolTip = 'Deletes this test. A test can only be deleted if it is not being used on an existing inspection.';

trigger OnAction()
begin
Rec.CheckDeleteConstraints(true);
Rec.Delete(true);
CurrPage.Update(false);
end;
}
}

area(Promoted)
{
actionref(DeleteRecordSafe_Promoted; DeleteRecordSafe)
{
}
}
}

var
QltyResultConditionMgmt: Codeunit "Qlty. Result Condition Mgmt.";
Expand Down
31 changes: 1 addition & 30 deletions src/Apps/W1/Quality Management/app/src/Configuration/Template/Test/QltyTests.Page.al
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,6 @@ page 20401 "Qlty. Tests"
CardPageId = "Qlty. Test Card";
Editable = false;
InsertAllowed = false;
DeleteAllowed = false;
RefreshOnActivate = true;
PageType = List;
SourceTable = "Qlty. Test";
Expand Down Expand Up @@ -219,32 +218,7 @@ page 20401 "Qlty. Tests"
}
}

actions
{
area(Processing)
{
action(DeleteRecordSafe)
{
Caption = 'Delete';
Image = Delete;
Scope = Repeater;
ToolTip = 'Deletes this test. A test can only be deleted if it is not being used on an existing inspection.';

trigger OnAction()
begin
Rec.CheckDeleteConstraints(true);
Rec.Delete(true);
CurrPage.Update(false);
end;
}
}
area(Promoted)
{
actionref(DeleteRecordSafe_Promoted; DeleteRecordSafe)
{
}
}
}

var
QltyResultConditionMgmt: Codeunit "Qlty. Result Condition Mgmt.";
Expand All @@ -270,10 +244,7 @@ page 20401 "Qlty. Tests"
begin
end;

trigger OnDeleteRecord(): Boolean
begin
Rec.CheckDeleteConstraints(true);
end;


trigger OnAfterGetRecord()
begin
Expand Down
Loading