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.

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

PHP Notice "Undefined variable: screen" in meta-boxes.php:1365 #11469

Closed
dmccan opened this issue Nov 4, 2018 · 1 comment
Closed

PHP Notice "Undefined variable: screen" in meta-boxes.php:1365 #11469

dmccan opened this issue Nov 4, 2018 · 1 comment
Labels
[Type] WP Core Ticket Requires an upstream change from WordPress. Core Trac ticket should be linked.

Comments

@dmccan
Copy link

dmccan commented Nov 4, 2018

When going to create a new Wiki post I receive the notice:

NOTICE: E:\laragon\www\betatest\wp-admin\includes\meta-boxes.php:1365 - Undefined variable: screen
include('E:\laragon\www\betatest\wp-admin\edit-form-advanced.php'), register_and_do_post_meta_boxes

To Reproduce
Steps to reproduce the behavior:

  • Install the latest version 5 beta

  • Install and activate the plugin Yada Wiki from the WP directory

  • With debug set to true, go to the Admin menu "Wiki Pages -> Add New" and you should see the warning notice.

  • In the file \wp-content\plugins\yada-wiki\includes\register-wiki-cpt.php change line 40 from this:

'supports'  => array( 'title', 'editor', 'author', 'thumbnail', 'revisions', 'publicize', 'wpcom-markdown' ),

To this (adding support for custom fields) to have the notice go away:

'supports'   => array( 'title', 'editor', 'author', 'thumbnail', 'revisions', 'publicize', 'wpcom-markdown', 'custom-fields' ),

Expected behavior
There is no warning notice when using WP 4.9.8. Expect the same with WP 5

Additional context

  • Using the default WP theme.
  • The only other plugins activated are WordPress Beta Tester and Debug Bar
@ocean90 ocean90 added the [Type] WP Core Ticket Requires an upstream change from WordPress. Core Trac ticket should be linked. label Nov 4, 2018
@ocean90
Copy link
Member

ocean90 commented Nov 4, 2018

Thanks for the report! I have reopened the related core ticket #45257.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Type] WP Core Ticket Requires an upstream change from WordPress. Core Trac ticket should be linked.
Projects
None yet
Development

No branches or pull requests

2 participants