Skip to content

Commit

Permalink
implement suggestions
Browse files Browse the repository at this point in the history
  • Loading branch information
wojsmol committed Apr 20, 2019
1 parent b833f0f commit 6a626dc
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 3 deletions.
3 changes: 2 additions & 1 deletion phpcs.xml.dist
Original file line number Diff line number Diff line change
Expand Up @@ -50,8 +50,9 @@
</property>
</properties>
</rule>

<!-- Exclude existing classes from the prefix rule as it would break BC to prefix them now. -->
<rule ref="WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedClassFound">
<exclude-pattern>*/src/Super_Admin_Command\.php$</exclude-pattern>
</rule>
</ruleset>
</ruleset>
5 changes: 3 additions & 2 deletions src/Super_Admin_Command.php
Original file line number Diff line number Diff line change
@@ -1,4 +1,5 @@
<?php
use WP_CLI\Fetchers\User as UserFetcher;

/**
* Lists, adds, or removes super admin users on a multisite installation.
Expand Down Expand Up @@ -27,7 +28,7 @@ class Super_Admin_Command extends WP_CLI_Command {
);

public function __construct() {
$this->fetcher = new \WP_CLI\Fetchers\User();
$this->fetcher = new UserFetcher();
}

/**
Expand Down Expand Up @@ -57,7 +58,7 @@ public function __construct() {
*
* @subcommand list
*/
public function list_( $_, $assoc_args ) {
public function list_subcommand( $_, $assoc_args ) {
$super_admins = self::get_admins();

if ( 'list' === $assoc_args['format'] ) {
Expand Down

0 comments on commit 6a626dc

Please sign in to comment.