Fix checking of ApplePay and GooglePay product status capabilities

modules/ppcp-api-client/src/Entity/SellerStatus.php

@@ -21,19 +21,37 @@ class SellerStatus {
 	 */
 	private $products;
 
+	/**
+	 * The capabilities.
+	 *
+	 * @var SellerStatusCapability[]
+	 */
+	private $capabilities;
+
 	/**
 	 * SellerStatus constructor.
 	 *
-	 * @param SellerStatusProduct[] $products The products.
+	 * @param SellerStatusProduct[]    $products The products.
+	 * @param SellerStatusCapability[] $capabilities The capabilities.
+	 *
+	 * @psalm-suppress RedundantConditionGivenDocblockType
 	 */
-	public function __construct( array $products ) {
+	public function __construct( array $products, array $capabilities ) {
 		foreach ( $products as $key => $product ) {
 			if ( is_a( $product, SellerStatusProduct::class ) ) {
 				continue;
 			}
 			unset( $products[ $key ] );
 		}
-		$this->products = $products;
+		foreach ( $capabilities as $key => $capability ) {
+			if ( is_a( $capability, SellerStatusCapability::class ) ) {
+				continue;
+			}
+			unset( $capabilities[ $key ] );
+		}
+
+		$this->products     = $products;
+		$this->capabilities = $capabilities;
 	}
 
 	/**
@@ -45,6 +63,15 @@ class SellerStatus {
 		return $this->products;
 	}
 
+	/**
+	 * Returns the capabilities.
+	 *
+	 * @return SellerStatusCapability[]
+	 */
+	public function capabilities() : array {
+		return $this->capabilities;
+	}
+
 	/**
 	 * Returns the enitity as array.
 	 *
@@ -58,8 +85,16 @@ class SellerStatus {
 			$this->products()
 		);
 
+		$capabilities = array_map(
+			function( SellerStatusCapability $capability ) : array {
+				return $capability->to_array();
+			},
+			$this->capabilities()
+		);
+
 		return array(
-			'products' => $products,
+			'products'     => $products,
+			'capabilities' => $capabilities,
 		);
 	}
 }

modules/ppcp-api-client/src/Entity/SellerStatusCapability.php

@@ -0,0 +1,77 @@
+<?php
+/**
+ * The capabilities of a seller status.
+ *
+ * @package WooCommerce\PayPalCommerce\ApiClient\Entity
+ */
+
+declare( strict_types=1 );
+
+namespace WooCommerce\PayPalCommerce\ApiClient\Entity;
+
+/**
+ * Class SellerStatusCapability
+ */
+class SellerStatusCapability {
+
+	const STATUS_ACTIVE = 'ACTIVE';
+
+	/**
+	 * The name of the product.
+	 *
+	 * @var string
+	 */
+	private $name;
+
+	/**
+	 * The status of the capability.
+	 *
+	 * @var string
+	 */
+	private $status;
+
+	/**
+	 * SellerStatusCapability constructor.
+	 *
+	 * @param string $name   The name of the product.
+	 * @param string $status The status of the capability.
+	 */
+	public function __construct(
+		string $name,
+		string $status
+	) {
+		$this->name   = $name;
+		$this->status = $status;
+	}
+
+	/**
+	 * Returns the name of the product.
+	 *
+	 * @return string
+	 */
+	public function name() : string {
+		return $this->name;
+	}
+
+	/**
+	 * Returns the status for this capability.
+	 *
+	 * @return string
+	 */
+	public function status() : string {
+		return $this->status;
+	}
+
+	/**
+	 * Returns the entity as array.
+	 *
+	 * @return array
+	 */
+	public function to_array() : array {
+		return array(
+			'name'   => $this->name(),
+			'status' => $this->status(),
+		);
+	}
+
+}

modules/ppcp-api-client/src/Factory/SellerStatusFactory.php

@@ -11,6 +11,7 @@ namespace WooCommerce\PayPalCommerce\ApiClient\Factory;
 
 use WooCommerce\PayPalCommerce\ApiClient\Entity\SellerStatus;
 use WooCommerce\PayPalCommerce\ApiClient\Entity\SellerStatusProduct;
+use WooCommerce\PayPalCommerce\ApiClient\Entity\SellerStatusCapability;
 
 /**
  * Class SellerStatusFactory
@@ -37,6 +38,17 @@ class SellerStatusFactory {
 			isset( $json->products ) ? (array) $json->products : array()
 		);
 
-		return new SellerStatus( $products );
+		$capabilities = array_map(
+			function( $json ) : SellerStatusCapability {
+				$capability = new SellerStatusCapability(
+					isset( $json->name ) ? (string) $json->name : '',
+					isset( $json->status ) ? (string) $json->status : ''
+				);
+				return $capability;
+			},
+			isset( $json->capabilities ) ? (array) $json->capabilities : array()
+		);
+
+		return new SellerStatus( $products, $capabilities );
 	}
 }

modules/ppcp-applepay/src/Assets/AppleProductStatus.php

@@ -11,6 +11,7 @@ namespace WooCommerce\PayPalCommerce\Applepay\Assets;
 
 use Throwable;
 use WooCommerce\PayPalCommerce\ApiClient\Endpoint\PartnersEndpoint;
+use WooCommerce\PayPalCommerce\ApiClient\Entity\SellerStatusCapability;
 use WooCommerce\PayPalCommerce\ApiClient\Helper\FailureRegistry;
 use WooCommerce\PayPalCommerce\Onboarding\State;
 use WooCommerce\PayPalCommerce\WcGateway\Settings\Settings;
@@ -133,21 +134,32 @@ class AppleProductStatus {
 		}
 
 		// Check the seller status for the intended capability.
+		$has_capability = false;
 		foreach ( $seller_status->products() as $product ) {
 			if ( $product->name() !== 'PAYMENT_METHODS' ) {
 				continue;
 			}
 
 			if ( in_array( self::CAPABILITY_NAME, $product->capabilities(), true ) ) {
-				// Capability found, persist status and return true.
+				$has_capability = true;
-				$this->settings->set( self::SETTINGS_KEY, self::SETTINGS_VALUE_ENABLED );
-				$this->settings->persist();
-
-				$this->current_status = true;
-				return $this->current_status;
 			}
 		}
 
+		foreach ( $seller_status->capabilities() as $capability ) {
+			if ( $capability->name() === self::CAPABILITY_NAME && $capability->status() === SellerStatusCapability::STATUS_ACTIVE ) {
+				$has_capability = true;
+			}
+		}
+
+		if ( $has_capability ) {
+			// Capability found, persist status and return true.
+			$this->settings->set( self::SETTINGS_KEY, self::SETTINGS_VALUE_ENABLED );
+			$this->settings->persist();
+
+			$this->current_status = true;
+			return $this->current_status;
+		}
+
 		// Capability not found, persist status and return false.
 		$this->settings->set( self::SETTINGS_KEY, self::SETTINGS_VALUE_DISABLED );
 		$this->settings->persist();

modules/ppcp-googlepay/src/Helper/ApmProductStatus.php

@@ -11,6 +11,7 @@ namespace WooCommerce\PayPalCommerce\Googlepay\Helper;
 
 use Throwable;
 use WooCommerce\PayPalCommerce\ApiClient\Endpoint\PartnersEndpoint;
+use WooCommerce\PayPalCommerce\ApiClient\Entity\SellerStatusCapability;
 use WooCommerce\PayPalCommerce\ApiClient\Helper\FailureRegistry;
 use WooCommerce\PayPalCommerce\Onboarding\State;
 use WooCommerce\PayPalCommerce\WcGateway\Settings\Settings;
@@ -133,21 +134,32 @@ class ApmProductStatus {
 		}
 
 		// Check the seller status for the intended capability.
+		$has_capability = false;
 		foreach ( $seller_status->products() as $product ) {
 			if ( $product->name() !== 'PAYMENT_METHODS' ) {
 				continue;
 			}
 
 			if ( in_array( self::CAPABILITY_NAME, $product->capabilities(), true ) ) {
-				// Capability found, persist status and return true.
+				$has_capability = true;
-				$this->settings->set( self::SETTINGS_KEY, self::SETTINGS_VALUE_ENABLED );
-				$this->settings->persist();
-
-				$this->current_status = true;
-				return $this->current_status;
 			}
 		}
 
+		foreach ( $seller_status->capabilities() as $capability ) {
+			if ( $capability->name() === self::CAPABILITY_NAME && $capability->status() === SellerStatusCapability::STATUS_ACTIVE ) {
+				$has_capability = true;
+			}
+		}
+
+		if ( $has_capability ) {
+			// Capability found, persist status and return true.
+			$this->settings->set( self::SETTINGS_KEY, self::SETTINGS_VALUE_ENABLED );
+			$this->settings->persist();
+
+			$this->current_status = true;
+			return $this->current_status;
+		}
+
 		// Capability not found, persist status and return false.
 		$this->settings->set( self::SETTINGS_KEY, self::SETTINGS_VALUE_DISABLED );
 		$this->settings->persist();