WordPress.org
Get WordPress

Make WordPress Core

Opened 5 years ago

Closed 5 years ago

Last modified 5 years ago

#53289 closed defect (bug) (fixed)

Add missing delete_widget action to REST endpoint

Reported by: isabel_brison Owned by: noisysocks
Priority: normal Milestone: 5.8
Component: REST API Version:
Severity: normal Keywords: has-patch
Cc: Focuses:

Description

As documented in ​https://github.com/WordPress/gutenberg/issues/31126, the delete_widget action is missing for the new Widgets screen.

Change History (6)

​TimothyBJacobs commented on ​PR #1309:
5 years ago #2

We need to do the same globals setup to match what they would have on the old screen, ​https://github.com/WordPress/wordpress-develop/blob/master/src/wp-admin/widgets-form.php#L157

#3 @SergeyBiryukov
5 years ago

  • Milestone Awaiting Review → 5.8

​noisysocks commented on ​PR #1309:
5 years ago #4

We need to do the same globals setup to match what they would have on the old screen, ​https://github.com/WordPress/wordpress-develop/blob/master/src/wp-admin/widgets-form.php#L157

​https://github.com/WordPress/wordpress-develop/pull/1323 adds that $_POST global.

@tellthemachines: Let's commit ​https://github.com/WordPress/wordpress-develop/pull/1323 first and then rebase this so that we have that $_POST global.

#5 @noisysocks
5 years ago

  • Owner set to noisysocks
  • Resolution → fixed
  • Status new → closed

In 51060:

REST API: Add 'delete_widget' action to delete widget endpoint

Triggers the 'delete_widget' action to the delete widget endpoint in the REST
API. This aligns with the behaviour in the widgets WP Admin screen.

Fixes #53289.
Props isabel_brison, TimothyBlynJacobs.

​tellthemachines commented on ​PR #1309:
5 years ago #6

Closing as this has been committed already.

Note: See TracTickets for help on using tickets.