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

Always allow accessing edit.php?post_type=wp_navigation page #38170

Merged
merged 1 commit into from
Jan 25, 2022

Conversation

noisysocks
Copy link
Member

@noisysocks noisysocks commented Jan 24, 2022

Fixes #38118.

The Navigation block is supported in both block themes and classic themes, so it is incorrect to only allow access to edit.php?post_type=wp_navigation when the current theme is a block theme.

This fixes a bug where clicking on 'Manage menus' in the toolbar of a Navigation block takes you an error page when running a classic theme.

It's the same fix as what's in https://core.trac.wordpress.org/ticket/54889 but in Gutenberg. We need to make the change in both codebases so that older versions of WP running the Gutenberg plugin don't have the issue.

The Navigation block is supported in both block themes and classic
themes, so it is incorrect to only allow access to
edit.php?post_type=wp_navigation when the current theme is a block
theme.

This fixes a bug where clicking on 'Manage menus' in the toolbar of a
Navigation block takes you an error page when running a classic theme.
@noisysocks noisysocks added [Type] Bug An existing feature does not function as intended [Block] Navigation Affects the Navigation Block labels Jan 24, 2022
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.

LGTM 👍

@talldan talldan merged commit 40971df into trunk Jan 25, 2022
@talldan talldan deleted the fix/manage-menus-link-when-using-classic-theme branch January 25, 2022 02:34
@github-actions github-actions bot added this to the Gutenberg 12.5 milestone Jan 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Block] Navigation Affects the Navigation Block [Type] Bug An existing feature does not function as intended
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Can't access navigation menu (post type) page in backend
3 participants