Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Backport: WP_Theme_JSON_Resolver changes #3901

Closed
wants to merge 16 commits into from
Closed

Backport: WP_Theme_JSON_Resolver changes #3901

Changes from 1 commit
Commits
Show all changes
16 commits
Select commit Hold shift + click to select a range
1ff63af
Add tests
Mamaduka Jan 24, 2023
f28b16c
Migrate WP_Query performance optimizations
Mamaduka Jan 24, 2023
91b86d8
Merged data should consider origin to return early
Mamaduka Jan 24, 2023
f3cc87c
Make child themes inherit parent's style variations
Mamaduka Jan 24, 2023
19b6516
Try assertSameSets
Mamaduka Jan 24, 2023
d85fa86
Update Global Styles controller tests
Mamaduka Jan 24, 2023
0e595e6
Add since mention to docblock
Mamaduka Jan 24, 2023
8c1d84a
Capitalize JSON here
Mamaduka Feb 3, 2023
8c1db45
Fix spacing
Mamaduka Feb 3, 2023
8b5cc6c
Reference ticket
Mamaduka Feb 3, 2023
9952fa3
Fix fomatting
Mamaduka Feb 3, 2023
3b40b25
plural
Mamaduka Feb 3, 2023
8820c62
Use plural
Mamaduka Feb 3, 2023
3a090ec
Reference the ticket
Mamaduka Feb 3, 2023
9870b44
ensure all code runs before the assertions
Mamaduka Feb 3, 2023
1cc848e
Spacing CS fix
Mamaduka Feb 3, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion tests/phpunit/tests/theme/wpThemeJsonResolver.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -970,7 +970,7 @@ public function test_get_style_variations_returns_all_variations() {
wp_recursive_ksort( $actual_settings );
wp_recursive_ksort( $expected_settings );

$this->assertSame(
$this->assertSameSets(
Mamaduka marked this conversation as resolved.
Show resolved Hide resolved
$expected_settings,
$actual_settings
);
Expand Down