[master] Description Field value gets automatically reverted the original value after changing it.

[neeleshsinghal]

Issue: After validating Quantity on a sales line, editing the Description reverts it to the original item value when "Calc. Inv. Discount" is enabled (Bug 638838).

Cause: In CalculateSalesSubPageTotals, NeedRefreshSalesLine := false; was reset after the if TotalsUpToDate then exit; early-exit, so the flag set true by the Quantity validation stayed stuck, and RefreshSalesLine's Rec.Find() reloaded the line from the DB, discarding the uncommitted Description edit.

Solution: Moved NeedRefreshSalesLine := false; before the early-exit, so when totals are already up-to-date (no recalculation), the flag is cleared and RefreshSalesLine no longer overwrites the user's edit.

Workitem: Bug 638838: [master] [all-e] RFH: [SaaS] Description Field value gets automatically reverted the original value after changing it.

Fixes AB#638838

[github-actions]

Copilot PR Review

Iteration 1 · Outcome: completed

All 16 sub-skills ran to completion (as completed or no-knowledge); one style finding surfaced. Self-review pass considered one cross-cutting candidate (event var-parameter behavior change) but did not emit it because it depends on external subscriber code not visible in this repository, per the DO precision bar.

Knowledge source: https://github.com/microsoft/BCQuality@186d8a131465475c79244d994acb872cd5c0d4bf

Findings by domain

Findings split into Knowledge-backed (cite a BCQuality article) and Agent (the agent's own judgement, no matching BCQuality rule).

Domain	Findings	Knowledge-backed	Agent	Inline	Fallback
Style	1	1	0	1	0

Totals: 1 knowledge-backed · 0 agent findings.

Orchestrator pre-filter (2 file(s) excluded)

• layer-disabled (knowledge) : 2 file(s)

_{Findings produced by the AL review agent v1.7.3. Reply 👎 on any inline comment to flag false positives.}