WordPress.org
Get WordPress

Make WordPress Core

Opened 16 months ago

Last modified 16 months ago

#60231 new defect (bug)

Replace constant ABSPATH . WPINC with BLOCKS_PATH in the require-dynamic-blocks.php

Reported by: vishalkakadiya's profile vishalkakadiya Owned by:
Milestone: Awaiting Review Priority: normal
Severity: normal Version:
Component: Build/Test Tools Keywords: has-patch
Focuses: Cc:

Description

Basically, we are already defining the BLOCKS_PATH constant in the wp-includes/blocks/index.php file which is loading the require-dynamic-blocks.php

So it is good practice to replace combination ABSPATH . WPINC constants with BLOCKS_PATH single constant in the require-dynamic-blocks.php file generated by tools/release/sync-stable-blocks.js file.

Attachments (1)

60231.diff​ (522 bytes) - added by vishalkakadiya 16 months ago.
Patch file

Download all attachments as: .zip

Change History (5)

@vishalkakadiya
16 months ago

Patch file

#1 @vishalkakadiya
16 months ago

I have added a patch file to this issue. Thanks!

​@mukesh27 commented on ​PR #5857:
16 months ago #3

Thanks @vishalkakadiya for the PR! Left one comment.

Could you please share steps how anyone test it?

#4 @mukesh27
16 months ago

  • Component changed from General to Build/Test Tools
  • Version trunk deleted
Note: See TracTickets for help on using tickets.