WordPress.org
Get WordPress

Make WordPress Core

Opened 5 years ago

Closed 5 years ago

#53441 closed enhancement (fixed)

Remove unneeded gutenberg functions

Reported by: spacedmonkey Owned by: noisysocks
Priority: normal Milestone: 5.8
Component: Widgets Version: 5.8
Severity: normal Keywords: has-patch
Cc: Focuses:

Description

Remove unneeded gutenberg dev functions from core.

Attachments (1)

fatal.png​ (209.5 KB ) - added by desrosj 5 years ago.

Download all attachments as: .zip

Change History (12)

#2 @spacedmonkey
5 years ago

  • Owner set to noisysocks
  • Status new → assigned

#3 @noisysocks
5 years ago

Thanks. The changes to src/wp-includes/blocks/legacy-widget.php need to happen in Gutenberg as that file is copied over by grunt from @wordpress/widgets.

#4 @spacedmonkey
5 years ago

@noisysocks I have fixed it as part of ​https://github.com/WordPress/gutenberg/pull/32781. If you are interested.

#5 @noisysocks
5 years ago

Cool let's wait for GB32781 to be backported.

@desrosj
5 years ago

#6 @desrosj
5 years ago

Just noting this is causing a fatal error for me currently. gutenberg_get_widget_key() was never included in trunk. I think this is good to go. Going to ping in #core-editor to get final confirmation this can be merged.

This ticket was mentioned in ​Slack in #core-editor by desrosj. ​View the logs.
5 years ago

#9 @spacedmonkey
5 years ago

I have a workaround in the gutenberg package. See ​https://github.com/WordPress/gutenberg/pull/32902

This would require merge, package to be updated and then packaged update in WordPress core.

This ticket was mentioned in ​Slack in #core by spacedmonkey. ​View the logs.
5 years ago

#11 @noisysocks
5 years ago

  • Resolution → fixed
  • Status assigned → closed

In 51219:

Widgets: Remove unnecessary gutenberg_ functions

Removes temporary gutenberg_ functions which were required by the Legacy Widget
block until the Legacy Widget block was updated in [51149].

Follows [50996].
Fixes #53441.
Props spacedmonkey.

Note: See TracTickets for help on using tickets.