Make WordPress Core

Opened 5 years ago

Last modified 3 months ago

#46264 new defect (bug)

Page attributes 'ORDER' does not saving data for post-type 'post'.

Reported by: tanvirul's profile tanvirul Owned by:
Milestone: Awaiting Review Priority: normal
Severity: normal Version: 5.0
Component: Editor Keywords: has-patch
Focuses: rest-api Cc:


I used add_post_type_support() function to support 'page-attributes' for Menu Order at post-type 'post'.

function prefix_add_posts_order() {
        add_post_type_support( 'post', 'page-attributes' );
add_action( 'admin_init', 'prefix_add_posts_order' );

When I save the post after putting an integer value in the Order field then no value is saving on 'wp_posts' table 'menu_order' column and it does not also update on the editor page.

But when I switch to TinyMCE editor it's working fine.

Attachments (2)

46264.diff (1.0 KB) - added by felipeelia 5 years ago.
46264.2.diff (2.0 KB) - added by websupporter 5 years ago.
unit test added

Download all attachments as: .zip

Change History (10)

5 years ago

#1 @felipeelia
5 years ago

  • Keywords has-patch needs-unit-tests added; needs-patch removed

Hi @tanvirul and welcome to WordPress trac :)

You shouldn't hook your function to admin_init, but to init. Anyway, I was able to reproduce the problem you reported.

That seems to happen because, for posts, pages, and attachments, the JSON Schema generated by WP_REST_Posts_Controller::get_item_schema() is restricted by the $fixed_schemas variable. That said, the menu_order field is never exposed to the Rest API.

I've uploaded a patch that uses $fixed_schemas for default attributes, but still checks if those post types support other attributes. Perhaps we should remove $fixed_schemas at all, and always check this dynamically, but I guess it was there for a reason, so let's wait for somebody else input here.

Also, for the Tests suite, I think we should probably move/copy test_update_page_menu_order_* functions from WP_Test_REST_Pages_Controller to use them for all post types (custom or not). But, again, someone more experienced may have better input for this.

#2 @lordspace
5 years ago

menu_order doesn't seem to retain its value in Gutenberg.
WP v5.2.2

5 years ago

unit test added

#3 @websupporter
5 years ago

  • Focuses rest-api added
  • Keywords needs-unit-tests removed

I just stumbled upon the same problem. Thank you for reporting @tanvirul!

I think the general idea of the fixed schema is the following: If you would generate those keys (menu_order, excerpt, etc) only dynamically, a lot of clients might break, as they might just assume every post object has an excerpt. The fixed schema ensures some stability for the core features the content types have.

That said, it is still possible to extend the basic data concept (register_rest_field()), so the fixed schema does not mean its intended to serve only those data values.

So while remove_post_type_support() should not remove properties from the JSON output, I think add_post_type_support() should add properties to it.

I tried @felipeelia fix and it works as intended. Thank you for providing it. My last patch added a unit test for the intended behavior.

In the meantime, this workaround helped me:

    function() {
                'get_callback' => function($object) {
                    if (! isset($object['menu_order'])) {
                        return 0;
                    return (int) $object['menu_order'];
                'schema' => [
                    'type' => 'integer',
Last edited 5 years ago by websupporter (previous) (diff)

This ticket was mentioned in Slack in #core-restapi by websupporter. View the logs.

5 years ago

#5 @Mamaduka
14 months ago

Here's a similar report from the Gutenberg repo:

#6 follow-up: @rudwolf
13 months ago

I just stumbled on this error today and the rest_api trick worked, but do we have any update on this? I see it was opened 4 years ago...

#7 in reply to: ↑ 6 @floo214
13 months ago


#8 @Krstarica
3 months ago

Experiencing the same in WordPress 6.4.3.

Note: See TracTickets for help on using tickets.