Ticket #25603: 25603.20.diff

wp-admin/about.php

@@ -52 +52 @@
                         <p><?php _e( 'You&#8217;ll still need to click &#8220;Update Now&#8221; once WordPress 3.8 is released, but we&#8217;ve never had more confidence in that beautiful blue button.' ); ?></p>
                 </div>
                 <?php
-                        $can_auto_update = wp_http_supports( 'ssl' );
+                if ( current_user_can( 'update_core' ) ) {
+                        $future_minor_update = (object) array(
+                                'current'       => $wp_version . '.1.next.minor',
+                                'version'       => $wp_version . '.1.next.minor',
+                                'php_version'   => $required_php_version,
+                                'mysql_version' => $required_mysql_version,
+                        );
+                        require_once ABSPATH . 'wp-admin/includes/class-wp-upgrader.php';
+                        $updater = new WP_Automatic_Updater;
+                        $can_auto_update = wp_http_supports( 'ssl' ) && $updater->should_update( 'core', $future_minor_update, ABSPATH );
+
                         if ( $can_auto_update ) {
-                                require_once ABSPATH . 'wp-admin/includes/class-wp-upgrader.php';
-                                $upgrader = new WP_Automatic_Updater;
-                                $future_minor_update = (object) array(
-                                        'current'       => $wp_version . '.1.next.minor',
-                                        'version'       => $wp_version . '.1.next.minor',
-                                        'php_version'   => $required_php_version,
-                                        'mysql_version' => $required_mysql_version,
-                                );
-                                $can_auto_update = $upgrader->should_update( 'core', $future_minor_update, ABSPATH );
+                                echo '<p class="about-auto-update cool">' . __( 'This site <strong>is</strong> able to apply these updates automatically. Cool!' ). '</p>';
+
+                        // If the updater is disabled entirely, don't show them anything.
+                        } elseif ( ! $updater->is_disabled() ) {
+                                echo '<p class="about-auto-update">';
+                                // If notify_core_update is filtered to false, they won't get emails.
+                                // Assumption: If the user can update core, they can see what the admin email is.
+                                if ( apply_filters( 'notify_core_update', true, $future_minor_update ) ) {
+                                        printf( __( 'This site <strong>is not</strong> able to apply these updates automatically. But we&#8217;ll email %s when there is a new security release.' ), esc_html( get_site_option( 'admin_email' ) ) );
+                                } else {
+                                        _e( 'This site <strong>is not</strong> able to apply these updates automatically.' );
+                                }
+                                echo '</p>';
                         }
-                        if ( $can_auto_update ) : ?>
-                                <p class="about-auto-update cool"><?php _e( 'This site <strong>is</strong> able to apply these updates automatically. Cool!' ); ?></p>
-                        <?php else : ?>
-                                <p class="about-auto-update"><?php printf( __( 'This site <strong>is not</strong> able to apply these updates automatically. But we&#8217;ll email %s when there is a new security release.' ), esc_html( get_site_option( 'admin_email' ) ) ); ?></p>
-                <?php endif; ?>
+                }
+                ?>
         </div>
 </div>
 

wp-admin/includes/class-wp-upgrader.php

@@ -1810 +1810 @@
                 if ( $notified && $notified['email'] == get_site_option( 'admin_email' ) && $notified['version'] == $item->current )
                         return false;
 
+                if ( ! apply_filters( 'notify_core_update', true, $item ) )
+                        return false;
+
                 $this->send_email( 'manual', $item );
                 return true;
         }
@@ -2127 +2130 @@
                  * @since 3.7.0
                  *
                  * @param bool   $send        Whether to send the email. Default true.
-                 * @param string $type        The type of email to send. Can be one of 'success', 'fail', 'manual', 'critical'.
+                 * @param string $type        The type of email to send. Can be one of 'success', 'fail', 'critical'.
                  * @param object $core_update The update offer that was attempted.
                  * @param mixed  $result      The result for the core update. Can be WP_Error.
                  */
-                if ( ! apply_filters( 'auto_core_update_send_email', true, $type, $core_update, $result ) )
+                if ( 'manual' !== $type && ! apply_filters( 'auto_core_update_send_email', true, $type, $core_update, $result ) )
                         return;
 
                 switch ( $type ) {