Skip to content

Plugin: Remove PHP functions slated for removal in 5.2#14090

Merged
aduth merged 2 commits intomasterfrom
remove/5-2-deprecated-php
Feb 27, 2019
Merged

Plugin: Remove PHP functions slated for removal in 5.2#14090
aduth merged 2 commits intomasterfrom
remove/5-2-deprecated-php

Conversation

@aduth
Copy link
Copy Markdown
Member

@aduth aduth commented Feb 25, 2019

This pull request seeks to remove functions deprecated as part of the Gutenberg 5.0 release, slated for removal with the upcoming version 5.2.

Implementation notes:

As noted at #14012 (comment), a few functions were wrongly labeled as deprecated with 5.0 when in fact they were deprecated in the 5.1 release. Those have not been removed here, and their version deprecations have been updated to align to the deprecations document, where they are correctly noted as being removed in the following 5.3 release:

https://github.com/WordPress/gutenberg/blob/master/docs/designers-developers/developers/backward-compatibility/deprecations.md

The script bin/get-server-blocks.php still included a reference to gutenberg_prepare_blocks_for_js. The deprecated warning was not logged because the file disables error reporting. This has been updated accordingly here. Separately, we should consider:

  • Enable error reporting
  • Run the fixtures regeneration, because its out of sync
  • Find ways to avoid ever allowing (a) the fixtures to fall out of sync or (b) silently swallow errors

Related: #13583 (comment)

Testing instructions:

Verify there are no regressions in the load and operation of the error.

Verify there are no warnings or errors logged (by PHP) when loading the editor (with and without meta boxes) or the reusable blocks list screen.

Ensure there are no remaining occurrences of deprecated functions to be removed in the 5.2 release.

Loading
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Gutenberg Plugin Issues or PRs related to Gutenberg Plugin management related efforts

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants