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

Timestamp:

06/04/2021 11:12:56 PM (4 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;