Media php strict comparison#7309
Conversation
… add_action() `error.message` LIKE 'Uncaught exception 'Error' with message 'Call to undefined function add_action()' in /var/www/wp-content/plugins/hello.php:69%' AND `error.class` = 'Error' AND transactionUiName = '/wp-content/plugins/hello.php' 'request_uri': '/wp-content/plugins/hello.php'
…dolly_uncaught_exception Fix Uncaught exception Error with message 'Call to undefined function…
…PATH direct access to wp-includes/rss.php on line 19 https://core.trac.wordpress.org/ticket/61277
Plugin URI http to https change
Trac Ticket MissingThis pull request is missing a link to a Trac ticket. For a contribution to be considered, there must be a corresponding ticket in Trac. To attach a pull request to a Trac ticket, please include the ticket's full URL in your pull request description. More information about contributing to WordPress on GitHub can be found in the Core Handbook. |
|
The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the Core Committers: Use this line as a base for the props when committing in SVN: To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook. |
Test using WordPress PlaygroundThe changes in this pull request can previewed and tested using a WordPress Playground instance. WordPress Playground is an experimental project that creates a full WordPress instance entirely within the browser. Some things to be aware of
For more details about these limitations and more, check out the Limitations page in the WordPress Playground documentation. |
|
Oh, it's very confusing. I wander that why there is only one change in the file but there are 27 commits? @deepakrohillas deepakrohilla Could you explain it for me? Thank you ! |
|
@DingGuodong, this is due to code committed in the branch and some upstream merges. Only the new changes need to be reviewed. When you have a moment, please review them and share your feedback here. |
mukeshpanchal27
left a comment
There was a problem hiding this comment.
Thanks for PR! In quick review LGTM.
Thanks for your reply. So, I think it is due to the committer pull some commits from another branch before request this PR. |
|
Thanks for the PR! Merged in r59497. |
https://core.trac.wordpress.org/ticket/62009