Skip to content
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.

Sign up for GitHub

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

Jump to bottom

PHP warnings appearing in the dashboard #960

Closed
AndreeaCristinaRadacina opened this issue Jul 16, 2024 · 4 comments · Fixed by #961 or #962
Closed

PHP warnings appearing in the dashboard #960

AndreeaCristinaRadacina opened this issue Jul 16, 2024 · 4 comments · Fixed by #961 or #962
Assignees
@girishpanchal30
Labels
bug This label could be used to identify issues that are caused by a defect in the product. customer report Indicates the request came from a customer. Priority-Medium Expected resolution time - up to 1 month. released Indicate that an issue has been resolved and released in a particular version of the product.

Comments

@AndreeaCristinaRadacina
Copy link

Description

Some users mention that the latest version of Feedzy contains PHP warnings:
PHP Warning: Undefined variable $version in /wp-content/plugins/feedzy-rss-feeds/includes/gutenberg/feedzy-rss-feeds-gutenberg-block.php on line 79
PHP Warning: Undefined variable $version in /wp-content/plugins/feedzy-rss-feeds/includes/gutenberg/feedzy-rss-feeds-gutenberg-block.php on line 80

I wasn't able to replicate the issue.
ref: https://wordpress.org/support/topic/php-warning-undefined-variable-version/#post-17884659

Step-by-step reproduction instructions

  1. Create a page with Gutenberg and add a Feedzy block
  2. Add a feed source and publish
  3. Update Feedzy to the latest version

Screenshots, screen recording, code snippet or Help Scout ticket

https://plugintests.com/plugins/wporg/feedzy-rss-feeds
https://plugintests.com/plugins/wporg/feedzy-rss-feeds/latest

Environment info

No response

Is the issue you are reporting a regression

Yes, this is a regression.
@AndreeaCristinaRadacina AndreeaCristinaRadacina added the bug This label could be used to identify issues that are caused by a defect in the product. label Jul 16, 2024
@pirate-bot pirate-bot added customer report Indicates the request came from a customer. regression Issue represents a change in behavior from a previous version that is not intended or desired.. labels Jul 16, 2024
@vytisbulkevicius
Copy link
Contributor

I can replicate this on a test instance.

https://sceptical-giraffe-96690f.vertisite.cloud/wp-admin
login: karelitewe0107
pw: 8feIa2CDk1UpHKoPBtl3

When refreshing a post in the dashboard, it's appearing on top: https://sceptical-giraffe-96690f.vertisite.cloud/wp-admin/post.php?post=34&action=edit

image

@vytisbulkevicius vytisbulkevicius removed the regression Issue represents a change in behavior from a previous version that is not intended or desired.. label Jul 19, 2024
@vytisbulkevicius vytisbulkevicius changed the title PHP warnings in the latest version PHP warnings appearing in the dashboard Jul 19, 2024
@vytisbulkevicius vytisbulkevicius added the Priority-Medium Expected resolution time - up to 1 month. label Jul 19, 2024
@girishpanchal30 girishpanchal30 mentioned this issue Jul 19, 2024
Merged
6 tasks
@girishpanchal30
Copy link
Contributor

@AndreeaCristinaRadacina @vytisbulkevicius I've reviewed this issue and fixed it here - #961

@girishpanchal30 girishpanchal30 linked a pull request Jul 19, 2024 that will close this issue
Fixed PHP warnings #961
Merged
6 tasks
@AndreeaCristinaRadacina
Copy link
Author

I tested, and it works fine now. 🚀

@vytisbulkevicius vytisbulkevicius linked a pull request Jul 25, 2024 that will close this issue
Release #962
Merged
@pirate-bot
Copy link
Contributor

🎉 This issue has been resolved in version 4.4.12 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@pirate-bot pirate-bot added the released Indicate that an issue has been resolved and released in a particular version of the product. label Jul 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@girishpanchal30 girishpanchal30

Labels
bug This label could be used to identify issues that are caused by a defect in the product. customer report Indicates the request came from a customer. Priority-Medium Expected resolution time - up to 1 month. released Indicate that an issue has been resolved and released in a particular version of the product.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fixed PHP warnings Codeinwp/feedzy-rss-feeds
Release Codeinwp/feedzy-rss-feeds
4 participants
@vytisbulkevicius @pirate-bot @girishpanchal30 @AndreeaCristinaRadacina