Changeset 12283

Timestamp:

11/29/2022 10:36:36 PM (3 years ago)

Author:

iandunn

Message:

o2 Follow: Send extra data to WPCOM to pass spam checks.

This applies the same fix as ​https://github.com/WordPress/wordcamp.org/pull/802, and fixes the bug where users would never receive a confirmation email after clicking the Follow button.

Props dd32

File:

• sites/trunk/wordpress.org/public_html/wp-content/mu-plugins/pub/class-o2-follow.php (modified) (1 diff)

sites/trunk/wordpress.org/public_html/wp-content/mu-plugins/pub/class-o2-follow.php

@@ -303 +303 @@
         $jetpack_subscriptions = Jetpack_Subscriptions::init();
 
-        $response = $jetpack_subscriptions->subscribe( $current_user->user_email, array( $post_id ), true );
+        $response = $jetpack_subscriptions->subscribe(
+            $current_user->user_email,
+            array( $post_id ),
+            true,
+            array(
+                'source'         => 'widget',
+                'widget-in-use'  => is_active_widget( false, false, 'blog_subscription', true ) ? 'yes' : 'no',
+                'comment_status' => '',
+                'server_data'    => jetpack_subscriptions_cherry_pick_server_data(),
+            )
+        );
+
+        // todo: This only checks that the data passed was valid, not that the remote request was successful. The
+        // code below counts the post as being followed even if the remote request failed. That's necessary
+        // because it's done async. Maybe refactor to be synchronous and update this to check all error
+        // conditions.
         if ( is_wp_error( $response ) ) {
             return;