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.

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

Add _wp_array_set and _wp_to_kebab_case to 5.8 compat #36399

Merged
merged 1 commit into from
Nov 11, 2021

Conversation

walbo
Copy link
Member

@walbo walbo commented Nov 11, 2021

Description

Replace gutenberg_experimental_set with _wp_array_set and gutenberg_experimental_to_kebab_case with _wp_to_kebab_case.

These functions was added in WordPress 5.8.

This fixes a issue with the page list block where gutenberg_experimental_to_kebab_case is used. WordPress doesn't have this function and the block will fail when used in 5.9.

Types of changes

  • Added compat functions in wordpress-5-8
  • Replaced all gutenberg_experimental_set with _wp_array_set and gutenberg_experimental_to_kebab_case with _wp_to_kebab_case

Replace `gutenberg_experimental_set` with `_wp_array_set` and `gutenberg_experimental_to_kebab_case` with `_wp_to_kebab_case`.

These functions was added in WordPress 5.8.

This fixes a issue with the page list block where `gutenberg_experimental_to_kebab_case` is used. WordPress doesn't have this function and the block will fail when used.
Copy link
Member

@Mamaduka Mamaduka left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, @walbo.

Changes look good to me.

@Mamaduka Mamaduka merged commit 1d08eaf into trunk Nov 11, 2021
@Mamaduka Mamaduka deleted the fix/utils-compat-array-set-cabel-case branch November 11, 2021 14:20
@github-actions github-actions bot added this to the Gutenberg 12.0 milestone Nov 11, 2021
noisysocks pushed a commit that referenced this pull request Nov 23, 2021
Replace `gutenberg_experimental_set` with `_wp_array_set` and `gutenberg_experimental_to_kebab_case` with `_wp_to_kebab_case`.

These functions was added in WordPress 5.8.

This fixes a issue with the page list block where `gutenberg_experimental_to_kebab_case` is used. WordPress doesn't have this function and the block will fail when used.
@bph bph added the Needs Dev Note Requires a developer note for a major WordPress release cycle label Mar 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Needs Dev Note Requires a developer note for a major WordPress release cycle
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants