[Quality Management] Fix missing validation of "Allowable Values" on Quality Test card

[JakovljevicDusan]

What & why

Linked work

Fixes AB#624240

[github-actions]

$\textbf{🟡\ Medium\ Severity\ —\ Performance} \quad \color{gray}{\texttt{\small Iteration\ 1}}$

Error() called inside [TryFunction] with empty placeholders

TryApplyBooleanFilter is marked [TryFunction] but raises Error() explicitly with empty-string placeholders ('' for field name and literal 'Boolean' for type). While AL [TryFunction] does catch explicit Error() calls and returns false, the inner error message is set with missing context, and the outer ValidateAllowableValuesFormat then raises its own error — meaning two error messages are generated internally before the user-visible one, which can confuse debugging and telemetry.

Recommendation:

• Remove the explicit Error() from inside TryApplyBooleanFilter and instead return without error from the try function; let the caller (ValidateAllowableValuesFormat) raise the user-visible error with full context. Alternatively, validate boolean values in a regular (non-try) procedure.

Suggested change

	local procedure TryApplyBooleanFilter(FilterExpression: Text)
	[TryFunction]
	local procedure TryApplyBooleanFilter(FilterExpression: Text)
	var
	QltyBooleanParsing: Codeunit "Qlty. Boolean Parsing";
	begin
	if not QltyBooleanParsing.CanTextBeInterpretedAsBooleanIsh(FilterExpression) then
	Error('');
	end;

_{👍 useful · ❤️ especially valuable · 👎 wrong - reply with why}