Upgrade/Install: Check if plugin or theme update results are available before applying the notification filters.
This avoids a PHP notice for an undefined index of either `plugin` or `theme` in the `auto_{plugin|theme}_update_send_email` filter hook. Props afragen, audrasjb. Merges [49166] to the 5.5 branch. Fixes #51400. git-svn-id: https://develop.svn.wordpress.org/branches/5.5@49167 602fd350-edb4-49c9-b593-d223f7449a82
This commit is contained in:
parent
8a8346a4ca
commit
042bfaed29
@ -887,44 +887,48 @@ class WP_Automatic_Updater {
|
||||
$successful_updates = array();
|
||||
$failed_updates = array();
|
||||
|
||||
/**
|
||||
* Filters whether to send an email following an automatic background plugin update.
|
||||
*
|
||||
* @since 5.5.0
|
||||
* @since 5.5.1 Added the $update_results parameter.
|
||||
*
|
||||
* @param bool $enabled True if plugins notifications are enabled, false otherwise.
|
||||
* @param array $update_results The results of plugins update tasks.
|
||||
*/
|
||||
$notifications_enabled = apply_filters( 'auto_plugin_update_send_email', true, $update_results['plugin'] );
|
||||
if ( ! empty( $update_results['plugin'] ) ) {
|
||||
/**
|
||||
* Filters whether to send an email following an automatic background plugin update.
|
||||
*
|
||||
* @since 5.5.0
|
||||
* @since 5.5.1 Added the `$update_results` parameter.
|
||||
*
|
||||
* @param bool $enabled True if plugin update notifications are enabled, false otherwise.
|
||||
* @param array $update_results The results of plugins update tasks.
|
||||
*/
|
||||
$notifications_enabled = apply_filters( 'auto_plugin_update_send_email', true, $update_results['plugin'] );
|
||||
|
||||
if ( ! empty( $update_results['plugin'] ) && $notifications_enabled ) {
|
||||
foreach ( $update_results['plugin'] as $update_result ) {
|
||||
if ( true === $update_result->result ) {
|
||||
$successful_updates['plugin'][] = $update_result;
|
||||
} else {
|
||||
$failed_updates['plugin'][] = $update_result;
|
||||
if ( $notifications_enabled ) {
|
||||
foreach ( $update_results['plugin'] as $update_result ) {
|
||||
if ( true === $update_result->result ) {
|
||||
$successful_updates['plugin'][] = $update_result;
|
||||
} else {
|
||||
$failed_updates['plugin'][] = $update_result;
|
||||
}
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
/**
|
||||
* Filters whether to send an email following an automatic background theme update.
|
||||
*
|
||||
* @since 5.5.0
|
||||
* @since 5.5.1 Added the $update_results parameter.
|
||||
*
|
||||
* @param bool $enabled True if notifications are enabled, false otherwise.
|
||||
* @param array $update_results The results of theme update tasks.
|
||||
*/
|
||||
$notifications_enabled = apply_filters( 'auto_theme_update_send_email', true, $update_results['theme'] );
|
||||
if ( ! empty( $update_results['theme'] ) ) {
|
||||
/**
|
||||
* Filters whether to send an email following an automatic background theme update.
|
||||
*
|
||||
* @since 5.5.0
|
||||
* @since 5.5.1 Added the `$update_results` parameter.
|
||||
*
|
||||
* @param bool $enabled True if theme update notifications are enabled, false otherwise.
|
||||
* @param array $update_results The results of theme update tasks.
|
||||
*/
|
||||
$notifications_enabled = apply_filters( 'auto_theme_update_send_email', true, $update_results['theme'] );
|
||||
|
||||
if ( ! empty( $update_results['theme'] ) && $notifications_enabled ) {
|
||||
foreach ( $update_results['theme'] as $update_result ) {
|
||||
if ( true === $update_result->result ) {
|
||||
$successful_updates['theme'][] = $update_result;
|
||||
} else {
|
||||
$failed_updates['theme'][] = $update_result;
|
||||
if ( $notifications_enabled ) {
|
||||
foreach ( $update_results['theme'] as $update_result ) {
|
||||
if ( true === $update_result->result ) {
|
||||
$successful_updates['theme'][] = $update_result;
|
||||
} else {
|
||||
$failed_updates['theme'][] = $update_result;
|
||||
}
|
||||
}
|
||||
}
|
||||
}
|
||||
|
Loading…
Reference in New Issue
Block a user