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

Fatal Error with Gutenberg plugin v5.3.0 because of deprecated/removed function #7557

Closed
deckerweb opened this issue Mar 20, 2019 · 2 comments
Closed

Fatal Error with Gutenberg plugin v5.3.0 because of deprecated/removed function #7557

deckerweb opened this issue Mar 20, 2019 · 2 comments
Labels
compatibility/3rd-party Indicates a compatibility problem with a 3rd-party plugin or theme. status/merged Indicates when a Pull Request has been merged to a Release.

Comments

@deckerweb
Copy link

The function gutenberg_can_edit_post_type() from the Gutenberg plugin was deprecated with 5.1.0 and now got removed in 5.3.0 of the plugin. See: WordPress/classic-editor#87

You should switch to the WP Core function use_block_editor_for_post_type()

This effects the file /elementor/includes/compatibility.php, L54, as of Elementor 2.5.9:
Change:
! gutenberg_can_edit_post_type( $typenow )
to:
! use_block_editor_for_post_type( $typenow )
bainternet added a commit to bainternet/elementor that referenced this issue Mar 20, 2019
@bainternet
Fix: Gutenberg Plugin compatibility (elementor#7557)
9674c1f
@bainternet bainternet added the compatibility/3rd-party Indicates a compatibility problem with a 3rd-party plugin or theme. label Mar 20, 2019
@bainternet
Copy link
Contributor

Thanks for reporting.

should be fixed in next version.

@bainternet bainternet added the status/has-pr Indicates that an Issue, or Discussion has a companion Pull Request awaiting to be merged. label Mar 20, 2019
KingYes added a commit that referenced this issue Mar 26, 2019
@KingYes
Merge pull request #7560 from bainternet/fix/i-7557
36b7cc6 
Fix: Gutenberg Plugin compatibility (#7557)
@shilo-ey shilo-ey added status/merged Indicates when a Pull Request has been merged to a Release. and removed status/has-pr Indicates that an Issue, or Discussion has a companion Pull Request awaiting to be merged. labels Mar 26, 2019
@shilo-ey
Copy link
Contributor

Hi @deckerweb

This issue has been fixed in v2.5.10

Feel free to update!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
compatibility/3rd-party Indicates a compatibility problem with a 3rd-party plugin or theme. status/merged Indicates when a Pull Request has been merged to a Release.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@bainternet @deckerweb @shilo-ey