Skip to content

Commit

Permalink
Options: Show error when updating all options exceeds max_input_vars.
Browse files Browse the repository at this point in the history
On sites with a large number of options, the number of inputs when saving all options via the /wp-admin/options.php page may exceed PHP's `max_input_vars` setting, this will prevent saving.

An error is now shown if this is the case.

Props ishitaka, jorbin, maguijo, pento, sergeybiryukov. 
Fixes #45438.


git-svn-id: https://develop.svn.wordpress.org/trunk@53151 602fd350-edb4-49c9-b593-d223f7449a82
  • Loading branch information
peterwilsoncc authored and adamziel committed Apr 12, 2022
1 parent 8800abc commit e1ca76e
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion src/wp-admin/options.php
Original file line number Diff line number Diff line change
Expand Up @@ -255,7 +255,7 @@
if ( is_multisite() && ! current_user_can( 'manage_network_options' ) ) {
wp_die( __( 'Sorry, you are not allowed to modify unregistered settings for this site.' ) );
}
$options = explode( ',', wp_unslash( $_POST['page_options'] ) );
$options = isset( $_POST['page_options'] ) ? explode( ',', wp_unslash( $_POST['page_options'] ) ) : null;
} else {
$options = $allowed_options[ $option_page ];
}
Expand Down Expand Up @@ -332,6 +332,8 @@
if ( $user_language_old !== $user_language_new ) {
load_default_textdomain( $user_language_new );
}
} else {
add_settings_error( 'general', 'settings_updated', __( 'Settings save failed.' ), 'error' );
}

/*
Expand Down

0 comments on commit e1ca76e

Please sign in to comment.