Changeset 12283

Timestamp:

11/29/2022 10:36:36 PM (4 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;