Skip to content

Refine logic in perflab_install_activate_plugin_callback() to rely only on validated slug #1170

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
mukeshpanchal27 merged 5 commits into from
Apr 24, 2024
Merged

Refine logic in perflab_install_activate_plugin_callback() to rely only on validated slug #1170

Changes from 1 commit
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
b674db8
Use plugin slug exclusively for plugin install
westonruter Apr 23, 2024
04b132d
Use plugin_file instead of misleading plugin_baseame
westonruter Apr 23, 2024
6700d94
Remove redundant variable and add comments
westonruter Apr 23, 2024
5b87e85
Use perflab_get_standalone_plugin_data() instead of parsing plugins.json
westonruter Apr 23, 2024
efe9711
Reuse perflab_get_standalone_plugins()
westonruter Apr 23, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Remove redundant variable and add comments
  • Loading branch information
@westonruter
westonruter committed Apr 23, 2024
commit 6700d944be7f7794b877b43d779da9a485f25293
13 changes: 6 additions & 7 deletions includes/admin/load.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -248,19 +248,18 @@ function perflab_install_activate_plugin_callback() {
wp_die( esc_html__( 'Invalid plugin.', 'performance-lab' ) );
}

// Check if installed and determine the plugin basename.
$is_plugin_installed = false;
$plugin_file = null;
// Check if plugin (by slug) is installed by obtaining the plugin file.
// Remember a plugin file typically looks like "{slug}/load.php" or "{slug}/{slug}.php".
$plugin_file = null;
foreach ( array_keys( get_plugins() ) as $installed_plugin_file ) {
if ( strtok( $installed_plugin_file, '/' ) === $plugin_slug ) {
$is_plugin_installed = true;
$plugin_file = $installed_plugin_file;
$plugin_file = $installed_plugin_file;
break;
}
}

// Install the plugin if it is not installed yet.
if ( ! $is_plugin_installed ) {
// Install the plugin if it is not installed yet (in which case the plugin file could not be discovered above).
if ( ! isset( $plugin_file ) ) {
// Check if the user have plugin installation capability.
if ( ! current_user_can( 'install_plugins' ) ) {
wp_die( esc_html__( 'Sorry, you are not allowed to install plugins on this site.', 'default' ) );
Expand Down