Make it work

Automattic · Mar 27, 2019 · a72a7c4 · a72a7c4
1 parent bd2284d
commit a72a7c4
Show file tree

Hide file tree

Showing 3 changed files with 21 additions and 20 deletions.
diff --git a/_inc/lib/core-api/wpcom-endpoints/memberships.php b/_inc/lib/core-api/wpcom-endpoints/memberships.php
@@ -79,7 +79,7 @@ public function create_product( $request ) {
 			$blog_id = Jetpack_Options::get_option( 'id' );
 			$response = Jetpack_Client::wpcom_json_api_request_as_blog(
 				"/sites/$blog_id/{$this->rest_base}/product",
-				'v2',
+				'2',
 				array(
 					'method' => 'POST',
 				),
@@ -88,7 +88,8 @@ public function create_product( $request ) {
 					'price' => $request['price'],
 					'currency' => $request['currency'],
 					'interval' => $request['interval'],
-				)
+				),
+				'wpcom'
 			);
 			if ( is_wp_error( $response ) ) {
 				return new WP_Error( 'wpcom_connection_error', 'Could not connect to WP.com', 404 );

diff --git a/extensions/blocks/membership-button/edit.jsx b/extensions/blocks/membership-button/edit.jsx
@@ -63,16 +63,12 @@ class MembershipsButtonEdit extends Component {
 		const fetch = { path, method };
 		apiFetch( fetch ).then(
 			result => {
-				this.setState( {
-					products: this.state.products.concat( [
-						{
-							id: result.id,
-							title: result.title,
-							interval: result.interval,
-							price: result.price,
-						},
-					] ),
-				} );
+				const connectURL = result.connect_url;
+				const products = result.products;
+				const connected = result.connected_account_id
+					? API_STATE_CONNECTED
+					: API_STATE_NOTCONNECTED;
+				this.setState( { connected, connectURL, products } );
 			}
 			// result => {
 			// 	const connectURL = null;
@@ -117,12 +113,16 @@ class MembershipsButtonEdit extends Component {
 		const fetch = { path, method, data };
 		apiFetch( fetch ).then(
 			result => {
-				const connectURL = result.connect_url;
-				const products = result.products;
-				const connected = result.connected_account_id
-					? API_STATE_CONNECTED
-					: API_STATE_NOTCONNECTED;
-				this.setState( { connected, connectURL, products } );
+				this.setState( {
+					products: this.state.products.concat( [
+						{
+							id: result.id,
+							title: result.title,
+							interval: result.interval,
+							price: result.price,
+						},
+					] ),
+				} );
 			}
 			// result => {
 			// 	const connectURL = null;

diff --git a/modules/memberships/class-jetpack-memberships.php b/modules/memberships/class-jetpack-memberships.php
@@ -17,7 +17,7 @@ class Jetpack_Memberships {
 	 *
 	 * @var string
 	 */
-	static private $css_classname_prefix = 'jetpack-memberships';
+	static public $css_classname_prefix = 'jetpack-memberships';
 	/**
 	 * Increase this number each time there's a change in CSS or JS to bust cache.
 	 *
@@ -245,7 +245,7 @@ public function render_button( $attrs ) {
 				$button_styles,
 				sprintf(
 					'color: %s',
-					sanitize_hex_color( $attr['customTextButtonColor'] )
+					sanitize_hex_color( $attrs['customTextButtonColor'] )
 				)
 			);
 		}