Changeset 43766
- Timestamp:
- 10/19/2018 11:13:57 AM (6 years ago)
- Location:
- branches/5.0
- Files:
-
- 4 edited
Legend:
- Unmodified
- Added
- Removed
-
branches/5.0
-
branches/5.0/src/wp-includes/rest-api.php
r43763 r43766 326 326 } 327 327 328 $path = '/' . ltrim( $path, '/' ); 329 328 330 if ( is_multisite() && get_blog_option( $blog_id, 'permalink_structure' ) || get_option( 'permalink_structure' ) ) { 329 331 global $wp_rewrite; … … 335 337 } 336 338 337 $url .= '/' . ltrim( $path, '/' );339 $url .= $path; 338 340 } else { 339 341 $url = trailingslashit( get_home_url( $blog_id, '', $scheme ) ); … … 343 345 $url .= 'index.php'; 344 346 } 345 346 $path = '/' . ltrim( $path, '/' );347 347 348 348 $url = add_query_arg( 'rest_route', $path, $url ); -
branches/5.0/tests/phpunit/includes/testcase-rest-controller.php
r43720 r43766 41 41 42 42 public function filter_rest_url_for_leading_slash( $url, $path ) { 43 if ( is_multisite() || get_option( 'permalink_structure' )) {43 if ( is_multisite() ) { 44 44 return $url; 45 45 } -
branches/5.0/tests/phpunit/tests/rest-api.php
r43763 r43766 487 487 set_current_screen( 'front' ); 488 488 489 } 490 491 /** 492 * @ticket 42452 493 */ 494 public function test_always_prepend_path_with_slash_in_rest_url_filter() { 495 $filter = new MockAction(); 496 add_filter( 'rest_url', array( $filter, 'filter' ), 10, 2 ); 497 498 // Passing no path should return a slash. 499 get_rest_url(); 500 $args = $filter->get_args(); 501 $this->assertEquals( '/', $args[0][1] ); 502 $filter->reset(); 503 504 // Paths without a prepended slash should have one added. 505 get_rest_url( null, 'wp/media/' ); 506 $args = $filter->get_args(); 507 $this->assertEquals( '/wp/media/', $args[0][1] ); 508 $filter->reset(); 509 510 // Do not modify paths with a prepended slash. 511 get_rest_url( null, '/wp/media/' ); 512 $args = $filter->get_args(); 513 $this->assertEquals( '/wp/media/', $args[0][1] ); 514 515 unset( $filter ); 489 516 } 490 517
Note: See TracChangeset
for help on using the changeset viewer.