fix: solves #88

vippsas · Apr 3, 2024 · d3f2710 · d3f2710
1 parent b0f0f3e
commit d3f2710
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/README.txt b/README.txt
@@ -164,6 +164,7 @@ Alternatively you could look into using WooCommerce "Early renewals": [https://d
 = Not released yet =
 * Added: A warning for when you select Vipps as your brand when your currency is not set to NOK.
 * Fixed: Frontend admin localization.
+* Fixed: A fatal error that could occur when calling `get_agreement_id_from_order` when handling renewals. Reported and diagnosed by magnuskl on GitHub.
 
 = 1.20.1 =
 * Fixed: Frontend scripts were not properly loaded.

diff --git a/includes/wc-vipps-recurring-helper.php b/includes/wc-vipps-recurring-helper.php
@@ -225,7 +225,7 @@ public static function get_agreement_id_from_order( $order ) {
 			return $agreement_id;
 		}
 
-		$subscriptions = wcs_get_subscriptions_for_order( $order );
+		$subscriptions = wcs_get_subscriptions_for_order( $order, [ 'order_type' => 'any' ] );
 		$subscription  = array_shift( $subscriptions );
 
 		return self::get_meta( $subscription, self::META_AGREEMENT_ID );