Opened 6 months ago
Closed 5 weeks ago
#61485 closed enhancement (fixed)
Refactor Etag-based cache busting for styles and scripts solution and add tests
Reported by: | vrajadas | Owned by: | peterwilsoncc |
---|---|---|---|
Milestone: | 6.7 | Priority: | normal |
Severity: | normal | Version: | |
Component: | Script Loader | Keywords: | has-patch has-unit-tests commit dev-reviewed |
Focuses: | Cc: |
Description
Following issue #58433, I wanted to add tests to the solution and then realised they is code duplication. I move the logic to method in the class both wp_script and wp_style extend.
I also added tests.
Test instructions are the same as in #58433, since this issue doesn't change functionality.
Github PR is https://github.com/WordPress/wordpress-develop/pull/6820
Change History (12)
This ticket was mentioned in PR #6820 on WordPress/wordpress-develop by @vrajadas.
6 months ago
#1
- Keywords has-patch has-unit-tests added
@vrajadas commented on PR #6820:
6 months ago
#2
Hey @peterwilsoncc I created a new trac ticket and updated the PR description with it's new number.
@martin.krcho commented on PR #6820:
5 months ago
#3
@mukeshpanchal27 would you please re-review this task?
#4
@
5 months ago
- Milestone changed from Awaiting Review to 6.7
- Version 6.6 deleted
The PR looks good to me and approved, happy to get more folks eye.
Move to 6.7 milestone.
#5
@
4 months ago
- Owner set to peterwilsoncc
- Resolution set to fixed
- Status changed from new to closed
In 58935:
@peterwilsoncc commented on PR #6820:
4 months ago
#6
#7
@
5 weeks ago
Bug identified that was introduced here: https://github.com/WordPress/wordpress-develop/pull/7715
cc @peterwilsoncc
Added tests to change make in PR for 58433
Moved the logic to a separate method under wp-dependencies class and added unit tests.
Trac ticket: https://core.trac.wordpress.org/ticket/61485