Merge pull request #218 from asamMarkus/develop

Bugfix in Subscription::scheduleNewOrderItemAt
laravel · Jun 5, 2020 · 26fa67c · 26fa67c
2 parents a77b211 + 9290cf8
commit 26fa67c
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/src/Subscription.php b/src/Subscription.php
@@ -418,7 +418,7 @@ public function scheduleNewOrderItemAt(Carbon $process_at, $item_overrides = [],
                 'unit_price' => (int) $plan->amount()->getAmount(),
                 'quantity' => $this->quantity ?: 1,
                 'tax_percentage' => $this->tax_percentage,
-                'description' => $this->plan()->description(),
+                'description' => $plan->description(),
             ], $item_overrides
         ));
 

diff --git a/tests/SwapSubscriptionPlanTest.php b/tests/SwapSubscriptionPlanTest.php
@@ -138,6 +138,7 @@ public function canSwapNextCycle()
         $this->assertCarbon($cycle_should_end_at, $new_order_item->process_at, 1); // based on previous plan's cycle
         $this->assertEquals(2200, $new_order_item->total);
         $this->assertEquals(200, $new_order_item->tax);
+        $this->assertEquals('Twice as expensive monthly subscription', $new_order_item->description);
 
         $this->assertFalse($user->fresh()->hasCredit());