Revert "Using json serializer for skproduct wrapper and related classes (flutter#1147)"

This reverts commit 9b4aaae.

Author: andreidiaconu

packages/in_app_purchase/lib/src/store_kit_wrappers/sk_product_wrapper.dart

@@ -3,18 +3,11 @@
 // found in the LICENSE file.
 
 import 'package:flutter/foundation.dart';
-import 'package:json_annotation/json_annotation.dart';
-
-// WARNING: Changes to `@JsonSerializable` classes need to be reflected in the
-// below generated file. Run `flutter packages pub run build_runner watch` to
-// rebuild and watch for further changes.
-part 'sk_product_wrapper.g.dart';
 
 /// Dart wrapper around StoreKit's [SKProductsResponse](https://developer.apple.com/documentation/storekit/skproductsresponse?language=objc).
 ///
 /// Represents the response object returned by [SKRequestMaker.startProductRequest].
 /// Contains information about a list of products and a list of invalid product identifiers.
-@JsonSerializable()
 class SkProductResponseWrapper {
   SkProductResponseWrapper(
       {@required this.products, @required this.invalidProductIdentifiers});
@@ -23,10 +16,11 @@ class SkProductResponseWrapper {
   ///
   /// This method should only be used with `map` values returned by [SKRequestMaker.startProductRequest].
   /// The `map` parameter must not be null.
-  factory SkProductResponseWrapper.fromJson(Map map) {
-    assert(map != null);
-    return _$SkProductResponseWrapperFromJson(map);
-  }
+  SkProductResponseWrapper.fromMap(Map<String, List<dynamic>> map)
+      : assert(map != null),
+        products = _getListFromMapList(_getProductMapListFromResponseMap(map)),
+        invalidProductIdentifiers =
+            List.castFrom<dynamic, String>(map['invalidProductIdentifiers']);
 
   /// Stores all matching successfully found products.
   ///
@@ -40,6 +34,18 @@ class SkProductResponseWrapper {
   /// found here https://developer.apple.com/documentation/storekit/skproductsresponse/1505985-invalidproductidentifiers?language=objc.
   /// Will be empty if all the product identifiers are valid.
   final List<String> invalidProductIdentifiers;
+
+  static List<Map<dynamic, dynamic>> _getProductMapListFromResponseMap(
+      Map<String, List<dynamic>> map) {
+    return map['products'].cast<Map<dynamic, dynamic>>();
+  }
+
+  static List<SKProductWrapper> _getListFromMapList(
+      List<Map<dynamic, dynamic>> mapList) {
+    return mapList
+        .map((Map<dynamic, dynamic> map) => SKProductWrapper.fromMap(map))
+        .toList();
+  }
 }
 
 /// Dart wrapper around StoreKit's [SKProductPeriodUnit](https://developer.apple.com/documentation/storekit/skproductperiodunit?language=objc).
@@ -48,34 +54,30 @@ class SkProductResponseWrapper {
 // The values of the enum options are matching the [SKProductPeriodUnit]'s values. Should there be an update or addition
 // in the [SKProductPeriodUnit], this need to be updated to match.
 enum SubscriptionPeriodUnit {
-  @JsonValue(0)
   day,
-  @JsonValue(1)
   week,
-  @JsonValue(2)
   month,
-  @JsonValue(3)
   year,
 }
 
 /// Dart wrapper around StoreKit's [SKProductSubscriptionPeriod](https://developer.apple.com/documentation/storekit/skproductsubscriptionperiod?language=objc).
 ///
 /// A period is defined by a [numberOfUnits] and a [unit], e.g for a 3 months period [numberOfUnits] is 3 and [unit] is a month.
 /// It is used as a property in [SKProductDiscountWrapper] and [SKProductWrapper].
-@JsonSerializable(nullable: true)
 class SKProductSubscriptionPeriodWrapper {
   SKProductSubscriptionPeriodWrapper(
       {@required this.numberOfUnits, @required this.unit});
 
   /// Constructing an instance from a map from the Objective-C layer.
-  ///
-  /// This method should only be used with `map` values returned by [SKProductDiscountWrapper.fromJson] or [SKProductWrapper.fromJson].
+  /// This method should only be used with `map` values returned by [SKProductDiscountWrapper.fromMap] or [SKProductWrapper.fromMap].
   /// The `map` parameter must not be null.
-  factory SKProductSubscriptionPeriodWrapper.fromJson(Map map) {
-    assert(map != null &&
-        (map['numberOfUnits'] == null || map['numberOfUnits'] > 0));
-    return _$SKProductSubscriptionPeriodWrapperFromJson(map);
-  }
+  SKProductSubscriptionPeriodWrapper.fromMap(Map<String, dynamic> map)
+      : assert(map != null &&
+            (map['numberOfUnits'] == null || map['numberOfUnits'] > 0)),
+        numberOfUnits = map['numberOfUnits'],
+        unit = (map['unit'] != null)
+            ? SubscriptionPeriodUnit.values[map['unit']]
+            : null;
 
   /// The number of [unit] units in this period.
   ///
@@ -93,15 +95,12 @@ class SKProductSubscriptionPeriodWrapper {
 // in the [SKProductDiscountPaymentMode], this need to be updated to match.
 enum ProductDiscountPaymentMode {
   /// Allows user to pay the discounted price at each payment period.
-  @JsonValue(0)
   payAsYouGo,
 
   /// Allows user to pay the discounted price upfront and receive the product for the rest of time that was paid for.
-  @JsonValue(1)
   payUpFront,
 
   /// User pays nothing during the discounted period.
-  @JsonValue(2)
   freeTrail,
 }
 
@@ -110,7 +109,6 @@ enum ProductDiscountPaymentMode {
 /// Most of the fields are identical to OBJC SKProduct.
 /// The only difference is instead of the locale object, we only exposed currencyCode for simplicity.
 /// It is used as a property in [SKProductWrapper].
-@JsonSerializable(nullable: true)
 class SKProductDiscountWrapper {
   SKProductDiscountWrapper(
       {@required this.price,
@@ -121,12 +119,20 @@ class SKProductDiscountWrapper {
 
   /// Constructing an instance from a map from the Objective-C layer.
   ///
-  /// This method should only be used with `map` values returned by [SKProductWrapper.fromJson].
+  /// This method should only be used with `map` values returned by [SKProductWrapper.fromMap].
   /// The `map` parameter must not be null.
-  factory SKProductDiscountWrapper.fromJson(Map map) {
-    assert(map != null);
-    return _$SKProductDiscountWrapperFromJson(map);
-  }
+  SKProductDiscountWrapper.fromMap(Map<String, dynamic> map)
+      : assert(map != null),
+        price = map['price'],
+        currencyCode = map['currencyCode'],
+        numberOfPeriods = map['numberOfPeriods'],
+        paymentMode = (map['paymentMode'] != null)
+            ? ProductDiscountPaymentMode.values[map['paymentMode']]
+            : null,
+        subscriptionPeriod = map['subscriptionPeriod'] != null
+            ? SKProductSubscriptionPeriodWrapper.fromMap(
+                map['subscriptionPeriod'].cast<String, dynamic>())
+            : null;
 
   /// The discounted price, in the currency that is defined in [currencyCode].
   final double price;
@@ -158,7 +164,6 @@ class SKProductDiscountWrapper {
 /// The only difference is instead of the locale object, we only exposed currencyCode for simplicity.
 /// A list of [SKProductWrapper] is returned in the [SKRequestMaker.startProductRequest] method, and
 /// should be stored for use when making a payment.
-@JsonSerializable(nullable: true)
 class SKProductWrapper {
   SKProductWrapper({
     @required this.productIdentifier,
@@ -176,12 +181,28 @@ class SKProductWrapper {
 
   /// Constructing an instance from a map from the Objective-C layer.
   ///
-  /// This method should only be used with `map` values returned by [SkProductResponseWrapper.fromJson].
+  /// This method should only be used with `map` values returned by [SkProductResponseWrapper.fromMap].
   /// The `map` parameter must not be null.
-  factory SKProductWrapper.fromJson(Map map) {
-    assert(map != null);
-    return _$SKProductWrapperFromJson(map);
-  }
+  SKProductWrapper.fromMap(Map<dynamic, dynamic> map)
+      : assert(map != null),
+        productIdentifier = map['productIdentifier'],
+        localizedTitle = map['localizedTitle'],
+        localizedDescription = map['localizedDescription'],
+        currencyCode = map['currencyCode'],
+        downloadContentVersion = map['downloadContentVersion'],
+        subscriptionGroupIdentifier = map['subscriptionGroupIdentifier'],
+        price = map['price'],
+        downloadable = map['downloadable'],
+        downloadContentLengths =
+            List.castFrom<dynamic, int>(map['downloadContentLengths']),
+        subscriptionPeriod = map['subscriptionPeriod'] != null
+            ? SKProductSubscriptionPeriodWrapper.fromMap(
+                map['subscriptionPeriod'].cast<String, dynamic>())
+            : null,
+        introductoryPrice = (map['introductoryPrice'] != null)
+            ? SKProductDiscountWrapper.fromMap(
+                map['introductoryPrice'].cast<String, dynamic>())
+            : null;
 
   /// The unique identifier of the product.
   final String productIdentifier;

packages/in_app_purchase/lib/src/store_kit_wrappers/sk_product_wrapper.g.dart

@@ -24,7 +24,7 @@ class SKRequestMaker {
   /// A [PlatformException] is thrown if the platform code making the request fails.
   Future<SkProductResponseWrapper> startProductRequest(
       List<String> productIdentifiers) async {
-    final Map productResponseMap = await channel.invokeMethod(
+    final Map<dynamic, dynamic> productResponseMap = await channel.invokeMethod(
       '-[InAppPurchasePlugin startProductRequest:result:]',
       productIdentifiers,
     );
@@ -34,6 +34,7 @@ class SKRequestMaker {
         message: 'StoreKit: Failed to get response from platform.',
       );
     }
-    return SkProductResponseWrapper.fromJson(productResponseMap);
+    return SkProductResponseWrapper.fromMap(
+        productResponseMap.cast<String, List<dynamic>>());
   }
 }

packages/in_app_purchase/lib/src/store_kit_wrappers/sk_request_maker.dart

@@ -41,7 +41,7 @@ void main() {
         'SKProductSubscriptionPeriodWrapper should have property values consistent with map',
         () {
       final SKProductSubscriptionPeriodWrapper wrapper =
-          SKProductSubscriptionPeriodWrapper.fromJson(subMap);
+          SKProductSubscriptionPeriodWrapper.fromMap(subMap);
       expect(wrapper.numberOfUnits, subMap['numberOfUnits']);
       expect(wrapper.unit, SubscriptionPeriodUnit.values[subMap['unit']]);
     });
@@ -50,7 +50,7 @@ void main() {
         'SKProductSubscriptionPeriodWrapper should have properties to be null if map is empty',
         () {
       final SKProductSubscriptionPeriodWrapper wrapper =
-          SKProductSubscriptionPeriodWrapper.fromJson(<String, dynamic>{});
+          SKProductSubscriptionPeriodWrapper.fromMap(<String, dynamic>{});
       expect(wrapper.numberOfUnits, null);
       expect(wrapper.unit, null);
     });
@@ -59,7 +59,7 @@ void main() {
         'SKProductDiscountWrapper should have property values consistent with map',
         () {
       final SKProductDiscountWrapper wrapper =
-          SKProductDiscountWrapper.fromJson(discountMap);
+          SKProductDiscountWrapper.fromMap(discountMap);
       expect(wrapper.price, discountMap['price']);
       expect(wrapper.currencyCode, discountMap['currencyCode']);
       expect(wrapper.numberOfPeriods, discountMap['numberOfPeriods']);
@@ -77,7 +77,7 @@ void main() {
         'SKProductDiscountWrapper should have properties to be null if map is empty',
         () {
       final SKProductDiscountWrapper wrapper =
-          SKProductDiscountWrapper.fromJson(<String, dynamic>{});
+          SKProductDiscountWrapper.fromMap(<String, dynamic>{});
       expect(wrapper.price, null);
       expect(wrapper.currencyCode, null);
       expect(wrapper.numberOfPeriods, null);
@@ -125,14 +125,15 @@ void main() {
 
     test('SKProductWrapper should have property values consistent with map',
         () {
-      final SKProductWrapper wrapper = SKProductWrapper.fromJson(productMap);
+      final SKProductWrapper wrapper = SKProductWrapper.fromMap(productMap);
       testMatchingProductMap(wrapper, productMap);
     });
 
-    test('SKProductWrapper should have properties to be null if map is empty',
+    test(
+        'SKProductDiscountWrapper should have properties to be null if map is empty',
         () {
       final SKProductWrapper wrapper =
-          SKProductWrapper.fromJson(<String, dynamic>{});
+          SKProductWrapper.fromMap(<String, dynamic>{});
       expect(wrapper.productIdentifier, null);
       expect(wrapper.localizedTitle, null);
       expect(wrapper.localizedDescription, null);
@@ -146,7 +147,7 @@ void main() {
 
     test('SKProductResponse wrapper should match', () {
       final SkProductResponseWrapper wrapper =
-          SkProductResponseWrapper.fromJson(productResponseMap);
+          SkProductResponseWrapper.fromMap(productResponseMap);
       testMatchingProductMap(
           wrapper.products[0], productResponseMap['products'][0]);
       expect(wrapper.invalidProductIdentifiers,
@@ -159,7 +160,7 @@ void main() {
         'invalidProductIdentifiers': <String>[],
       };
       final SkProductResponseWrapper wrapper =
-          SkProductResponseWrapper.fromJson(productResponseMapEmptyList);
+          SkProductResponseWrapper.fromMap(productResponseMapEmptyList);
       expect(wrapper.products.length, 0);
       expect(wrapper.invalidProductIdentifiers.length, 0);
     });

packages/in_app_purchase/test/store_kit_wrappers/sk_product_test.dart

@@ -48,7 +48,7 @@ void main() {
     test('platform call should get result', () async {
       stubPlatform.addResponse(
           name: '-[InAppPurchasePlugin startProductRequest:result:]',
-          value: productResponseMap);
+          value: productResponseMap.cast<String, dynamic>());
       final SKRequestMaker request = SKRequestMaker();
       final SkProductResponseWrapper response =
           await request.startProductRequest(<String>['123']);