Changeset 56082

Timestamp:

06/28/2023 03:33:23 AM (16 months ago)

Author:

isabel_brison

Message:

REST API: add revisions endpoint for global styles.

Adds an endpoint that returns revisions to the global styles custom post.

Props ramonopoly, peterwilsoncc, spacedmonkey, mukesh27, timothyblynjacobs.
Fixes #58524.

Location:

trunk

Files:

• package-lock.json (modified) (4 diffs)
• src/wp-includes/rest-api.php (modified) (1 diff)
• src/wp-includes/rest-api/endpoints/class-wp-rest-global-styles-controller.php (modified) (2 diffs)
• src/wp-includes/rest-api/endpoints/class-wp-rest-global-styles-revisions-controller.php (added)
• src/wp-settings.php (modified) (1 diff)
• tests/phpunit/tests/rest-api/rest-global-styles-revisions-controller.php (added)
• tests/phpunit/tests/rest-api/rest-schema-setup.php (modified) (1 diff)
• tests/qunit/fixtures/wp-api-generated.js (modified) (1 diff)

trunk/package-lock.json

@@ -12729 +12729 @@
             "version": "1.1.0",
             "resolved": "https://registry.npmjs.org/from2-string/-/from2-string-1.1.0.tgz",
-            "integrity": "sha1-GCgrJ9CKJnyzAwzSuLSw8hKvdSo=",
+            "integrity": "sha512-m8vCh+KnXXXBtfF2VUbiYlQ+nczLcntB0BrtNgpmLkHylhObe9WF1b2LZjBBzrZzA6P4mkEla6ZYQoOUTG8cYA==",
             "requires": {
                 "from2": "^2.0.3"
@@ -21800 +21800 @@
             "version": "1.1.6",
             "resolved": "https://registry.npmjs.org/promise-polyfill/-/promise-polyfill-1.1.6.tgz",
-            "integrity": "sha1-zQTv9G9clcOn0EVZHXm14+AfEtc="
+            "integrity": "sha512-7rrONfyLkDEc7OJ5QBkqa4KI4EBhCd340xRuIUPGCfu13znS+vx+VDdrT9ODAJHlXm7w4lbxN3DRjyv58EuzDg=="
         },
         "prompts": {
@@ -23987 +23987 @@
             "version": "1.0.0",
             "resolved": "https://registry.npmjs.org/stream-from-promise/-/stream-from-promise-1.0.0.tgz",
-            "integrity": "sha1-djaH9913fkyJT2QIMz/Gs/yKYbs="
+            "integrity": "sha512-j84KLkudt+gr8KJ21RB02btPLx61uGbrLnewsWz6QKmsz8/c4ZFqXw6mJh5+G4oRN7DgDxdbjPxnpySpg1mUig=="
         },
         "stream-to-string": {
@@ -25375 +25375 @@
             "version": "2.0.2",
             "resolved": "https://registry.npmjs.org/toposort/-/toposort-2.0.2.tgz",
-            "integrity": "sha1-riF2gXXRVZ1IvvNUILL0li8JwzA="
+            "integrity": "sha512-0a5EOkAUp8D4moMi2W8ZF8jcga7BgZd91O/yabJCFY8az+XSzeGyTKs0Aoo897iV1Nj6guFq8orWDS96z91oGg=="
         },
         "totalist": {

trunk/src/wp-includes/rest-api.php

@@ -316 +316 @@
     // Block Types.
     $controller = new WP_REST_Block_Types_Controller();
+    $controller->register_routes();
+
+    // Global Styles revisions.
+    $controller = new WP_REST_Global_Styles_Revisions_Controller();
     $controller->register_routes();
 

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

@@ -421 +421 @@
      *
      * @since 5.9.0
+     * @since 6.3.0 Adds revisions count and rest URL href to version-history.
      *
      * @param integer $id ID.
@@ -433 +434 @@
             ),
         );
+
+        if ( post_type_supports( $this->post_type, 'revisions' ) ) {
+            $revisions       = wp_get_latest_revision_id_and_total_count( $id );
+            $revisions_count = ! is_wp_error( $revisions ) ? $revisions['count'] : 0;
+            $revisions_base  = sprintf( '/%s/%d/revisions', $base, $id );
+            $links['version-history'] = array(
+                'href'  => rest_url( $revisions_base ),
+                'count' => $revisions_count,
+            );
+        }
 
         return $links;

trunk/src/wp-settings.php

@@ -262 +262 @@
 require ABSPATH . WPINC . '/rest-api/endpoints/class-wp-rest-attachments-controller.php';
 require ABSPATH . WPINC . '/rest-api/endpoints/class-wp-rest-global-styles-controller.php';
+require ABSPATH . WPINC . '/rest-api/endpoints/class-wp-rest-global-styles-revisions-controller.php';
 require ABSPATH . WPINC . '/rest-api/endpoints/class-wp-rest-post-types-controller.php';
 require ABSPATH . WPINC . '/rest-api/endpoints/class-wp-rest-post-statuses-controller.php';

trunk/tests/phpunit/tests/rest-api/rest-schema-setup.php

@@ -135 +135 @@
             '/wp/v2/comments/(?P<id>[\\d]+)',
             '/wp/v2/global-styles/(?P<id>[\/\w-]+)',
+            '/wp/v2/global-styles/(?P<parent>[\d]+)/revisions',
             '/wp/v2/global-styles/themes/(?P<stylesheet>[\/\s%\w\.\(\)\[\]\@_\-]+)/variations',
             '/wp/v2/global-styles/themes/(?P<stylesheet>[^\/:<>\*\?"\|]+(?:\/[^\/:<>\*\?"\|]+)?)',

trunk/tests/qunit/fixtures/wp-api-generated.js

@@ -9619 +9619 @@
             ]
         },
+        "/wp/v2/global-styles/(?P<parent>[\\d]+)/revisions": {
+            "namespace": "wp/v2",
+            "methods": [
+                "GET"
+            ],
+            "endpoints": [
+                {
+                    "methods": [
+                        "GET"
+                    ],
+                    "args": {
+                        "parent": {
+                            "description": "The ID for the parent of the revision.",
+                            "type": "integer",
+                            "required": false
+                        },
+                        "context": {
+                            "description": "Scope under which the request is made; determines fields present in response.",
+                            "type": "string",
+                            "enum": [
+                                "view",
+                                "embed",
+                                "edit"
+                            ],
+                            "default": "view",
+                            "required": false
+                        },
+                        "page": {
+                            "description": "Current page of the collection.",
+                            "type": "integer",
+                            "default": 1,
+                            "minimum": 1,
+                            "required": false
+                        },
+                        "per_page": {
+                            "description": "Maximum number of items to be returned in result set.",
+                            "type": "integer",
+                            "minimum": 1,
+                            "maximum": 100,
+                            "required": false
+                        },
+                        "offset": {
+                            "description": "Offset the result set by a specific number of items.",
+                            "type": "integer",
+                            "required": false
+                        }
+                    }
+                }
+            ]
+        },
         "/wp/v2/global-styles/themes/(?P<stylesheet>[\\/\\s%\\w\\.\\(\\)\\[\\]\\@_\\-]+)/variations": {
             "namespace": "wp/v2",