[Bug Fix] ADO #624745: QM - Inspection Selection Criteria default value#7245
[Bug Fix] ADO #624745: QM - Inspection Selection Criteria default value#7245ventselartur wants to merge 1 commit intomainfrom
Conversation
ADO #624745: QM - Quality Management Setup. Quality Inspection Selection Criteria - default value
Root Cause:
- The "Inspection Selection Criteria" field in Qlty. Management Setup had no
InitValue, defaulting to enum value 0 ("Any inspection that matches") instead
of the desired "Only the newest inspection/re-inspection"
Changes:
- Added InitValue = "Only the newest inspection/re-inspection" to field 28 in
QltyManagementSetup.Table.al
Test Coverage:
- Added test codeunit 139973 "Qlty. Tests - Setup" with 2 tests
- Fix iterations: 1
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
|
Could not find a linked ADO work item. Please link one by using the pattern 'AB#' followed by the relevant work item number. You may use the 'Fixes' keyword to automatically resolve the work item when the pull request is merged. E.g. 'Fixes AB#1234' |
|
@ventselartur please read the following Contributor License Agreement(CLA). If you agree with the CLA, please reply with the following information.
Contributor License AgreementContribution License AgreementThis Contribution License Agreement (“Agreement”) is agreed to by the party signing below (“You”),
|
| QltyManagementSetup: Record "Qlty. Management Setup"; | ||
| QltyInspSelectionCriteria: Enum "Qlty. Insp. Selection Criteria"; | ||
| begin | ||
| // [FEATURE] [AI test 0.3] |
|
Bug Reference
Azure DevOps: ADO #624745
Summary
Set the default
InitValuefor the "Inspection Selection Criteria" field in Quality Management Setup to "Only the newest inspection/re-inspection". Without this, the field defaults to "Any inspection that matches" (enum value 0), causing transactions to remain blocked even after a successful re-inspection.Root Cause
The
Inspection Selection Criteriafield (field 28) inQlty. Management Setuptable had noInitValueproperty. This caused the enum to default to its first value (0 = "Any inspection that matches"), which considers ALL matching inspections when blocking transactions — including older failed inspections that have already been superseded by a successful re-inspection.Changes Made
QltyManagementSetup.Table.al: AddedInitValue = "Only the newest inspection/re-inspection"to field 28 "Inspection Selection Criteria"Implementation Process
Test Evidence
Pre-Fix Test Results (Baseline)
Tests run before implementing the fix (expected to fail):
Post-Fix Test Results (Final)
Tests run after implementing the fix (expected to pass):
Total Tests Run: 2
All Tests Passing: ✅ Yes
Iteration Summary
InitValueto field definition → ✅ All tests passingTest Coverage
Testing Checklist
InitValueproperty additionReview Notes
InitValueonly affects new records. Existing tenants retain their current value.🤖 Generated with Claude Code using /bc-fix-bug