vhad/wordpress-develop
1
0
Fork 0
mirror of https://github.com/gosticks/wordpress-develop.git synced 2025-10-16 12:05:38 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

I18n: Standardize the script paths for blocks

Browse Source 
When providing file paths to scripts (editorScript, script or viewScript), when there is a trailing ./ included then there was a different md5 generated for the file that didn't match the one used with the file generated in the translations folder.

Props Rahe.
See #54797.



git-svn-id: https://develop.svn.wordpress.org/trunk@52699 602fd350-edb4-49c9-b593-d223f7449a82
This commit is contained in:
Greg Ziółkowski 2022-02-11 12:12:56 +00:00
parent 968b3fc5a4
commit af07f6af4b
2 changed files with 15 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
src/wp-includes/blocks.php
View File

@ -20,10 +20,14 @@ function remove_block_asset_path_prefix( $asset_handle_or_path ) {
if ( 0 !== strpos( $asset_handle_or_path, $path_prefix ) ) {
return $asset_handle_or_path;
}
return substr(
$path = substr(
$asset_handle_or_path,
strlen( $path_prefix )
);
if ( strpos( $path, './' ) === 0 ) {
$path = substr( $path, 2 );
}
return $path;
}
/**

11
tests/phpunit/tests/blocks/register.php
View File

@ -122,9 +122,18 @@ class Tests_Blocks_Register extends WP_UnitTestCase {
* @ticket 50263
*/
public function test_removes_block_asset_path_prefix() {
$result = remove_block_asset_path_prefix( 'file:block.js' );
$this->assertSame( 'block.js', $result );
}
/**
* @ticket 54797
*/
public function test_removes_block_asset_path_prefix_and_current_directory() {
$result = remove_block_asset_path_prefix( 'file:./block.js' );
$this->assertSame( './block.js', $result );
$this->assertSame( 'block.js', $result );
}
/**