-
Notifications
You must be signed in to change notification settings - Fork 2k
Premium Content: Correct "non-static method call" errors. #44243
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
This PR does not affect the size of JS and CSS bundles shipped to the user's browser. Generated by performance advisor bot at iscalypsofastyet.com. |
|
Caution: This PR affects files in the FSE Plugin on WordPress.com D46534-code has been created so you can easily test it on your sandbox. See this FieldGuide page about developing in the FSE Plugin for more info: PCYsg-ly5-p2 |
...ing-plugin/premium-content/subscription-service/class-jetpack-token-subscription-service.php
Show resolved
Hide resolved
gwwar
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @kwight I verified that this ties the non-static method call, and that the premium content block still behaves as expected when subscribing.
|
Going to help land to help clear up the team board. |

The Premium Content block causes log errors by calling a non-static method statically to get tokens. This PR corrects the code usage to avoid the errors.
Discovered by @blackjackkent while testing #44192 .
Testing Instructions
FSE plugin / Build FSE plugin (pull_request)task below as a ZIP plugin that you can upload to the test site.