Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
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
26 changes: 26 additions & 0 deletions includes/Admin/Admin_Page.php
Original file line number Diff line number Diff line change
Expand Up @@ -21,6 +21,7 @@ class Admin_Page {
*/
public function add_hooks() {
add_action( 'admin_menu', array( $this, 'add_page' ) );
add_filter( 'plugin_action_links', array( $this, 'filter_plugin_action_links' ), 10, 2 );
}

/**
Expand Down Expand Up @@ -69,6 +70,31 @@ private function get_available_plugins() {
public function render_page() {
$available_plugins = $this->get_available_plugins();

$selected_plugin_basename = filter_input( INPUT_GET, 'plugin', FILTER_SANITIZE_STRING );

require WP_PLUGIN_CHECK_PLUGIN_DIR_PATH . '/templates/admin-page.php';
}

/**
* Adds "check this plugin" link in the plugins list table.
*
* @since n.e.x.t
*
* @param array $actions List of actions.
* @param string $plugin_file Plugin main file.
* @return array The modified list of actions.
*/
public function filter_plugin_action_links( $actions, $plugin_file ) {

if ( current_user_can( 'activate_plugins' ) ) {

$actions[] = sprintf(
'<a href="%1$s">%2$s</a>',
esc_url( admin_url() . 'tools.php?page=plugin-check&plugin=' . $plugin_file ),
esc_html__( 'Check this plugin', 'plugin-check' )
);
Comment thread
felixarntz marked this conversation as resolved.
}

return $actions;
}
}
2 changes: 1 addition & 1 deletion templates/admin-page.php
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@
<select id="plugin-check__plugins" name="plugin_check_plugins">
<option><?php esc_html_e( 'Select Plugin', 'plugin-check' ); ?></option>
<?php foreach ( $available_plugins as $plugin_basename => $available_plugin ) { ?>
<option value="<?php echo esc_attr( $plugin_basename ); ?>">
<option value="<?php echo esc_attr( $plugin_basename ); ?>"<?php selected( $selected_plugin_basename, $plugin_basename ); ?>>
<?php echo esc_html( $available_plugin['Name'] ); ?>
</option>
<?php } ?>
Expand Down
27 changes: 27 additions & 0 deletions tests/Admin/Admin_Page_Tests.php
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,7 @@ public function set_up() {
public function test_add_hooks() {
$this->admin_page->add_hooks();
$this->assertEquals( 10, has_action( 'admin_menu', array( $this->admin_page, 'add_page' ) ) );
$this->assertEquals( 10, has_filter( 'plugin_action_links', array( $this->admin_page, 'filter_plugin_action_links' ) ) );
}

public function test_add_page() {
Expand Down Expand Up @@ -118,4 +119,30 @@ public function test_render_page_with_no_plugins() {
$this->assertStringContainsString( 'No plugins available.', $output );
$this->assertStringNotContainsString( '<select id="plugin-check__plugins"', $output );
}

public function test_filter_plugin_action_links() {

$base_file = plugin_basename( WP_PLUGIN_CHECK_MAIN_FILE );

$action_links = $this->admin_page->filter_plugin_action_links( array(), $base_file );
$this->assertEmpty( $action_links );

/** Administrator check */
$admin_user = self::factory()->user->create( array( 'role' => 'administrator' ) );

if ( is_multisite() ) {
grant_super_admin( $admin_user );
}
wp_set_current_user( $admin_user );
$action_links = $this->admin_page->filter_plugin_action_links( array(), $base_file );

$this->assertEquals(
sprintf(
'<a href="%1$s">%2$s</a>',
esc_url( admin_url() . 'tools.php?page=plugin-check&plugin=' . $base_file ),
esc_html__( 'Check this plugin', 'plugin-check' )
),
$action_links[0]
);
}
Comment thread
felixarntz marked this conversation as resolved.
}