From 802139d185a16900701d1d6d79663b25d0aaeef7 Mon Sep 17 00:00:00 2001 From: Weston Ruter Date: Thu, 14 Jun 2018 11:39:35 +0000 Subject: [PATCH] Privacy: Only link to menus panel in Customizer if selected privacy page can be accessed there. Props dlh. Fixes #44117. git-svn-id: https://develop.svn.wordpress.org/trunk@43343 602fd350-edb4-49c9-b593-d223f7449a82 --- src/wp-admin/privacy.php | 29 ++++++++++++++++++++++++----- 1 file changed, 24 insertions(+), 5 deletions(-) diff --git a/src/wp-admin/privacy.php b/src/wp-admin/privacy.php index 991ad72eff..9c360dd18f 100644 --- a/src/wp-admin/privacy.php +++ b/src/wp-admin/privacy.php @@ -22,14 +22,33 @@ if ( ! empty( $action ) ) { $privacy_policy_page_id = isset( $_POST['page_for_privacy_policy'] ) ? (int) $_POST['page_for_privacy_policy'] : 0; update_option( 'wp_page_for_privacy_policy', $privacy_policy_page_id ); + $privacy_page_updated_message = __( 'Privacy policy page updated successfully.' ); + + if ( $privacy_policy_page_id ) { + /* + * Don't always link to the menu customizer: + * + * - Unpublished pages can't be selected by default. + * - `WP_Customize_Nav_Menus::__construct()` checks the user's capabilities. + * - Themes might not "officially" support menus. + */ + if ( + 'publish' === get_post_status( $privacy_policy_page_id ) + && current_user_can( 'edit_theme_options' ) + && current_theme_supports( 'menus' ) + ) { + $privacy_page_updated_message = sprintf( + /* translators: %s: URL to Customizer -> Menus */ + __( 'Privacy policy page updated successfully. Remember to update your menus!' ), + esc_url( add_query_arg( 'autofocus[panel]', 'nav_menus', admin_url( 'customize.php' ) ) ) + ); + } + } + add_settings_error( 'page_for_privacy_policy', 'page_for_privacy_policy', - sprintf( - /* translators: %s: URL to Customizer -> Menus */ - __( 'Privacy policy page updated successfully. Remember to update your menus!' ), - 'customize.php?autofocus[panel]=nav_menus' - ), + $privacy_page_updated_message, 'updated' ); } elseif ( 'create-privacy-page' === $action ) {