Ticket #43954: 43954.2.diff

src/wp-admin/css/forms.css

@@ -1089 +1089 @@
 }
 
 .tools-privacy-edit {
-        margin: 2.3em 0;
+        margin: 1.5em 0;
 }
 
 .tools-privacy-policy-page span {

src/wp-admin/includes/admin-filters.php

@@ -138 +138 @@
 add_action( 'wp_privacy_personal_data_export_file', 'wp_privacy_generate_personal_data_export_file', 10 );
 
 // Privacy policy text changes check.
-add_action( 'admin_init', array( 'WP_Privacy_Policy_Content', 'text_change_check' ), 20 );
+add_action( 'admin_init', array( 'WP_Privacy_Policy_Content', 'text_change_check' ), 100 );
 
 // Show a "postbox" with the text suggestions for a privacy policy.
 add_action( 'edit_form_after_title', array( 'WP_Privacy_Policy_Content', 'notice' ) );
@@ -146 +146 @@
 // Add the suggested policy text from WordPress.
 add_action( 'admin_init', array( 'WP_Privacy_Policy_Content', 'add_suggested_content' ), 1 );
 
-// Stop checking for text changes after the policy page is updated.
+// Update the cached policy info when the policy page is updated.
 add_action( 'post_updated', array( 'WP_Privacy_Policy_Content', '_policy_page_updated' ) );

src/wp-admin/includes/misc.php

@@ -1320 +1320 @@
 
                 // The site doesn't have a privacy policy.
                 if ( empty( $policy_page_id ) ) {
-                        return;
+                        return false;
                 }
 
                 if ( ! current_user_can( 'edit_post', $policy_page_id ) ) {
-                        return;
+                        return false;
                 }
 
-                // Also run when the option doesn't exist yet.
-                if ( get_option( '_wp_privacy_text_change_check' ) === 'no-check' ) {
-                        return;
+                $old = (array) get_post_meta( $policy_page_id, '_wp_suggested_privacy_policy_content' );
+
+                // The users have never seen the Privacy Policy Guide screen (usually happens after fresh install or update).
+                // No point to show warnings about updated content there.
+                if ( empty( $old ) ) {
+                        return false;
                 }
 
-                $old = (array) get_post_meta( $policy_page_id, '_wp_suggested_privacy_policy_content' );
+                $cached = get_option( '_wp_suggested_policy_text_has_changed' );
+
+                // Too early to get the privacy info, use the cache.
+                // This check also runs on the 'admin_init' action and will update the cache if needed.
+                if ( ! did_action( 'admin_init' ) ) {
+                        return $cached === 'changed';
+                }
+
                 $new = self::$policy_content;
 
                 // Remove the extra values added to the meta.
                 foreach ( $old as $key => $data ) {
-                        $old[ $key ] = array(
-                                'plugin_name' => $data['plugin_name'],
-                                'policy_text' => $data['policy_text'],
-                        );
+                        if ( ! empty( $data['removed'] ) ) {
+                                unset( $old[ $key ] );
+                        } else {
+                                $old[ $key ] = array(
+                                        'plugin_name' => $data['plugin_name'],
+                                        'policy_text' => $data['policy_text'],
+                                );
+                        }
                 }
 
                 // The == operator (equal, not identical) was used intentionally.
@@ -1347 +1361 @@
                 // See http://php.net/manual/en/language.operators.array.php
                 if ( $new != $old ) {
                         // A plugin was activated or deactivated, or some policy text has changed.
-                        // Show a notice on all screens in wp-admin.
+                        // Show a notice on the Edit Page screen when editing the policy page.
                         add_action( 'admin_notices', array( 'WP_Privacy_Policy_Content', 'policy_text_changed_notice' ) );
+                        $state = 'changed';
                 } else {
-                        // Stop checking.
-                        update_option( '_wp_privacy_text_change_check', 'no-check' );
+                        $state = 'not-changed';
                 }
+
+                if ( $cached !== $state ) {
+                        update_option( '_wp_suggested_policy_text_has_changed', $state );
+                }
+
+                return $state === 'changed';
         }
 
         /**
-         * Output an admin notice when some privacy info has changed.
+         * Output a warning when some privacy info has changed.
          *
          * @since 4.9.6
          */
         public static function policy_text_changed_notice() {
                 global $post;
+
+                $screen = get_current_screen()->id;
                 $policy_page_id = (int) get_option( 'wp_page_for_privacy_policy' );
 
-                ?>
-                <div class="policy-text-updated notice notice-warning is-dismissible">
-                        <p><?php
-
-                                _e( 'The suggested privacy policy text has changed.' );
-
-                                if ( empty( $post ) || $post->ID != $policy_page_id ) {
-                                        ?>
-                                        <a href="<?php echo get_edit_post_link( $policy_page_id ); ?>"><?php _e( 'Edit the privacy policy.' ); ?></a>
-                                        <?php
-                                }
-
-                        ?></p>
-                </div>
-                <?php
+                if ( $screen === 'privacy' || ( $screen === 'page' && ! empty( $post ) && (int) $post->ID === $policy_page_id ) ) {
+                        ?>
+                        <div class="policy-text-updated notice notice-warning is-dismissible">
+                                <p><?php _e( 'The suggested privacy policy text has changed. Please update your privacy policy.' ); ?></p>
+                        </div>
+                        <?php
+                }
         }
 
         /**
-         * Stop checking for changed privacy info when the policy page is updated.
+         * Update the cached policy info when the policy page is updated.
          *
          * @since 4.9.6
          * @access private
@@ -1394 +1408 @@
                         return;
                 }
 
-                // The policy page was updated.
-                // Stop checking for text changes.
-                update_option( '_wp_privacy_text_change_check', 'no-check' );
-
                 // Remove updated|removed status.
                 $old = (array) get_post_meta( $policy_page_id, '_wp_suggested_privacy_policy_content' );
                 $done = array();
@@ -1496 +1506 @@
                         foreach ( $new as $new_data ) {
                                 if ( ! empty( $new_data['plugin_name'] ) && ! empty( $new_data['policy_text'] ) ) {
                                         $new_data['added'] = $time;
-                                        array_unshift( $checked, $new_data );
+                                        $checked[] = $new_data;
                                 }
                         }
                         $update_cache = true;
@@ -1511 +1521 @@
                                                 'policy_text' => $old_data['policy_text'],
                                                 'removed'     => $time,
                                         );
-                                        array_unshift( $checked, $data );
+
+                                        $checked[] = $data;
                                 }
                         }
                         $update_cache = true;
@@ -1525 +1536 @@
                         }
                 }
 
-                // Stop checking for changes after the page has been loaded.
-                if ( get_option( '_wp_privacy_text_change_check' ) !== 'no-check' ) {
-                        update_option( '_wp_privacy_text_change_check', 'no-check' );
-                }
-
                 return $checked;
         }
 

src/wp-admin/menu.php

@@ -263 +263 @@
         $submenu['tools.php'][50] = array( __( 'Network Setup' ), 'setup_network', 'network.php' );
 }
 
-$menu[80]                               = array( __( 'Settings' ), 'manage_options', 'options-general.php', '', 'menu-top menu-icon-settings', 'menu-settings', 'dashicons-admin-settings' );
+$change_notice = '';
+if ( current_user_can( 'manage_privacy_options' ) && WP_Privacy_Policy_Content::text_change_check() ) {
+        $change_notice = ' <span class="update-plugins 1"><span class="plugin-count">' . number_format_i18n( 1 ) . '</span></span>';
+}
+
+$menu[80]                               = array( sprintf( __( 'Settings %s' ), $change_notice ), 'manage_options', 'options-general.php', '', 'menu-top menu-icon-settings', 'menu-settings', 'dashicons-admin-settings' );
         $submenu['options-general.php'][10] = array( _x( 'General', 'settings screen' ), 'manage_options', 'options-general.php' );
         $submenu['options-general.php'][15] = array( __( 'Writing' ), 'manage_options', 'options-writing.php' );
         $submenu['options-general.php'][20] = array( __( 'Reading' ), 'manage_options', 'options-reading.php' );
@@ -270 +275 @@
         $submenu['options-general.php'][25] = array( __( 'Discussion' ), 'manage_options', 'options-discussion.php' );
         $submenu['options-general.php'][30] = array( __( 'Media' ), 'manage_options', 'options-media.php' );
         $submenu['options-general.php'][40] = array( __( 'Permalinks' ), 'manage_options', 'options-permalink.php' );
-        $submenu['options-general.php'][45] = array( __( 'Privacy' ), 'manage_privacy_options', 'privacy.php' );
+        $submenu['options-general.php'][45] = array( sprintf( __( 'Privacy %s' ), $change_notice ), 'manage_privacy_options', 'privacy.php' );
 
 $_wp_last_utility_menu = 80; // The index of the last top-level menu in the utility menu group
 

src/wp-includes/default-filters.php

@@ -562 +562 @@
 // Capabilities
 add_filter( 'user_has_cap', 'wp_maybe_grant_install_languages_cap', 1 );
 
-// Trigger the check for policy text changes after active plugins change.
-add_action( 'update_site_option_active_sitewide_plugins', '_wp_privacy_active_plugins_change' );
-add_action( 'update_option_active_plugins', '_wp_privacy_active_plugins_change' );
-
 unset( $filter, $action );

src/wp-includes/functions.php

@@ -6249 +6249 @@
 }
 
 /**
- * Trigger the check for policy text changes.
- *
- * @since 4.9.6
- * @access private
- */
-function _wp_privacy_active_plugins_change() {
-        update_option( '_wp_privacy_text_change_check', 'check' );
-}
-
-/**
  * Schedule a `WP_Cron` job to delete expired export files.
  *
  * @since 4.9.6