Allows subscription quantities of zero on Pricing instances

recurly · Jul 16, 2024 · 0513ed5 · 0513ed5
1 parent dbe1dbc
commit 0513ed5
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/lib/recurly/pricing/subscription/calculations.js b/lib/recurly/pricing/subscription/calculations.js
@@ -272,7 +272,9 @@ export default class Calculations {
     const plan = this.items.plan;
     if (!plan) return { amount: 0, setup_fee: 0 };
     let price = plan.price[this.items.currency];
-    price.amount = price.unit_amount * (plan.quantity || 1);
+    let quantity = plan.quantity;
+    if (typeof quantity !== 'number') quantity = 1;
+    price.amount = price.unit_amount * quantity;
     return price;
   }
 

diff --git a/lib/recurly/pricing/subscription/index.js b/lib/recurly/pricing/subscription/index.js
@@ -371,8 +371,8 @@ export default class SubscriptionPricing extends Pricing {
 
     // options.quantity, plan.quantity, 1
     if (plan && plan.quantity) quantity = plan.quantity;
-    if (options.quantity) quantity = parseInt(options.quantity, 10);
-    if (!quantity || quantity < 1) quantity = 1;
+    if (typeof Number(options.quantity) === 'number') quantity = parseInt(options.quantity, 10);
+    if (typeof quantity !== 'number' || quantity < 0) quantity = 1;
 
     return { currentPlan: plan, quantity, planCode, options, done };
   }