The variable $user does not seem to be defined for all execution paths leading up to this point.
If you define a variable conditionally, it can happen that it is not defined
for all execution paths.
Let’s take a look at an example:
functionmyFunction($a){switch($a){case'foo':$x=1;break;case'bar':$x=2;break;}// $x is potentially undefined here.echo$x;}
In the above example, the variable $x is defined if you pass “foo” or “bar”
as argument for $a. However, since the switch statement has no default
case statement, if you pass any other value, the variable $x would be undefined.
Available Fixes
Check for existence of the variable explicitly:
functionmyFunction($a){switch($a){case'foo':$x=1;break;case'bar':$x=2;break;}if(isset($x)){// Make sure it's always set.echo$x;}}
Define a default value for the variable:
functionmyFunction($a){$x='';// Set a default which gets overridden for certain paths.switch($a){case'foo':$x=1;break;case'bar':$x=2;break;}echo$x;}
Add a value for the missing path:
functionmyFunction($a){switch($a){case'foo':$x=1;break;case'bar':$x=2;break;// We add support for the missing case.default:$x='';break;}echo$x;}
Loading history...
153
WP_CLI::success( __( 'User has been successfully disconnected.', 'jetpack' ) );
154
} else {
155
/* translators: %s is a username */
156
WP_CLI::error( sprintf( __( "User %s could not be disconnected. Are you sure they're connected currently?", 'jetpack' ), "{$user->login} <{$user->email}>" ) );
157
}
158
break;
159
case 'prompt':
160
WP_CLI::error( __( 'Please specify if you would like to disconnect a blog or user.', 'jetpack' ) );
161
break;
162
}
163
}
164
165
/**
166
* Reset Jetpack options and settings to default
167
*
168
* ## OPTIONS
169
*
170
* modules: Resets modules to default state ( get_default_modules() )
171
*
172
* options: Resets all Jetpack options except:
173
* - All private options (Blog token, user token, etc...)
174
* - id (The Client ID/WP.com Blog ID of this site)
WP_CLI::error( sprintf( __( '%s is not a valid command.', 'jetpack' ), $action ) );
356
}
357
// Check if module is active
358
if ( ! Jetpack::is_module_active( __FUNCTION__ ) ) {
359
WP_CLI::error( sprintf( _x( '%s is not active. You can activate it with "wp jetpack module activate %s"', '"wp jetpack module activate" is a command - do not translate', 'jetpack' ), __FUNCTION__, __FUNCTION__ ) );
360
}
361
if ( in_array( $action, array( 'whitelist' ) ) ) {
WP_CLI::success( sprintf( __( '%s has been whitelisted.', 'jetpack' ), $new_ip ) );
444
break;
445
case 'prompt':
446
WP_CLI::error(
447
__( 'No command found.', 'jetpack' ) . "\n" .
448
__( 'Please enter the IP address you want to whitelist.', 'jetpack' ) . "\n" .
449
_x( 'You can save a range of IPs {low_range}-{high_range}. No spaces allowed. (example: 1.1.1.1-2.2.2.2)', 'Instructions on how to whitelist IP ranges - low_range/high_range should be translated.', 'jetpack' ) . "\n" .
450
_x( "You can also 'list' or 'clear' the whitelist.", "'list' and 'clear' are commands and should not be translated", 'jetpack' ) . "\n"
451
);
452
break;
453
}
454
}
455
456
/**
457
* Manage Jetpack Options
458
*
459
* ## OPTIONS
460
*
461
* list : List all jetpack options and their values
462
* delete : Delete an option
463
* - can only delete options that are white listed.
WP_CLI::line( "\t" . str_pad( $option, 30 ) . 'Array - Use "get <option>" to read option array.' );
559
}
560
}
561
$option_text = '{' . _x( 'option', 'a variable command that a user can write, provided in the printed instructions', 'jetpack' ) . '}';
562
$value_text = '{' . _x( 'value', 'the value that they want to update the option to', 'jetpack' ) . '}';
563
564
WP_CLI::success(
565
_x( "Above are your options. You may 'get', 'delete', and 'update' them.", "'get', 'delete', and 'update' are commands - do not translate.", 'jetpack' ) . "\n" .
Automattic /
jetpack
georgestephanis
If you define a variable conditionally, it can happen that it is not defined for all execution paths.
Let’s take a look at an example:
In the above example, the variable $x is defined if you pass “foo” or “bar” as argument for $a. However, since the switch statement has no default case statement, if you pass any other value, the variable $x would be undefined.
Available Fixes
Check for existence of the variable explicitly:
Define a default value for the variable:
Add a value for the missing path: