Changeset 51267

Timestamp:

06/30/2021 01:45:10 AM (4 years ago)

Author:

desrosj

Message:

Script Loader: Fix PHP notice caused by the viewScript for the core/file block.

This fixes a PHP notice caused by the viewScript for the core/file block having .min.js instead of just .js.

register_block_script_handle() was incorrectly looking for view.min.asset.php, which does not exist, and caused the _doing_it_wrong() notice.

This adds a check for .min.js in the viewScript field of block.json and corrects it to .js in order to match the expected pattern of view.asset.php until a more permanent fix can be created.

Follow up to [51259].

Props ipstenu, pbiron, peterwilsoncc.
See #53397.

File:

• trunk/src/wp-includes/blocks.php (modified) (1 diff)

trunk/src/wp-includes/blocks.php

@@ -83 +83 @@
     }
 
-    $script_handle     = generate_block_asset_handle( $metadata['name'], $field_name );
+    $script_handle = generate_block_asset_handle( $metadata['name'], $field_name );
+
+    if ( 'viewScript' === $field_name ) {
+        $script_path = str_replace( '.min.js', '.js', $script_path );
+    }
+
     $script_asset_path = realpath(
         dirname( $metadata['file'] ) . '/' .