[28.x] Classify restored E-Document table sensitivities - #11278
[28.x] Classify restored E-Document table sensitivities#11278Milica Đukić (djukicmilica) wants to merge 1 commit into
Conversation
|
Could not find linked issues in the pull request description. Please make sure the pull request description contains a line that contains 'Fixes #' followed by the issue number being fixed. Use that pattern for every issue you want to link. |
Good Sense Reviewer - Round 1Recommendation: AcceptWhat this PR doesThis change adds the three restored E-Document message tables to the data sensitivity initialization list in EDocumentSubscribers.Codeunit.al. The subscriber already classifies sensitive E-Document fields through DataClassificationEvalData.SetTableFieldsToNormal; adding tables 6432, 6433, and 6434 makes the restored CustomerContent fields use the same path. I checked the surrounding method and the analogous PEPPOL subscriber pattern; the added calls are in the right place and no follow-on code changes the classification result. Problem-solution fitFit: Strong The reported failure is an unclassified sensitive field on a restored E-Document table. The diff covers that table and the two related message/reference tables in the same message area, so the solution matches the validation issue without adding runtime behavior. SuggestionsNone. Risk assessment and necessityRisk: The regression surface is narrow: company data sensitivity setup for the E-Document app. The change does not alter posting, payment calculation, data writes, public APIs, or event signatures. The changed code only extends an existing classification list, and the current PR checks show the relevant W1 build and test jobs passing. Necessity: The change is needed because restored E-Document message data left sensitive fields unclassified during validation. The scope is right: all three restored message-area tables are included, and no unrelated E-Document tables or behavior are changed.
|
Why
The restored E-Document message infrastructure introduces tables whose runtime data sensitivity remains unclassified. This causes
Data Classs Demo Data Tests.TestDataSensitivitiesto fail across localization test buckets.This backports the classifications from #11170 and #11276 to
releases/28.x.Summary
Fixes
AB#648705
AB#648955