fix: set product variant items and subscription plans accordingly

CrystallizeAPI · Nov 21, 2023 · df54201 · df54201
1 parent ea6fd28
commit df54201
Show file tree

Hide file tree

Showing 2 changed files with 52 additions and 45 deletions.
diff --git a/.changeset/grumpy-berries-kneel.md b/.changeset/grumpy-berries-kneel.md
@@ -0,0 +1,5 @@
+---
+'@crystallize/import-utilities': patch
+---
+
+fix importing variants with images and subscription plans
diff --git a/src/bootstrap-tenant/bootstrapper/items/createOrUpdateItem.ts b/src/bootstrap-tenant/bootstrapper/items/createOrUpdateItem.ts
@@ -385,60 +385,62 @@ export async function createOrUpdateItem(
       } else {
         delete variant.components
       }
+    }
 
-      if (jsonVariant.subscriptionPlans) {
-        variant.subscriptionPlans = jsonVariant.subscriptionPlans.map((sP) => {
-          const meteredVariables = getSubscriptionPlanMeteredVariables({
-            planIdentifier: sP.identifier,
-            context,
-          })
+    if (jsonVariant.images) {
+      variant.images = await createImagesInput({
+        images: jsonVariant.images,
+        language,
+        context,
+        onUpdate,
+      })
+    }
 
-          return {
-            identifier: sP.identifier,
-            periods: sP.periods.map((p) => {
-              const id = getSubscriptionPlanPeriodId({
-                planIdentifier: sP.identifier,
-                periodName: p.name,
-                context,
-              })
+    // This causes an internal error at the API right now. Setting the value to an empty
+    // array has the same outcome as setting it to null
+    if (variant.images === null) {
+      variant.images = []
+    }
 
-              if (!id) {
-                throw new Error('Plan period id is null')
-              }
 
-              return {
-                id,
-                ...(p.initial && {
-                  initial: subscriptionPlanPrincingJsonToInput(
+    if (jsonVariant.subscriptionPlans) {
+      variant.subscriptionPlans = jsonVariant.subscriptionPlans.map((sP) => {
+        const meteredVariables = getSubscriptionPlanMeteredVariables({
+          planIdentifier: sP.identifier,
+          context,
+        })
+
+        return {
+          identifier: sP.identifier,
+          periods: sP.periods.map((p) => {
+            const id = getSubscriptionPlanPeriodId({
+              planIdentifier: sP.identifier,
+              periodName: p.name,
+              context,
+            })
+
+            if (!id) {
+              throw new Error('Plan period id is null')
+            }
+
+            return {
+              id,
+              ...(p.initial && {
+                initial: subscriptionPlanPrincingJsonToInput(
                     p.initial,
                     meteredVariables
-                  ),
-                }),
-                recurring: subscriptionPlanPrincingJsonToInput(
+                ),
+              }),
+              recurring: subscriptionPlanPrincingJsonToInput(
                   p.recurring,
                   meteredVariables
-                ),
-              }
-            }),
-          }
-        })
-      }
-
-      if (jsonVariant.images) {
-        variant.images = await createImagesInput({
-          images: jsonVariant.images,
-          language,
-          context,
-          onUpdate,
-        })
-      }
-
-      // This causes an internal error at the API right now. Setting the value to an empty
-      // array has the same outcome as setting it to null
-      if (variant.images === null) {
-        variant.images = []
-      }
+              ),
+            }
+          }),
+        }
+      })
     }
+
     return variant
   }