Remove Read-Only Fields from Subscription Update Request (#63)

onairmarc · EncoreBot · web-flow · commit b0bc95dc04b3 · 2025-11-02T19:08:09.000-06:00
Co-authored-by: EncoreBot &lt;ghbot@encoredigitalgroup.com&gt;
diff --git a/.git-blame-ignore-revs b/.git-blame-ignore-revs
@@ -30,3 +30,5 @@ cc49dd08428da674282d3ee3421ae9c47166bd2d
 753421ed297eb787e4eaf14a0dd759a304636217
 c40f4bf03470e0150b96d1e7494b8c2c07cc9f0d
 be4c69669e82a85394df83a714225fafe2409a35
+f5037b59ad70b9cec45fda109e63af97ba610cf0
+7a00910446edf363cdc887611d8714b1ac94d208
diff --git a/src/php/Services/StripeSubscriptionService.php b/src/php/Services/StripeSubscriptionService.php
@@ -60,8 +60,8 @@ public function update(string $subscriptionId, StripeSubscription $subscription)
     {
         $data = $subscription->toArray();
 
-        // Remove id from update data
-        unset($data["id"]);
+        // Remove read-only fields from update data
+        unset($data["id"], $data["currency"], $data["status"], $data["customer"]);
 
         $stripeSubscription = $this->stripe->subscriptions->update($subscriptionId, $data);
 

Original file line number	Diff line number	Diff line change
`@@ -60,8 +60,8 @@ public function update(string $subscriptionId, StripeSubscription $subscription)`
`60`	`60`	`{`
`61`	`61`	`$data = $subscription->toArray();`
`62`	`62`
`63`		`- // Remove id from update data`
`64`		`- unset($data["id"]);`
	`63`	`+ // Remove read-only fields from update data`
	`64`	`+ unset($data["id"], $data["currency"], $data["status"], $data["customer"]);`
`65`	`65`
`66`	`66`	`$stripeSubscription = $this->stripe->subscriptions->update($subscriptionId, $data);`
`67`	`67`