Make WordPress Core

Changeset 52699


Ignore:
Timestamp:
02/11/2022 12:12:56 PM (2 years ago)
Author:
gziolo
Message:

I18n: Standardize the script paths for blocks

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.

Location:
trunk
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/wp-includes/blocks.php

    r52597 r52699  
    2121        return $asset_handle_or_path;
    2222    }
    23     return substr(
     23    $path = substr(
    2424        $asset_handle_or_path,
    2525        strlen( $path_prefix )
    2626    );
     27    if ( strpos( $path, './' ) === 0 ) {
     28        $path = substr( $path, 2 );
     29    }
     30    return $path;
    2731}
    2832
  • trunk/tests/phpunit/tests/blocks/register.php

    r52388 r52699  
    123123     */
    124124    public function test_removes_block_asset_path_prefix() {
     125        $result = remove_block_asset_path_prefix( 'file:block.js' );
     126
     127        $this->assertSame( 'block.js', $result );
     128    }
     129
     130    /**
     131     * @ticket 54797
     132     */
     133    public function test_removes_block_asset_path_prefix_and_current_directory() {
    125134        $result = remove_block_asset_path_prefix( 'file:./block.js' );
    126135
    127         $this->assertSame( './block.js', $result );
     136        $this->assertSame( 'block.js', $result );
    128137    }
    129138
Note: See TracChangeset for help on using the changeset viewer.