Skip to content

Commit b02cd01

Browse files
dmallory42mgascam
andauthored
Cherry-pick #11113: Fix WCPay Subscriptions toggle not saving when unchecked (#11114)
Co-authored-by: Miguel Gasca <[email protected]>
1 parent d032d56 commit b02cd01

File tree

3 files changed

+9
-5
lines changed

3 files changed

+9
-5
lines changed
Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,4 @@
1+
Significance: patch
2+
Type: fix
3+
4+
Fix - WCPay Subscriptions setting not persisting when unchecked

includes/admin/class-wc-rest-payments-settings-controller.php

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -134,7 +134,7 @@ public function register_routes() {
134134
'type' => 'boolean',
135135
'validate_callback' => 'rest_validate_request_arg',
136136
],
137-
'is_wcpay_subscription_enabled' => [
137+
'is_wcpay_subscriptions_enabled' => [
138138
'description' => sprintf(
139139
/* translators: %s: WooPayments */
140140
__( '%s Subscriptions feature flag setting.', 'woocommerce-payments' ),
@@ -803,11 +803,11 @@ private function update_is_multi_currency_enabled( WP_REST_Request $request ) {
803803
* @param WP_REST_Request $request Request object.
804804
*/
805805
private function update_is_wcpay_subscriptions_enabled( WP_REST_Request $request ) {
806-
if ( ! $request->has_param( 'is_wcpay_subscription_enabled' ) ) {
806+
if ( ! $request->has_param( 'is_wcpay_subscriptions_enabled' ) ) {
807807
return;
808808
}
809809

810-
$is_wcpay_subscriptions_enabled = $request->get_param( 'is_wcpay_subscription_enabled' );
810+
$is_wcpay_subscriptions_enabled = $request->get_param( 'is_wcpay_subscriptions_enabled' );
811811

812812
// Prevent enabling bundled subscriptions - feature has been removed in 10.2.0.
813813
// Only allow disabling the feature if it was previously enabled.

tests/unit/admin/test-class-wc-rest-payments-settings-controller.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -654,7 +654,7 @@ public function test_update_settings_disables_wcpay_subscriptions() {
654654
->method( 'store_setup_sync' );
655655

656656
$request = new WP_REST_Request();
657-
$request->set_param( 'is_wcpay_subscription_enabled', false );
657+
$request->set_param( 'is_wcpay_subscriptions_enabled', false );
658658

659659
$this->controller->update_settings( $request );
660660

@@ -670,7 +670,7 @@ public function test_update_settings_does_not_enable_wcpay_subscriptions() {
670670
->method( 'store_setup_sync' );
671671

672672
$request = new WP_REST_Request();
673-
$request->set_param( 'is_wcpay_subscription_enabled', true );
673+
$request->set_param( 'is_wcpay_subscriptions_enabled', true );
674674

675675
$this->controller->update_settings( $request );
676676

0 commit comments

Comments
 (0)