Changeset 51074 for trunk/src/wp-includes/rest-api/endpoints/class-wp-rest-widgets-controller.php

Timestamp:

06/04/2021 11:12:56 PM (5 years ago)

Author:

SergeyBiryukov

Message:

REST API: Restore the $creating parameter of rest_after_save_widget action.

This is consistent with other similar REST API actions.

Partially reverts [51071], except for DocBlock formatting fixes.

Props TimothyBlynJacobs.
See #53317.

File:

• trunk/src/wp-includes/rest-api/endpoints/class-wp-rest-widgets-controller.php (modified) (3 diffs)

trunk/src/wp-includes/rest-api/endpoints/class-wp-rest-widgets-controller.php

@@ -415 +415 @@
                         $number        = isset( $parsed_id['number'] ) ? $parsed_id['number'] : null;
                         $widget_object = $wp_widget_factory->get_widget_object( $id_base );
-                        $update        = true;
+                        $creating      = false;
                 } elseif ( $request['id_base'] ) {
                         // Saving a new widget.
@@ -422 +422 @@
                         $number        = $widget_object ? next_widget_id_number( $id_base ) : null;
                         $id            = $widget_object ? $id_base . '-' . $number : $id_base;
-                        $update        = false;
+                        $creating      = true;
                 } else {
                         return new WP_Error(
@@ -526 +526 @@
                  * @param string          $sidebar_id ID of the sidebar containing the widget being saved.
                  * @param WP_REST_Request $request    Request object.
-                 * @param bool            $update     Whether this is an existing widget being updated.
+                 * @param bool            $creating   True when creating a widget, false when updating.
                  */
-                do_action( 'rest_after_save_widget', $id, $sidebar_id, $request, $update );
+                do_action( 'rest_after_save_widget', $id, $sidebar_id, $request, $creating );
 
                 return $id;