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

Fix PHP Notice: Check that the query arg exists before using it #25006

Merged
merged 1 commit into from Sep 2, 2020
Merged

Fix PHP Notice: Check that the query arg exists before using it #25006

merged 1 commit into from Sep 2, 2020

Conversation

aristath
Copy link
Member

@aristath aristath commented Sep 2, 2020 •
edited

Got an error log full of errors a few days ago while working on FSE:

PHP Notice:  Undefined index: query in wp-content/plugins/gutenberg/build/block-library/blocks/query-loop.php on line 24
PHP Stack trace:
PHP   1. {main}() index.php:0
PHP   2. require() index.php:17
PHP   3. require_once() wp-blog-header.php:19
PHP   4. include() wp-includes/template-loader.php:106
PHP   5. gutenberg_render_the_template() wp-content/plugins/gutenberg/lib/template-canvas.php:19
PHP   6. do_blocks() wp-content/plugins/gutenberg/lib/template-loader.php:378
PHP   7. render_block() wp-includes/blocks.php:758
PHP   8. apply_filters() wp-includes/blocks.php:670
PHP   9. WP_Hook->apply_filters() wp-includes/plugin.php:206
PHP  10. gutenberg_render_block_with_assigned_block_context() wp-includes/class-wp-hook.php:287
PHP  11. WP_Block->render() wp-content/plugins/gutenberg/lib/compat.php:431
PHP  12. WP_Block->render() wp-includes/class-wp-block.php:211
PHP  13. gutenberg_render_block_core_query_loop() wp-includes/class-wp-block.php:217

This PR adds some additional checks to ensure that $block->context['query'] is set before using it.

How has this been tested?

PHP notices no longer occur after the commit.

Types of changes

Bug fix: Added an isset() check to wrap parts of the query that use the previously undefined properties.

Checklist:

  • My code is tested.
  • My code follows the WordPress code style.
  • My code follows the accessibility standards.
  • My code has proper inline documentation. ( Not applicable)
  • I've included developer documentation if appropriate. ( Not applicable)
  • I've updated all React Native files affected by any refactorings/renamings in this PR. ( Not applicable)

@aristath aristath mentioned this pull request Sep 2, 2020
Closed
6 tasks
@github-actions
Copy link

Size Change: 0 B

Total Size: 1.17 MB

ℹ️ View Unchanged
Filename Size Change
build/a11y/index.js 1.14 kB 0 B
build/annotations/index.js 3.67 kB 0 B
build/api-fetch/index.js 3.41 kB 0 B
build/autop/index.js 2.82 kB 0 B
build/blob/index.js 620 B 0 B
build/block-directory/index.js 7.99 kB 0 B
build/block-directory/style-rtl.css 953 B 0 B
build/block-directory/style.css 952 B 0 B
build/block-editor/index.js 128 kB 0 B
build/block-editor/style-rtl.css 10.8 kB 0 B
build/block-editor/style.css 10.8 kB 0 B
build/block-library/editor-rtl.css 8.55 kB 0 B
build/block-library/editor.css 8.55 kB 0 B
build/block-library/index.js 137 kB 0 B
build/block-library/style-rtl.css 7.47 kB 0 B
build/block-library/style.css 7.47 kB 0 B
build/block-library/theme-rtl.css 741 B 0 B
build/block-library/theme.css 742 B 0 B
build/block-serialization-default-parser/index.js 1.88 kB 0 B
build/block-serialization-spec-parser/index.js 3.1 kB 0 B
build/blocks/index.js 47.7 kB 0 B
build/components/index.js 200 kB 0 B
build/components/style-rtl.css 15.5 kB 0 B
build/components/style.css 15.5 kB 0 B
build/compose/index.js 9.67 kB 0 B
build/core-data/index.js 12.3 kB 0 B
build/data-controls/index.js 1.29 kB 0 B
build/data/index.js 8.55 kB 0 B
build/date/index.js 5.38 kB 0 B
build/deprecated/index.js 772 B 0 B
build/dom-ready/index.js 568 B 0 B
build/dom/index.js 4.48 kB 0 B
build/edit-navigation/index.js 11.7 kB 0 B
build/edit-navigation/style-rtl.css 1.16 kB 0 B
build/edit-navigation/style.css 1.16 kB 0 B
build/edit-post/index.js 305 kB 0 B
build/edit-post/style-rtl.css 6.26 kB 0 B
build/edit-post/style.css 6.24 kB 0 B
build/edit-site/index.js 17 kB 0 B
build/edit-site/style-rtl.css 3.06 kB 0 B
build/edit-site/style.css 3.06 kB 0 B
build/edit-widgets/index.js 12 kB 0 B
build/edit-widgets/style-rtl.css 2.46 kB 0 B
build/edit-widgets/style.css 2.45 kB 0 B
build/editor/editor-styles-rtl.css 492 B 0 B
build/editor/editor-styles.css 493 B 0 B
build/editor/index.js 45.4 kB 0 B
build/editor/style-rtl.css 3.81 kB 0 B
build/editor/style.css 3.81 kB 0 B
build/element/index.js 4.65 kB 0 B
build/escape-html/index.js 733 B 0 B
build/format-library/index.js 7.71 kB 0 B
build/format-library/style-rtl.css 547 B 0 B
build/format-library/style.css 548 B 0 B
build/hooks/index.js 2.13 kB 0 B
build/html-entities/index.js 621 B 0 B
build/i18n/index.js 3.57 kB 0 B
build/is-shallow-equal/index.js 710 B 0 B
build/keyboard-shortcuts/index.js 2.52 kB 0 B
build/keycodes/index.js 1.94 kB 0 B
build/list-reusable-blocks/index.js 3.12 kB 0 B
build/list-reusable-blocks/style-rtl.css 476 B 0 B
build/list-reusable-blocks/style.css 476 B 0 B
build/media-utils/index.js 5.32 kB 0 B
build/notices/index.js 1.79 kB 0 B
build/nux/index.js 3.4 kB 0 B
build/nux/style-rtl.css 671 B 0 B
build/nux/style.css 668 B 0 B
build/plugins/index.js 2.56 kB 0 B
build/primitives/index.js 1.41 kB 0 B
build/priority-queue/index.js 789 B 0 B
build/redux-routine/index.js 2.85 kB 0 B
build/rich-text/index.js 13.9 kB 0 B
build/server-side-render/index.js 2.77 kB 0 B
build/shortcode/index.js 1.7 kB 0 B
build/token-list/index.js 1.27 kB 0 B
build/url/index.js 4.06 kB 0 B
build/viewport/index.js 1.85 kB 0 B
build/warning/index.js 1.14 kB 0 B
build/wordcount/index.js 1.17 kB 0 B

compressed-size-action

@aristath aristath changed the title Fix PHP Notice: Check that the query arg exists before using it Fix PHP Notice: Check that the query arg exists before using it Sep 2, 2020
ntsekouras
ntsekouras approved these changes Sep 2, 2020
View reviewed changes
Copy link
Contributor

@ntsekouras ntsekouras left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM thanks! 👍

@aristath aristath merged commit 668182c into master Sep 2, 2020
@aristath aristath deleted the fix/query-loop-php-notice branch September 2, 2020 13:21
@github-actions github-actions bot added this to the Gutenberg 9.0 milestone Sep 2, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ntsekouras ntsekouras ntsekouras approved these changes

@ajitbohra ajitbohra Awaiting requested review from ajitbohra

@Soean Soean Awaiting requested review from Soean

@talldan talldan Awaiting requested review from talldan

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
Gutenberg 9.0
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@aristath @ntsekouras