WordPress.org

Make WordPress Core


Ignore:
Timestamp:
12/16/2016 05:45:50 AM (5 years ago)
Author:
dd32
Message:

REST API: Do not include the password argument when getting media items

Currently, attachment is the only post type exposed via the REST API that
does not support password protection, but it's possible for other post types to
remove password support.

Props jnylen0.
Merges [39595] to the 4.7 branch.
Fixes #38977.

Location:
branches/4.7
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • branches/4.7

  • branches/4.7/tests/phpunit/tests/rest-api/rest-attachments-controller.php

    r39182 r39610  
    171171        }
    172172        $this->assertEqualSets( $media_types, $data['endpoints'][0]['args']['media_type']['enum'] );
     173    }
     174
     175    public function test_registered_get_item_params() {
     176        $id1 = $this->factory->attachment->create_object( $this->test_file, 0, array(
     177            'post_mime_type' => 'image/jpeg',
     178            'post_excerpt'   => 'A sample caption',
     179        ) );
     180        $request = new WP_REST_Request( 'OPTIONS', sprintf( '/wp/v2/media/%d', $id1 ) );
     181        $response = $this->server->dispatch( $request );
     182        $data = $response->get_data();
     183        $keys = array_keys( $data['endpoints'][0]['args'] );
     184        sort( $keys );
     185        $this->assertEquals( array( 'context' ), $keys );
    173186    }
    174187
Note: See TracChangeset for help on using the changeset viewer.