Skip to content

Commit 5b3ee01

Browse files
Jetpack: Tidy up some logging calls, and add them to the WP-CLI code.
1 parent d9e75b8 commit 5b3ee01

File tree

2 files changed

+6
-2
lines changed

2 files changed

+6
-2
lines changed

class.jetpack-cli.php

+4
Original file line numberDiff line numberDiff line change
@@ -85,11 +85,13 @@ public function disconnect( $args, $assoc_args ) {
8585

8686
switch ( $action ) {
8787
case 'blog':
88+
Jetpack::log( 'disconnect' );
8889
Jetpack::disconnect();
8990
WP_CLI::success( __( 'Jetpack has been successfully disconnected.', 'jetpack' ) );
9091
break;
9192
case 'user':
9293
if ( Jetpack::unlink_user( $user->ID ) ) {
94+
Jetpack::log( 'unlink', $user->ID );
9395
WP_CLI::success( sprintf( __( '%s has been successfully disconnected.', 'jetpack' ), $action ) );
9496
} else {
9597
WP_CLI::error( sprintf( __( '%s could not be disconnected. Are you sure they\'re connected currently?', 'jetpack' ), "{$user->login} <{$user->email}>" ) );
@@ -156,11 +158,13 @@ public function module( $args, $assoc_args ) {
156158
break;
157159
case 'activate':
158160
$module = Jetpack::get_module( $module_slug );
161+
Jetpack::log( 'activate', $module_slug );
159162
Jetpack::activate_module( $module_slug, false );
160163
WP_CLI::success( sprintf( __( '%s has been activated.', 'jetpack' ), $module['name'] ) );
161164
break;
162165
case 'deactivate':
163166
$module = Jetpack::get_module( $module_slug );
167+
Jetpack::log( 'deactivate', $module_slug );
164168
Jetpack::deactivate_module( $module_slug );
165169
WP_CLI::success( sprintf( __( '%s has been deactivated.', 'jetpack' ), $module['name'] ) );
166170
break;

class.jetpack.php

+2-2
Original file line numberDiff line numberDiff line change
@@ -2036,7 +2036,7 @@ function admin_page_load() {
20362036

20372037
$module = stripslashes( $_GET['module'] );
20382038
check_admin_referer( "jetpack_activate-$module" );
2039-
Jetpack::log( 'activate' );
2039+
Jetpack::log( 'activate', $module );
20402040
Jetpack::activate_module( $module );
20412041
// The following two lines will rarely happen, as Jetpack::activate_module normally exits at the end.
20422042
wp_safe_redirect( Jetpack::admin_url( 'page=jetpack' ) );
@@ -2081,8 +2081,8 @@ function admin_page_load() {
20812081

20822082
$modules = stripslashes( $_GET['module'] );
20832083
check_admin_referer( "jetpack_deactivate-$modules" );
2084-
Jetpack::log( 'deactivate' );
20852084
foreach ( explode( ',', $modules ) as $module ) {
2085+
Jetpack::log( 'deactivate', $module );
20862086
Jetpack::deactivate_module( $module );
20872087
Jetpack::state( 'message', 'module_deactivated' );
20882088
}

0 commit comments

Comments
 (0)