Skip to content

Backport: WP_Theme_JSON_Resolver changes #3901

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

Closed
Mamaduka 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
Prev Previous commit
Next Next commit
Fix spacing 
Co-authored-by: Colin Stewart <[email protected]>
  • Loading branch information
@Mamaduka @costdev
Mamaduka and costdev authored Feb 3, 2023
commit 8c1db45a46619d98de2ceaf71f821ca7b68361da
14 changes: 7 additions & 7 deletions tests/phpunit/tests/theme/wpThemeJsonResolver.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -783,15 +783,15 @@ public function test_get_theme_data_does_not_parse_theme_json_if_not_present() {
*
* @dataProvider data_get_merged_data_returns_origin
*
* @param string $origin What origin to get data from.
* @param bool $core_palette Whether the core palette is present.
* @param string $core_palette_text Message.
* @param string $block_styles Whether the block styles are present.
* @param string $block_styles_text Message.
* @param string $origin What origin to get data from.
* @param bool $core_palette Whether the core palette is present.
* @param string $core_palette_text Message.
* @param string $block_styles Whether the block styles are present.
* @param string $block_styles_text Message.
* @param bool $theme_palette Whether the theme palette is present.
* @param string $theme_palette_text Message.
* @param bool $user_palette Whether the user palette is present.
* @param string $user_palette_text Message.
* @param bool $user_palette Whether the user palette is present.
* @param string $user_palette_text Message.
*/
public function test_get_merged_data_returns_origin( $origin, $core_palette, $core_palette_text, $block_styles, $block_styles_text, $theme_palette, $theme_palette_text, $user_palette, $user_palette_text ) {
// Make sure there is data from the blocks origin.
Expand Down