fix: variable name > constant

dkoo commented 3 weeks ago

All Submissions:

[x] Have you followed the Newspack Contributing guideline?
[ ] Does your code follow the WordPress' coding standards and VIP Go coding standards?
[x] Have you checked to ensure there aren't other open Pull Requests for the same update/change?

Changes proposed in this Pull Request:

Fixes an error I noticed while testing something else.

How to test the changes in this Pull Request:

On release, NEWSPACK_LOG_LEVEL set to 2 in wp-config.php, log in as a reader. In your debug.log, observe a warning:

[26-Jun-2024 20:54:56 UTC] [4875] [NEWSPACK-DATA-EVENTS][Newspack\Data_Events::handle][ERROR]: Access to undeclared static property Newspack\WooCommerce_Connection::$active_subscription_statuses

Check out this branch, repeat, and confirm no error.

Other information:

[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes, as applicable?
[x] Have you successfully ran tests with your changes locally?

matticbot commented 3 weeks ago

:tada: This PR is included in version 4.3.4 :tada:

The release is available on GitHub release

Your semantic-release bot :package::rocket:

matticbot commented 2 weeks ago

:tada: This PR is included in version 4.4.0-alpha.4 :tada: