Comment missing corresponding RCP status and remove duplicate/unneces…

…sary check.
pronamic · Mar 13, 2024 · 43a2db0 · 43a2db0
1 parent 63dfff6
commit 43a2db0
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 4 deletions.
diff --git a/src/Extension.php b/src/Extension.php
@@ -423,10 +423,6 @@ public function payment_status_update( Payment $payment ) {
 	 * @return void
 	 */
 	public function subscription_status_update( Subscription $pronamic_subscription ) {
-		if ( SubscriptionStatus::COMPLETED === $pronamic_subscription->get_status() ) {
-			return;
-		}
-
 		$membership_id = $pronamic_subscription->get_source_id();
 
 		$rcp_membership = \rcp_get_membership( $membership_id );

diff --git a/src/MembershipStatus.php b/src/MembershipStatus.php
@@ -102,6 +102,7 @@ public static function transform_from_pronamic( $status ) {
 			case SubscriptionStatus::OPEN:
 				return self::PENDING;
 			case SubscriptionStatus::COMPLETED:
+				// Restrict Content Pro does not have a corresponding status.
 				return null;
 			default:
 				return null;