Plugin: Bump minimum WordPress version to 5.3 by mcsf · Pull Request #20628 · WordPress/gutenberg

mcsf · 2020-03-04T17:17:50Z

Description

Fixes #20484

With the release of RC1 for WordPress 5.4, and given the constraints raised in #20484, this PR bumps the required WordPress version for running the Gutenberg plugin from 5.2 to 5.3.

In the process, this PR eliminates the interception of core's block_editor_preload_paths filter, which was in place to provide compatibility for the following features:

Preload autosaves (Add preload paths for autosaves #15067). Landed in core in 5.3.
Preload blocks permissions (Reusable Blocks: Preload user permissions #15061). Landed in core in 5.2.

@talldan, @aduth: As authors of the aforementioned PRs, could you confirm these removals for me?

How has this been tested?

Screenshots

Types of changes

Checklist:

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

github-actions · 2020-03-04T17:22:42Z

Size Change: -2.26 kB (0%)

Total Size: 863 kB

Filename	Size	Change
`build/annotations/index.js`	3.43 kB	-1 B
`build/api-fetch/index.js`	3.39 kB	-1 B
`build/block-editor/index.js`	104 kB	-411 B (0%)
`build/block-editor/style-rtl.css`	10.5 kB	+24 B (0%)
`build/block-editor/style.css`	10.5 kB	+21 B (0%)
`build/block-library/editor-rtl.css`	7.26 kB	-100 B (1%)
`build/block-library/editor.css`	7.26 kB	-100 B (1%)
`build/block-library/index.js`	115 kB	-776 B (0%)
`build/blocks/index.js`	57.7 kB	+50 B (0%)
`build/components/index.js`	191 kB	-126 B (0%)
`build/components/style-rtl.css`	15.5 kB	-18 B (0%)
`build/components/style.css`	15.5 kB	-12 B (0%)
`build/compose/index.js`	5.75 kB	-2 B (0%)
`build/date/index.js`	5.36 kB	-4 B (0%)
`build/dom-ready/index.js`	569 B	+1 B
`build/edit-post/index.js`	91.3 kB	-213 B (0%)
`build/edit-post/style-rtl.css`	8.71 kB	+41 B (0%)
`build/edit-post/style.css`	8.71 kB	+44 B (0%)
`build/edit-site/style-rtl.css`	2.48 kB	-28 B (1%)
`build/edit-site/style.css`	2.48 kB	-27 B (1%)
`build/edit-widgets/style-rtl.css`	2.59 kB	-2 B (0%)
`build/edit-widgets/style.css`	2.58 kB	-2 B (0%)
`build/editor/index.js`	43.8 kB	-117 B (0%)
`build/escape-html/index.js`	734 B	+1 B
`build/format-library/index.js`	7.11 kB	-492 B (6%)	✅
`build/hooks/index.js`	1.92 kB	+1 B
`build/keyboard-shortcuts/index.js`	2.3 kB	+1 B
`build/list-reusable-blocks/index.js`	2.99 kB	+1 B
`build/nux/index.js`	3.01 kB	-16 B (0%)
`build/plugins/index.js`	2.54 kB	-1 B
`build/redux-routine/index.js`	2.84 kB	+1 B
`build/rich-text/index.js`	14.3 kB	+1 B
`build/server-side-render/index.js`	2.55 kB	+6 B (0%)
`build/url/index.js`	4 kB	-1 B
`build/warning/index.js`	1.14 kB	-1 B

ℹ️ View Unchanged

Filename	Size	Change
`build/a11y/index.js`	1.01 kB	0 B
`build/autop/index.js`	2.58 kB	0 B
`build/blob/index.js`	620 B	0 B
`build/block-directory/index.js`	6.02 kB	0 B
`build/block-directory/style-rtl.css`	760 B	0 B
`build/block-directory/style.css`	760 B	0 B
`build/block-library/style-rtl.css`	7.5 kB	0 B
`build/block-library/style.css`	7.51 kB	0 B
`build/block-library/theme-rtl.css`	669 B	0 B
`build/block-library/theme.css`	671 B	0 B
`build/block-serialization-default-parser/index.js`	1.65 kB	0 B
`build/block-serialization-spec-parser/index.js`	3.1 kB	0 B
`build/core-data/index.js`	10.6 kB	0 B
`build/data-controls/index.js`	1.03 kB	0 B
`build/data/index.js`	8.22 kB	0 B
`build/deprecated/index.js`	772 B	0 B
`build/dom/index.js`	3.06 kB	0 B
`build/edit-site/index.js`	4.64 kB	0 B
`build/edit-widgets/index.js`	4.42 kB	0 B
`build/editor/editor-styles-rtl.css`	325 B	0 B
`build/editor/editor-styles.css`	327 B	0 B
`build/editor/style-rtl.css`	3.98 kB	0 B
`build/editor/style.css`	3.98 kB	0 B
`build/element/index.js`	4.45 kB	0 B
`build/format-library/style-rtl.css`	502 B	0 B
`build/format-library/style.css`	502 B	0 B
`build/html-entities/index.js`	622 B	0 B
`build/i18n/index.js`	3.48 kB	0 B
`build/is-shallow-equal/index.js`	710 B	0 B
`build/keycodes/index.js`	1.68 kB	0 B
`build/list-reusable-blocks/style-rtl.css`	226 B	0 B
`build/list-reusable-blocks/style.css`	226 B	0 B
`build/media-utils/index.js`	4.85 kB	0 B
`build/notices/index.js`	1.57 kB	0 B
`build/nux/style-rtl.css`	616 B	0 B
`build/nux/style.css`	613 B	0 B
`build/primitives/index.js`	1.49 kB	0 B
`build/priority-queue/index.js`	780 B	0 B
`build/shortcode/index.js`	1.7 kB	0 B
`build/token-list/index.js`	1.27 kB	0 B
`build/viewport/index.js`	1.61 kB	0 B
`build/wordcount/index.js`	1.18 kB	0 B

_{compressed-size-action}

Soean · 2020-03-04T19:29:43Z

We also can remove gutenberg_safe_style_css_column_flex_basis and maybe gutenberg_register_vendor_scripts? @gziolo

talldan · 2020-03-05T09:18:07Z

@mcsf That looks good to me, thanks for handling that!

gziolo · 2020-03-05T10:18:02Z

We also can remove gutenberg_safe_style_css_column_flex_basis and maybe gutenberg_register_vendor_scripts? @gziolo

I would keep gutenberg_register_vendor_scripts because we are going to bump React and lodash in the future and we need to test that in the plugin first. Those versions align with WP core but at least React had 4 minor releases in the meantime. We can remove lodash from the function because it's still the latest version.

Soean · 2020-03-05T10:35:04Z

@gziolo Ok, then we should adjust the the TODO inside the function:

gutenberg/lib/client-assets.php

Lines 214 to 215 in 287498f

    
           // TODO: Overrides for react, react-dom and lodash are necessary 
        
           // until WordPress 5.3 is released.

mcsf · 2020-03-05T15:03:53Z

@Soean, @gziolo: I've pushed 1018968 and 9ac1e2a, please have a look. 🙇 For the TODO note, I'll let you work it out.

aduth · 2020-03-05T15:34:45Z

Preload blocks permissions (Reusable Blocks: Preload user permissions #15061). Landed in core in 5.2.

@talldan, @aduth: As authors of the aforementioned PRs, could you confirm these removals for me?

Confirmed via: https://github.com/WordPress/WordPress/blob/5.3/wp-admin/edit-form-blocks.php#L51

aduth · 2020-03-05T15:57:01Z

I don't think we need the React or Lodash registrations? As of WordPress 5.3, the versions are the same as what we're overriding:

https://github.com/WordPress/WordPress/blob/5.3/wp-includes/script-loader.php#L94-L97

To @gziolo's point in #20628 (comment), it might be something where we want to keep the function for the sake of anticipating some future use. I don't feel strongly one way or the other, but it would be easy enough to refer back to the history of the file to restore the function if and when it's needed once more.

mcsf · 2020-03-05T16:38:02Z

I really don't feel strongly about this; I can either remove gutenberg_register_vendor_scripts entirely, or hollow it out, or something else. State your preferences. :)

aduth · 2020-03-05T18:39:47Z

I'd suggest to just remove it altogether.

mcsf · 2020-03-05T19:12:32Z

I'd suggest to just remove it altogether.

I starting doing this, then realised that gutenberg_register_vendor_scripts is used by the script bin/get-vendor-scripts.php, itself used by bin/build-plugin-zip.sh. I could remove get-vendor-scripts and remove the logic in build-plugin-zip to skip that step, but at this point I'm thinking that it's a lot of useful machinery that we might bring back soon. Despite YAGNI, now I'm leaning more towards emptying the function and leaving it in place, so that every other piece can still rely on it.

I've tentatively pushed 1be218b. Thoughts?

aduth · 2020-03-05T19:20:12Z

Fair enough. I don't feel too strongly about it, and agree that it would be a pain to bring everything back in place at a later point. One suggestion I might make is to include an inline code comment explaining why we have an empty function there.

gziolo · 2020-03-05T21:53:51Z

Agreed, a code comment with a link to this PR should make it easier to restore code when we need another override in the future.

mcsf · 2020-03-06T00:16:28Z

Done.

aduth · 2020-03-06T16:12:55Z

I'm glad you raised the point about the plugin build script and its implementation for downloading vendor dependencies. In revisiting my own #20225, I sense that the use of gutenberg_register_vendor_script there would not be sufficient to account for the polyfill in the plugin build. It's already the case prior to that pull request, too:

https://plugins.trac.wordpress.org/browser/gutenberg/tags/7.6.0/vendor
https://plugins.trac.wordpress.org/browser/gutenberg/tags/7.6.0/lib/compat.php#L58
https://plugins.trac.wordpress.org/browser/gutenberg/tags/7.6.0/lib/compat.php#L101

Trying to think how best to address this (without also needlessly blocking this pull request or #20225). I imagine the solution would be one of either (a) move these gutenberg_register_vendor_script into gutenberg_register_vendor_scripts or (b) update the implementation of get-vendor-scripts.php to account for more than just gutenberg_register_vendor_scripts.

aduth

Per my previous comment, I think it will be easiest to merge this, then I can rebase my other pull request with the necessary changes to account for the polyfill vendor scripts.

The changes here look sensible to me 👍

mcsf · 2020-03-06T16:23:22Z

Thanks, I appreciate the deep dive and unblocking.

Plugin: Bump minimum WordPress version to 5.3

f33e00d

mcsf added the Gutenberg Plugin Issues or PRs related to Gutenberg Plugin management related efforts label Mar 4, 2020

mcsf requested review from aduth and talldan March 4, 2020 17:17

mcsf requested a review from TimothyBJacobs as a code owner March 4, 2020 17:17

mcsf mentioned this pull request Mar 4, 2020

Gutenberg is unintentionally incompatible with WordPress 5.2.x. #20484

Closed

mcsf added 2 commits March 5, 2020 14:43

Compat: Remove gutenberg_safe_style_css_column_flex_basis

1018968

Remove overriding of lodash script

9ac1e2a

mcsf force-pushed the update/minimum-wp-version-53 branch from 8bdd5ee to 9ac1e2a Compare March 5, 2020 15:03

Remove overriding of react and react-dom scripts

1be218b

mcsf added 2 commits March 5, 2020 22:09

Explain emptying of gutenberg_register_vendor_scripts

d457ecf

Appease phpcs

904bf9b

aduth approved these changes Mar 6, 2020

View reviewed changes

mcsf merged commit 3ab3727 into master Mar 6, 2020

mcsf deleted the update/minimum-wp-version-53 branch March 6, 2020 16:23

github-actions Bot added this to the Gutenberg 7.7 milestone Mar 6, 2020

aduth mentioned this pull request Mar 6, 2020

Compat: Use core-js-url-browser for URL polyfill #20225

Merged

aduth mentioned this pull request Mar 16, 2020

Framework: Migrate/remove temporary compatibility script initialization #19178

Merged

ellatrix mentioned this pull request Jun 16, 2020

WP 5.5: merge all PHP changes #23210

Closed

12 tasks

Uh oh!

Conversation

mcsf commented Mar 4, 2020

Description

How has this been tested?

Screenshots

Types of changes

Checklist:

Uh oh!

github-actions Bot commented Mar 4, 2020 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Soean commented Mar 4, 2020

Uh oh!

talldan commented Mar 5, 2020

Uh oh!

gziolo commented Mar 5, 2020

Uh oh!

Soean commented Mar 5, 2020

Uh oh!

mcsf commented Mar 5, 2020

Uh oh!

aduth commented Mar 5, 2020

Uh oh!

aduth commented Mar 5, 2020

Uh oh!

mcsf commented Mar 5, 2020

Uh oh!

aduth commented Mar 5, 2020

Uh oh!

mcsf commented Mar 5, 2020

Uh oh!

aduth commented Mar 5, 2020

Uh oh!

gziolo commented Mar 5, 2020

Uh oh!

mcsf commented Mar 6, 2020

Uh oh!

aduth commented Mar 6, 2020

Uh oh!

aduth left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

mcsf commented Mar 6, 2020

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

github-actions Bot commented Mar 4, 2020 •

edited

Loading

aduth left a comment •

edited

Loading