Commit 2658226
Fixes: #1206
This PR restores the QueryProductDetailsResult type and fixes the
QueryProductDetailsAsync behavior that was inadvertently broken in PR
#1200.
### Problem
In PR #1200, the billing client was updated from version 7.1.1 to
8.0.0, but the changes introduced a breaking change to the API:
The `QueryProductDetailsResult` class was removed the
`QueryProductDetailsAsync` method was changed to pass through a
`QueryProductDetailsResult` parameter directly instead of creating a new
instance This created an inconsistent state where the code referenced
a type that no longer existed.
### Solution
Updated the nugetVersion from 8.0.0 to 8.0.0.1 to reflect the latest
version of the NuGet package.
Enhancements to Product Details Handling:
* Restored the `QueryProductDetailsResult` class with properties for
`BillingResult` and `ProductDetailsList`. Marked the
`ProductDetails` property as obsolete to encourage the use of
`ProductDetailsList`.
* Updated the `OnProductDetailsResponse` method in
`InternalProductDetailsResponseListener` to initialize
`QueryProductDetailsResult` if null and set the Result property
before invoking the response handler.
Co-authored-by: Jonathan Peppers <jonathan.peppers@microsoft.com>
1 parent c4f2da4 commit 2658226
2 files changed
Lines changed: 20 additions & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
2167 | 2167 | | |
2168 | 2168 | | |
2169 | 2169 | | |
2170 | | - | |
| 2170 | + | |
2171 | 2171 | | |
2172 | 2172 | | |
2173 | 2173 | | |
| |||
Lines changed: 19 additions & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
27 | 27 | | |
28 | 28 | | |
29 | 29 | | |
| 30 | + | |
| 31 | + | |
| 32 | + | |
| 33 | + | |
| 34 | + | |
| 35 | + | |
| 36 | + | |
| 37 | + | |
| 38 | + | |
| 39 | + | |
| 40 | + | |
| 41 | + | |
| 42 | + | |
| 43 | + | |
30 | 44 | | |
31 | 45 | | |
32 | 46 | | |
| |||
241 | 255 | | |
242 | 256 | | |
243 | 257 | | |
244 | | - | |
| 258 | + | |
| 259 | + | |
| 260 | + | |
| 261 | + | |
| 262 | + | |
245 | 263 | | |
246 | 264 | | |
247 | 265 | | |
| |||
0 commit comments