Detect when we are running on a WooCommerce Subscriptions staging site

[daledupreez]

Fixes STRIPE-751
Related to:

• More protections against detaching payment methods on a cloned site. #4131
• STRIPE-367

Changes proposed in this Pull Request:

This PR detects when we are running on WooCommerce Subscriptions staging sites as a way to detect whether we should detach customers from payment methods. This expands on our previous logic that only looked at the return value from wp_get_environment_type(). This PR does not implement a full fix for the issues flagged in #4131, but it does mean that staging sites using WooCommerce Subscriptions should behave better, at least when we're calling WC_Stripe_API::should_detach_payment_method_from_customer() in the code path.

Testing instructions

• Apply a build containing the code from this PR/branch to your site
• Ensure WooCommerce Subscriptions is installed and active on the site
• Ensure you are in live mode, either by creating a live connection or by setting the testmode sub-option to no - wp option patch update woocommerce_stripe_settings testmode no
• Without making other changes, access the WP CLI shell via wp shell
• Run the following command to confirm the default behaviour:

return WC_Stripe_API::should_detach_payment_method_from_customer();

• Verify that the return is true, which should be encoded as => bool(true) in the screen.
• Add the following code snippet to force staging mode for WooCommerce Subscriptions:

add_filter( 'woocommerce_subscriptions_is_duplicate_site', '__return_true', 10, 1 );

• Ensure your shell picks up the new code, which may require you to exit, and restart wp shell
• Run the same command as above:

return WC_Stripe_API::should_detach_payment_method_from_customer();

• Verify that this time the return value is false (encoded as => bool(false)).

---

• Covered with tests (or have a good reason not to test in description ☝️)
• [N/A] Tested on mobile (or does not apply)

Changelog entry

• This Pull Request does not require a changelog entry. (Comment required below)

Changelog Entry Comment

Comment

Post merge

• Added testing instructions to the Release Testing Instructions wiki page (or does not apply)
• Added the needs docs label (or does not apply)
• Included this PR in the Release Thread scope (or does not apply)

[wjrosa]

Nice work, Dale! Code is good and works as intended

[malithsen]

Looks good!