[6.0] SqlDecimal Extract Data

[benrr101]

Description

Due to legal reasons, we have to take a change to the SqlDecimal type workarounds in netfx.

Issues

N/A

Testing

This is a drop-in replacement for the existing method, CI should successfully verify.

[Copilot]

Pull Request Overview

Backport of the .NET 6.0 change to simplify how SqlDecimal data fields are extracted in the netfx build, replacing the old serialization-based decomposer with a direct call to SqlDecimal.Data.

• Removed complex serialization/DynamicMethod-based decomposer for SqlDecimal
• Added a straightforward SqlDecimalExtractData implementation using Data property
• Cleaned up unused usings and added XML doc comments and links

Comments suppressed due to low confidence (1)

src/Microsoft.Data.SqlClient/netfx/src/Microsoft/Data/SqlTypes/SqlTypeWorkarounds.netfx.cs:137

• [nitpick] Add unit tests for SqlDecimalExtractData, especially for the SqlDecimal.Null case and typical non-null values, to verify that data fields are correctly extracted and nulls are handled safely.

        internal static void SqlDecimalExtractData(

[paulmedynski]

/azp run

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[mdaigle]

/azp run

[azure-pipelines]

Azure Pipelines successfully started running 2 pipeline(s).

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 92.58%. Comparing base (90bc83c) to head (2c610f8).
⚠️ Report is 5 commits behind head on release/6.0.

Additional details and impacted files

@@               Coverage Diff                @@
##           release/6.0    #3466       +/-   ##
================================================
+ Coverage        72.73%   92.58%   +19.84%     
================================================
  Files              285        6      -279     
  Lines            59162      310    -58852     
================================================
- Hits             43034      287    -42747     
+ Misses           16128       23    -16105     

Flag	Coverage Δ
addons	92.58% <ø> (ø)
netcore	?
netfx	?

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.